From owner-freebsd-rc@FreeBSD.ORG Mon May 20 16:40:01 2013 Return-Path: Delivered-To: freebsd-rc@smarthost.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 814159AE for ; Mon, 20 May 2013 16:40:01 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id 7391760F for ; Mon, 20 May 2013 16:40:01 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.7/8.14.7) with ESMTP id r4KGe16k058304 for ; Mon, 20 May 2013 16:40:01 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.7/8.14.7/Submit) id r4KGe1iZ058303; Mon, 20 May 2013 16:40:01 GMT (envelope-from gnats) Date: Mon, 20 May 2013 16:40:01 GMT Message-Id: <201305201640.r4KGe1iZ058303@freefall.freebsd.org> To: freebsd-rc@FreeBSD.org Cc: From: Daniel Shahaf Subject: Re: conf/178760: rc.d/jail: set jname at jail start time X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: Daniel Shahaf List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 20 May 2013 16:40:01 -0000 The following reply was made to PR conf/178760; it has been noted by GNATS. From: Daniel Shahaf To: Miroslav Lachman <000.fbsd@quip.cz> Cc: bug-followup@FreeBSD.org Subject: Re: conf/178760: rc.d/jail: set jname at jail start time Date: Mon, 20 May 2013 17:34:20 +0300 Miroslav Lachman wrote on Sun, May 19, 2013 at 11:09:25 +0200: > I don't know if it's worth the change as you can already use > jail_jena_flags="-l -U root -n jena" > in you rc.conf. > True. However, flags set in rc.conf replace the default ones, not append to them, so we cut the middleman and patched our rc.d/jail locally, to ensure that if the default flags change, we won't miss that when we upgrade. Another reason for just patching rc.d/jail is that we want the -n flag for every jail. (That's also the reason we upstream the change.) > On the other side, the change is really simple and should be there for > years (from the first time the jail can have own name) > > The forthcoming new jail system with /etc/jail.conf will have all jails > named and there will be completely different rc.d/jail script for it. That's great. However, if jail.conf is scheduled to appear in 9.2, I think the patch still makes sense for 9.1.