Mail archive
alpine-devel

Re: [alpine-devel] [PATCH 1/2] Initial commit of Netatalk Version 3.0.1

From: Natanael Copa <ncopa_at_alpinelinux.org>
Date: Wed, 31 Oct 2012 11:39:37 +0100

Thank you very much!

Some comments.

On Sun, 28 Oct 2012 21:21:53 +0100
Oliver Loch <grimeton_at_gmx.net> wrote:

> ---
> main/netatalk/APKBUILD | 80

We normally add new stuff to testing first, so we can let the build
servers build package and test that it is not completely broken before
we move it to main.

...
> +++ b/main/netatalk/APKBUILD
> _at_@ -0,0 +1,80 @@
> +# Contributor: Oliver Loch <grimeton_at_gmx.net>
> +# Maintainer:

And we would really only want maintained stuff in main.

...

> +package() {
> + cd "$_builddir"
> + make DESTDIR="$pkgdir" install || return 1
> + [ ! -d "$pkgdir/usr/" ] && mkdir "$pkgdir/usr/"
> + [ ! -d "$pkgdir/etc/netatalk/" ] && mkdir
> "$pkgdir/etc/netatalk/"

you can do mkdir -p "$pkgdir"/usr "$pkgdir"/etc/netatalk

but the make install should have done it really.

> + [ -d "$pkgdir/sbin/" ] && mv "$pkgdir/sbin/"
> "$pkgdir/usr/sbin/"
> + [ -d "$pkgdir/bin/" ] && mv "$pkgdir/bin/" "$pkgdir/usr/bin/"
> + [ -d "$pkgdir/lib/" ] && mv "$pkgdir/lib/" "$pkgdir/usr/lib/"
> + [ -f "$pkgdir/etc/afp.conf" ] && mv "$pkgdir/etc/afp.conf"
> "$pkgdir/etc/netatalk/afp.conf" +

Those should not be needed. If they are, the makefiles are broke and
should be patched. (and submitted upstream)

...

> diff --git a/main/netatalk/netatalk.pre-install
> b/main/netatalk/netatalk.pre-install new file mode 100644
> index 0000000..46079e0
> --- /dev/null
> +++ b/main/netatalk/netatalk.pre-install
> _at_@ -0,0 +1,4 @@
> +#!/bin/sh
> +
> +# add something which happends before install
> +

This file could be removed. It would generate an extra fork (=slow
down) during install for no reason.

Do you think you could fix up the mentioned issues and resend a single
new patch with the things fixed?

do something like:

# fix things and test that it builds
cp -r main/netatalk testing/netatalk
git add testing/netatalk
git commit -v

# as commit message you have something like:
testing/netatalk: new aport

<short descroption what it is>
<url>
# git send-email --to alpine-devel_at_lists.alpinelinux.org HEAD^


If you are familiar with git can ofcourse also do:
 git pull --rebase
 git mv main/netatalk testing/
 git commit --amend
 git rebase -i
# merge the commits into one and fix the commit message

and then resend it.

Thanks!


---
Unsubscribe:  alpine-devel+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-devel+help_at_lists.alpinelinux.org
---
Received on Wed Oct 31 2012 - 11:39:37 UTC