[aur-general] Review request for a new PKGBUILD
Hi AUR List! We would like to introduce a new package to the AUR. You are welcome to check out more about it on http://rbld.io Please notice, that this app is in the very early development stages, so any feedback is welcomed! ;) I am a long time Arch user, but this is my first PKGBUILD. I'd love to hear comments, if any, before I'll actually submit it to the AUR. Thanks a lot! Leonid. File below: ~~~~~~~~~ # Package maintainer: Leonid B <leonid at daynix dot com> # Upstream contact: rbld@rbld.io pkgname=rbld-git pkgver=r4.96410ab pkgrel=1 pkgdesc="Zero-dependency, reproducible build environments" arch=('any') url="http://rbld.io" license=('Apache') depends=('bash' 'docker') makedepends=('git') provides=('rbld') conflicts=('rbld') backup=('usr/etc/rebuild.conf') source=("${pkgname%-git}::git+https://github.com/daynix/rebuild") md5sums=('SKIP') pkgver() { cd "$srcdir/${pkgname%-git}" printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)" } package() { cd "$srcdir/${pkgname%-git}/cli" make prefix="$pkgdir/usr/" install }
Hi! There's just one thing that stood out to me: Why are you putting configuration files in `/usr/etc/` rather than `/etc/`? (See [1]) Other than that, your PKGBUILD looks just fine to me. Welcome aboard! :) [1]: https://wiki.archlinux.org/index.php/Arch_packaging_standards#Directories /Emil On Sun, 3 Jul 2016, 20:12 Leonid Bloch, <leonid@daynix.com> wrote:
Hi AUR List!
We would like to introduce a new package to the AUR. You are welcome to check out more about it on http://rbld.io
Please notice, that this app is in the very early development stages, so any feedback is welcomed! ;)
I am a long time Arch user, but this is my first PKGBUILD. I'd love to hear comments, if any, before I'll actually submit it to the AUR.
Thanks a lot! Leonid.
File below: ~~~~~~~~~ # Package maintainer: Leonid B <leonid at daynix dot com> # Upstream contact: rbld@rbld.io pkgname=rbld-git pkgver=r4.96410ab pkgrel=1 pkgdesc="Zero-dependency, reproducible build environments" arch=('any') url="http://rbld.io" license=('Apache') depends=('bash' 'docker') makedepends=('git') provides=('rbld') conflicts=('rbld') backup=('usr/etc/rebuild.conf') source=("${pkgname%-git}::git+https://github.com/daynix/rebuild") md5sums=('SKIP')
pkgver() { cd "$srcdir/${pkgname%-git}" printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)" }
package() { cd "$srcdir/${pkgname%-git}/cli" make prefix="$pkgdir/usr/" install }
Hi, Emil - thanks for your review! After a discussion with the guys who are actively developing this app now, I'm not going to push it to the AUR just yet, but will rather wait until it will be more mature and will change less frequently. Cheers, Leonid. ___ On Thu, Jul 7, 2016 at 1:28 PM, Emil Lundberg via aur-general < aur-general@archlinux.org> wrote:
Hi!
There's just one thing that stood out to me: Why are you putting configuration files in `/usr/etc/` rather than `/etc/`? (See [1])
Other than that, your PKGBUILD looks just fine to me. Welcome aboard! :)
[1]: https://wiki.archlinux.org/index.php/Arch_packaging_standards#Directories
/Emil
On Sun, 3 Jul 2016, 20:12 Leonid Bloch, <leonid@daynix.com> wrote:
Hi AUR List!
We would like to introduce a new package to the AUR. You are welcome to check out more about it on http://rbld.io
Please notice, that this app is in the very early development stages, so any feedback is welcomed! ;)
I am a long time Arch user, but this is my first PKGBUILD. I'd love to hear comments, if any, before I'll actually submit it to the AUR.
Thanks a lot! Leonid.
File below: ~~~~~~~~~ # Package maintainer: Leonid B <leonid at daynix dot com> # Upstream contact: rbld@rbld.io pkgname=rbld-git pkgver=r4.96410ab pkgrel=1 pkgdesc="Zero-dependency, reproducible build environments" arch=('any') url="http://rbld.io" license=('Apache') depends=('bash' 'docker') makedepends=('git') provides=('rbld') conflicts=('rbld') backup=('usr/etc/rebuild.conf') source=("${pkgname%-git}::git+https://github.com/daynix/rebuild") md5sums=('SKIP')
pkgver() { cd "$srcdir/${pkgname%-git}" printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)" }
package() { cd "$srcdir/${pkgname%-git}/cli" make prefix="$pkgdir/usr/" install }
participants (2)
-
Emil Lundberg
-
Leonid Bloch