[aur-general] PKGBUILD review request: understand-bin
Alex Sarum
rum.274.4 at gmail.com
Tue Jun 14 12:31:46 UTC 2016
> What's with the "scitools" variable? Doesn't seem to do anything useful.
Deleted.
> source and checksum arrays support architecture specific arrays now. Check the
> man page/wiki page.
I known about it, but I think that it will be shorter.
> You shouldn't be changing anything in $srcdir in the package function. What
> happens when someone runs makepkg --repackage?
Corrected.
> There's really no reason to change the name of the license file.
Corrected.
> "install -d "$pkgdir/usr/bin"" doesn't really do anything since you're using
> the -D option on the next line. That will create the dir.
Corrected.
> You should be consistent with how you use variables. Sometimes you have them in
> braces and sometimes you don't, even in the same path.
Corrected.
I can share my PKGBUILD now?
P.S.
I have not received a reply. I waited digest and copied ID message to
In-Reply-To. I hope this will works. So it should be?
And sorry for my EnGlIsH.
More information about the aur-general
mailing list