[aur-general] Request for comment on gnuplot-nox PKGBUILD
Hi, I recently adopted the gnuplot-nox ( https://aur.archlinux.org/packages/?O=0&K=gnuplot-nox) package and I would like to have your advices before publishing a new version of the PKGBUILD (this is my first package in AUR). I have read the wiki pages about arch packaging standard and AUR (eh! that is even why I am posting to the list) I mostly adapted the pkgbuild of the former maintainer so you should not fire me on the spot (I hope). PKGBUILD : http://pastebin.com/RGh7jPYA patch : http://pastebin.com/MwAQTqm2 Two main questions : - should the package "Provides" gnuplot or "Conflict" with gnuplot (from [extra]) ? - is it the right way to patch sources ? Regards, -- Jonathan Gibert Consultant www.iconsultants.fr
----------------------------------------
From: jokot3@gmail.com Date: Mon, 25 Mar 2013 19:13:18 +0100 To: aur-general@archlinux.org Subject: [aur-general] Request for comment on gnuplot-nox PKGBUILD
Hi,
I recently adopted the gnuplot-nox ( https://aur.archlinux.org/packages/?O=0&K=gnuplot-nox) package and I would like to have your advices before publishing a new version of the PKGBUILD (this is my first package in AUR).
I have read the wiki pages about arch packaging standard and AUR (eh! that is even why I am posting to the list) I mostly adapted the pkgbuild of the former maintainer so you should not fire me on the spot (I hope).
PKGBUILD : http://pastebin.com/RGh7jPYA patch : http://pastebin.com/MwAQTqm2
Two main questions : - should the package "Provides" gnuplot or "Conflict" with gnuplot (from [extra]) ? - is it the right way to patch sources ?
Regards, -- Jonathan Gibert Consultant www.iconsultants.fr
I would suggest taking a look at the official PKBUILD for ideas. Doing make install like that instead of using DESTDIR is kind of ridiculous, for example. https://projects.archlinux.org/svntogit/packages.git/tree/trunk?h=packages/g... For the question on provides and conflicts, it's not one or the other. Does your package conflict with gnuplot, ie contain the same files in the same place? Does your package provide gnuplot, ie is usable by other packages that may depend on gnuplot? Many (or even most) times, it's both. On the patch, that way is fine, although if you're just changing all occurrences of subsubsecion to subsecion, using sed in the PKGBUILD would be much simpler IMO. sed -i 's/subsubsection/subsection/g' docs/gnuplot.texi I didn't look at the file being patched, so if you're just changing certain ones, that won't work. As general notes, please quote anything that contains variables like $pkgdir and $srcdir, and the end of the configure command should not have a "\" after it ("--with-readline=gnu \" should be just "--with-readline=gnu")
Hi Doug, Thanks a lot for your remarks, I applied all the changes you were suggesting (http://pastebin.com/9XVgmytw). Regarding the patch, it only applies to the penultimate @section of the document, so your second supposition is right. Regards On Mon, Mar 25, 2013 at 7:52 PM, Doug Newgard <scimmia22@outlook.com> wrote:
From: jokot3@gmail.com Date: Mon, 25 Mar 2013 19:13:18 +0100 To: aur-general@archlinux.org Subject: [aur-general] Request for comment on gnuplot-nox PKGBUILD
Hi,
I recently adopted the gnuplot-nox ( https://aur.archlinux.org/packages/?O=0&K=gnuplot-nox) package and I would like to have your advices before publishing a new version of the PKGBUILD (this is my first package in AUR).
I have read the wiki pages about arch packaging standard and AUR (eh!
---------------------------------------- that
is even why I am posting to the list) I mostly adapted the pkgbuild of the former maintainer so you should not fire me on the spot (I hope).
PKGBUILD : http://pastebin.com/RGh7jPYA patch : http://pastebin.com/MwAQTqm2
Two main questions : - should the package "Provides" gnuplot or "Conflict" with gnuplot (from [extra]) ? - is it the right way to patch sources ?
Regards, -- Jonathan Gibert Consultant www.iconsultants.fr
I would suggest taking a look at the official PKBUILD for ideas. Doing make install like that instead of using DESTDIR is kind of ridiculous, for example.
https://projects.archlinux.org/svntogit/packages.git/tree/trunk?h=packages/g...
For the question on provides and conflicts, it's not one or the other. Does your package conflict with gnuplot, ie contain the same files in the same place? Does your package provide gnuplot, ie is usable by other packages that may depend on gnuplot? Many (or even most) times, it's both.
On the patch, that way is fine, although if you're just changing all occurrences of subsubsecion to subsecion, using sed in the PKGBUILD would be much simpler IMO. sed -i 's/subsubsection/subsection/g' docs/gnuplot.texi I didn't look at the file being patched, so if you're just changing certain ones, that won't work.
As general notes, please quote anything that contains variables like $pkgdir and $srcdir, and the end of the configure command should not have a "\" after it ("--with-readline=gnu \" should be just "--with-readline=gnu")
-- Jonathan Gibert Consultant www.iconsultants.fr
participants (2)
-
Doug Newgard
-
JoKoT3