[aur-general] [Review request] chrontel-wetab-rpm

[Florian Pritz]

On 08.01.2012 10:23, Markus Unterwaditzer wrote:
> Could somebody look at this package
> 
> http://aur.archlinux.org/packages.php?ID=55619
> 
> and tell me if the PKGBUILD is okay and "standards compilant"?

You could replace the version in _rpmname with $pkgver so it's easier to
update.

Why is $arch x86_64 only and not i686? Does it fail to build on i686?

Use install -Dm755 in package() when you are installing binaries.
-D will create the target directory automatically so you don't need
mkdir and -m755 will ensure that the permissions are correct.

> cp "${srcdir}/${_rpmextractname}/resources/fw7036.bin" "${pkgdir}/lib/firmware/chrontel"

If you do something like this please either add a slash at the end so
it's obvious that chrontel is a directory or just add the whole filename.

Looks good otherwise.

-- 
Florian Pritz

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://mailman.archlinux.org/pipermail/aur-general/attachments/20120108/1631a1d2/attachment.asc>