From owner-freebsd-rc@FreeBSD.ORG Mon Jan 10 19:03:58 2005 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id B380116A4CE; Mon, 10 Jan 2005 19:03:58 +0000 (GMT) Received: from niobe.ijs.si (mail.ijs.si [193.2.4.66]) by mx1.FreeBSD.org (Postfix) with ESMTP id B29B343D45; Mon, 10 Jan 2005 19:03:57 +0000 (GMT) (envelope-from dejan.lesjak@ijs.si) Received: from localhost (localhost.ijs.si [127.0.0.1]) by niobe.ijs.si (Postfix) with ESMTP id 474FF1DD496; Mon, 10 Jan 2005 20:03:56 +0100 (CET) Received: from niobe.ijs.si ([127.0.0.1]) by localhost (niobe.ijs.si [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 78372-02; Mon, 10 Jan 2005 20:03:38 +0100 (CET) Received: from metatron.ijs.si (metatron.ijs.si [193.2.4.152]) by niobe.ijs.si (Postfix) with ESMTP id 85C1D1DD4F5; Mon, 10 Jan 2005 20:03:37 +0100 (CET) Received: from idefix.ijs.si (idefix.ijs.si [193.2.4.33]) (using TLSv1 with cipher RC4-MD5 (128/128 bits)) (No client certificate requested) by metatron.ijs.si (Postfix) with ESMTP id 5D5131C0008A; Mon, 10 Jan 2005 20:03:36 +0100 (CET) From: Dejan Lesjak To: freebsd-x11@freebsd.org Date: Mon, 10 Jan 2005 20:03:34 +0100 User-Agent: KMail/1.7.2 References: <1105321614.8452.54.camel@leguin> <41E23F8F.4040701@redesjm.local> <1105382156.2497.6.camel@leguin> In-Reply-To: <1105382156.2497.6.camel@leguin> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Message-Id: <200501102003.35785.dejan.lesjak@ijs.si> X-Virus-Scanned: amavisd-new at ijs.si cc: freebsd-rc@freebsd.org cc: Eric Anholt cc: x11@freebsd.org cc: Jose M Rodriguez Subject: Re: x11 /tmp preparation rc.d script X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list 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, 10 Jan 2005 19:03:59 -0000 [rc@ list CCed as this threads on their territory, the start of thread is=20 here:=20 http://lists.freebsd.org/pipermail/freebsd-x11/2005-January/001474.html] On Monday 10 of January 2005 19:35, Eric Anholt wrote: > On Mon, 2005-01-10 at 09:40 +0100, Jose M Rodriguez wrote: > > Jose M Rodriguez escribi=F3: > > > Eric Anholt escribi=F3: > > >> Attached are my proposed patches to deal with the X11 ICE issue. To > > >> review, it's required because having .ICE not owned by root is a > > >> security issue, one that's been papered over with a printed warning > > >> and sleep(5) in libICE for years, and has recently been changed into > > >> an actual error by the X.Org folks. > > > > ... > > > > As a latter think about this, consider take also periodic related fixes > > (We clear this directories by default) and try to get a OS_VERSION bump > > closest to this. > > I'm sorry, I'm not sure what exactly you're talking about here. Are you > saying that /etc/periodic contains something that will wipe out X's > files in /tmp? That would be rather broken. /etc/periodic/daily/110.clean-tmps cleans out empty directories that have n= ot=20 been modified in $daily_clean_tmps_days days. This ones should therefore be= =20 added to $daily_clean_tmps_ignore in /etc/defaults/periodic.conf, just to b= e=20 on the safe side. Other than that, I don't really know what would be the best way to handle=20 creation of this directories and haven't commented so far, but since I'm=20 already writing (mostly because I thought rc@ list should be CCed), here's = my=20 opinion FWIW: the simplest seems to be a patch from Pawel Worach at=20 http://lists.freebsd.org/mailman/htdig/freebsd-current/2004-November/042445= =2Ehtml The benefit of using this simple approach is that it is simple (of course := )=20 and furthermore it only adds two more directories to /tmp at startup as=20 oposed to adding a file in /etc/rc.d. The difference being one inode. But=20 then again, perhaps I don't see all the implications and this is too simple= =2E=20 Is there a real benefit in creating another rc.d script for doing this and= =20 adding a knob to turn creation of these directories off? Yes of course that would only solve things on -current and -stable, however= =20 there is already an UPDATING entry for this and we can always add a script = to=20 be installed from a port that would take care of transition period (probabl= y=20 as soon in dependency tree as possible, ie -libraries). Dejan