X-Original-To: alpine-aports@mail.alpinelinux.org Delivered-To: alpine-aports@mail.alpinelinux.org Received: from mail.alpinelinux.org (dallas-a1.alpinelinux.org [127.0.0.1]) by mail.alpinelinux.org (Postfix) with ESMTP id BA96CDC03B8; Sat, 5 Sep 2015 11:15:57 +0000 (UTC) Received: from ncopa-laptop (unknown [79.160.13.130]) (using TLSv1 with cipher ECDHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: n@tanael.org) by mail.alpinelinux.org (Postfix) with ESMTPSA id 22FB2DC00DE; Sat, 5 Sep 2015 11:15:56 +0000 (UTC) Date: Sat, 5 Sep 2015 13:15:51 +0200 From: Natanael Copa To: =?UTF-8?B?U8O2cmVu?= Tempel Cc: alpine-aports@lists.alpinelinux.org Subject: Re: [alpine-aports] [PATCH] main/openntpd: fixes privelege seperation defaults Message-ID: <20150905131551.5453121d@ncopa-laptop> In-Reply-To: <20150903091752.GA3122@calcium.lan> References: <1440703097-6498-1-git-send-email-systmkor@gmail.com> <20150829101544.GA25793@francium.lan> <20150903091752.GA3122@calcium.lan> X-Mailer: Claws Mail 3.12.0 (GTK+ 2.24.28; x86_64-alpine-linux-musl) X-Mailinglist: alpine-aports Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Virus-Scanned: ClamAV using ClamSMTP On Thu, 3 Sep 2015 11:17:52 +0200 S=C3=B6ren Tempel wrote: > On 29.08.15, Orion Miller wrote: > > Okay, so what are you saying? Because I thought I wrote what you > > suggested in the last patch. And now you are suggesting to do what you > > said I should do last time. [1] >=20 > No, I suggested to create that directory in the OpenRC service not in > the package function. Did you even read your own reference? >=20 > On 29.08.15, Orion Miller wrote: > > If you look at the OpenSSH APKBUILD file it creates /var/empty in it's > > package function. So I'm not sure what you are actually suggesting. >=20 > Turns out: OpenSSH creates /var/empty in the package function and it > creates it again in the OpenRC service: >=20 > See: > - http://git.alpinelinux.org/cgit/aports/tree/main/openssh/sshd.initd#n= 39 > - http://git.alpinelinux.org/cgit/aports/tree/main/openssh/APKBUILD#n75 >=20 > I personally believe that this is a fault in the OpenSSH aport. Still > the question remains what the best way to create that directory is=E2=80= =A6I > personally dislike the creation of /var/empty in the package() function. I think it is perfectly fine that packages that uses a directory creates it. > Maybe we should simply create that in alpine-baselayout? I think we should and I just did. Thanks! -nc --- Unsubscribe: alpine-aports+unsubscribe@lists.alpinelinux.org Help: alpine-aports+help@lists.alpinelinux.org ---