Yes that helps, thanks Paul. On Sun, 17 Mar 2019 at 23:07, Paul Hentschel <[email protected]> wrote:
> On 3/17/19 5:29 AM, Martin Wilson wrote: > > I've put together a package for upload to the AUR, but since it's my > first, > > I would like to know if there are any problems with it: > > > > https://github.com/martin3141/tarquin_arch > > > > cheers, > > > > Martin > > > > Hello Martin. Here are my notes. Please note that this is my first > review, so I would wait to hear from some of the more experienced > community members to let you know if I get anything wrong. > > Quote all variables. > > Prepend $pkgname-$pkgver.tar.gz:: to source to rename the source tarball: > "$pkgname-$pkgver.tar.gz:: > https://github.com/martin3141/tarquin/archive/v$pkgver.tar.gz"; > Add CMakeListsPatch.txt to source array. > Don't forget to update the checksums after adding patch to source array. > > Change patch line in prepare() function from relative path to > "$srcdir"/CMakeListsPatch.txt > Also in the prepare() function, I would rather see proto.sh incorporated > in the PKGBUILD directly. It is only 3 lines. > > The package() function can be shortened to one install line: > install -Dm755 "$srcdir/$pkgname-$pkgver/src/build/redist/"tarquin{,gui} > -t "$pkgdir"/usr/bin > > Add a .gitignore to the repository. I prefer to exclude everything and > explicitly add what is needed. Others prefer to explicitly exclude > instead. My default .gitignore for AUR looks like this: > * > !PKGBUILD > !.SRCINFO > !.gitignore > > To that you would need to add !CMakeListsPatch.txt for this package. > > I hope this is helpful. > Paul > >
