Mail archive
alpine-devel

Re: [alpine-devel] [PATCH] sems: create var/run/sems dir on install

From: Natanael Copa <ncopa_at_alpinelinux.org>
Date: Mon, 2 May 2011 08:19:15 +0200

On Mon, 2 May 2011 12:43:26 +0000
Nathan Angelacos <nangel_at_alpinelinux.org> wrote:

> ---
> testing/sems/APKBUILD | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/testing/sems/APKBUILD b/testing/sems/APKBUILD
> index 49a61ad..d9cb870 100644
> --- a/testing/sems/APKBUILD
> +++ b/testing/sems/APKBUILD
> _at_@ -76,6 +76,7 @@ package() {
> rm -f "$pkgdir"/etc/sems/etc/conf_auth.conf
>
> install -o sems -g sems -d "$pkgdir"/var/spool/voicebox
> + install -d -o sems "$pkgdir"/var/run/sems
> }
>
> _mv_sub() {

Is this really needed? I think we create the dir from init.d script in
case /var/run happens to be a tmpfs mount (yeah, we probably need a
major cleanup in lots of init.d scripts to get that actually working)

Secondly, always bump pkgrel when contents of package changes.
Otherwise it will not be rebuilt. And even if it would, people who has
it installed will not have it upgraded unless pkgver/pkgrel changes.

I applied it anyways and bumped pkgrel.

Thanks!

-nc


---
Unsubscribe:  alpine-devel+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-devel+help_at_lists.alpinelinux.org
---
Received on Mon May 02 2011 - 08:19:15 UTC