On Fri, Feb 4, 2011 at 09:37, Dave Reisner wrote:

On Fri, Feb 04, 2011 at 09:07:07AM -0500, Adam Weiss wrote:

...

Hey all, I've been lurking for quite a while and recently decided to take the plunge and maintain a couple packages.

Please give me some feedback on them since I probably missed some things: http://aur.archlinux.org/packages.php?ID=34191 http://aur.archlinux.org/packages.php?ID=45976

Also, this is the first time I've tried posting from Gmail, so let me know if something looks funny.

Thanks, -Adam

in openscenegraph-svn:

...

if [ $_downloadsampledata == 'True' ] ; then

this is invalid syntax, as == is not a valid operator for [. makepkg uses bash, so please use the keyword [[ instead of the builtin [.

I disagree with your decision to do checkouts in the package() function. Do this in the build function instead. package() should do strictly that. package.

Neither of these packages arch of arch 'any'. 'any' means that nothing is compiled, as is usually the case with an interpreted language like perl or python. Both these packages compile code, and so they are architecture specific. Please specify this as arch=('i686' 'x86_64').

optdepends are malformed in both PKGBUILDs. Please follow the format of 'depend: description' stated in PKGBUILD(5).

Your usage of _exampledir in both PKGBUILDs is a little strange -- hard code the directory path. It's not likely going to change, and it makes the process a little more readable.

dave

Thanks for the advice Dave, I'll make these changes tonight. -Adam

On Fri, Feb 4, 2011 at 13:47, Vojtech Horky wrote:

2011/2/4 Dave Reisner :

...

On Fri, Feb 04, 2011 at 09:07:07AM -0500, Adam Weiss wrote:

...

Hey all, ...

in openscenegraph-svn:

...

if [ $_downloadsampledata == 'True' ] ; then

this is invalid syntax, as == is not a valid operator for [. makepkg uses bash, so please use the keyword [[ instead of the builtin [.

Just a thought: how about assigning _downloadsampledata=true (the executable /bin/true or /bin/false) and then making the "if" even shorter and maybe more readable: if $_downloadsampledata; then ... fi

- Vojta

Vojta, I'll have to try it out a bit, the use for 'True' is that it can be used directly by CMake that way, I'll have to see what it does if bash boolean is printed out for the value. Thanks, -Adam