Date: Tue, 22 Jul 2014 22:50:45 -0700
From: Isaac Dunham <ibid.ag@gmail.com>
To: Timo Teras <timo.teras@iki.fi>
Cc: Natanael Copa <ncopa@alpinelinux.org>,
	William Pitcock <nenolod@dereferenced.org>,
	alpine-devel@lists.alpinelinux.org
Subject: Re: [alpine-devel] [PATCH] testing/sword: add new package
Message-ID: <20140723055044.GA17046@newbook>
References: <1405457044-3186-1-git-send-email-ibid.ag@gmail.com>
 <20140715162818.0f5adca9@ncopa-desktop.alpinelinux.org>
 <CA+T2pCEbZcdwnXgZfUKLqBWc20MAmj-FtOqdvkuEx=UnR-2A_g@mail.gmail.com>
 <20140716212248.GA1780@newbook>
 <20140722142728.6a7efbbe@ncopa-desktop.alpinelinux.org>
 <20140723040141.GA3905@newbook>
 <20140723081936.2ca2eb87@vostro>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20140723081936.2ca2eb87@vostro>
User-Agent: Mutt/1.5.23 (2014-03-12)

On Wed, Jul 23, 2014 at 08:19:36AM +0300, Timo Teras wrote:
> On Tue, 22 Jul 2014 21:01:42 -0700
> Isaac Dunham <ibid.ag@gmail.com> wrote:
> 
> > On Tue, Jul 22, 2014 at 02:27:28PM +0200, Natanael Copa wrote:
> > > How about using /var/lib/sword as default and have a
> > > 'sword' group with write permissions there?
> > > 
> > > They you can just add your users to the sword group to be able to
> > > install the data files.
> > > 
> > That should work.
> > One question/minor issue:
> > 
> > abuild warns about chown in the post-install script.
> > 
> > But it would seem that there's no other sane way to make sure that
> > /var/lib/sword belongs to the "sword" group: to make it belong to
> > root:sword in the package, this group must exist when the package is
> > built and when it is extracted.
> > I presume pre-install could do the latter, but manipulating groups in
> > a build script is insanity.
> > 
> > If I can ignore that warning (assuming it belongs in abuild), I have a
> > script ready to go.
> > 
> > What's the preferred approach?
> 
> Use pkgusers and pkggroups in APKBUILD, and do chown in the package()
> or subpkg split function. See 'transmission' or any other package with
> pkgusers in it for an example. And create the user in pre-install
> script.
> 
> This allows apk to keep track of the file permissions properly.
> Doing chown in post-install would make the file permissions mismatch
> with what is in the apk database, and thus make 'apk audit' give
> spurious results.

Ok, thanks for the explanation. Patch should be incoming as soon as I've
built and tested it.

...
I might drop clucene as a build dependency, since sword seems to pick up
the libc regex interface. But I'd rather not do that right now.

Thanks,
Isaac Dunham



---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---