From owner-freebsd-x11@FreeBSD.ORG Fri Jan 7 18:18:30 2005 Return-Path: Delivered-To: freebsd-x11@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id B445F16A4CE; Fri, 7 Jan 2005 18:18:30 +0000 (GMT) Received: from odin.ac.hmc.edu (Odin.AC.HMC.Edu [134.173.32.75]) by mx1.FreeBSD.org (Postfix) with ESMTP id 77E1743D49; Fri, 7 Jan 2005 18:18:30 +0000 (GMT) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (localhost.localdomain [127.0.0.1]) by odin.ac.hmc.edu (8.13.0/8.13.0) with ESMTP id j07IKPQE001748; Fri, 7 Jan 2005 10:20:25 -0800 Received: (from brdavis@localhost) by odin.ac.hmc.edu (8.13.0/8.13.0/Submit) id j07IKOqt001747; Fri, 7 Jan 2005 10:20:24 -0800 Date: Fri, 7 Jan 2005 10:20:24 -0800 From: Brooks Davis To: Jose M Rodriguez Message-ID: <20050107182024.GB30931@odin.ac.hmc.edu> References: <41DDC4F2.5090709@yahoo.com> <20050107003806.GA14003@odin.ac.hmc.edu> <41DE5242.4030606@redesjm.local> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="KFztAG8eRSV9hGtP" Content-Disposition: inline In-Reply-To: <41DE5242.4030606@redesjm.local> User-Agent: Mutt/1.4.1i X-Virus-Scanned: by amavisd-new X-Spam-Status: No, hits=0.0 required=8.0 tests=none autolearn=no version=2.63 X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on odin.ac.hmc.edu cc: Rob cc: freebsd-current cc: x11@freebsd.org Subject: Re: Xorg ICE vs. Xfce4 (4.2-RC3) needs fixing /etc/rc.d/cleartmp X-BeenThere: freebsd-x11@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: X11 on FreeBSD -- maintaining and support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 07 Jan 2005 18:18:30 -0000 --KFztAG8eRSV9hGtP Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Jan 07, 2005 at 10:11:30AM +0100, Jose M Rodriguez wrote: > Brooks Davis escribi=F3: >=20 > >Could you please try the following patch? It does the same thing, but > >gives the inode paranoid a way to disable the creation of these > >directories or only create the ones they need. > > > >-- Brooks > > > >Index: rc.d/cleartmp > >=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > >RCS file: /usr/cvs/src/etc/rc.d/cleartmp,v > >retrieving revision 1.11 > >diff -u -p -r1.11 cleartmp > >--- rc.d/cleartmp 7 Oct 2004 13:55:25 -0000 1.11 > >+++ rc.d/cleartmp 7 Jan 2005 00:31:51 -0000 > >@@ -35,5 +35,7 @@ run_rc_command "$1" > ># restarting X > ># > >rm -f /tmp/.X[0-9]-lock > >-rm -fr /tmp/.X11-unix > >-mkdir -m 1777 /tmp/.X11-unix > >+if [ -n ${clear_tmp_xdirs} ]; then > >+ rm -fr ${clear_tmp_xdirs} > >+ mkdir -m 1777 ${clear_tmp_xdirs} > >+fi > >Index: defaults/rc.conf > >=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > >RCS file: /usr/cvs/src/etc/defaults/rc.conf,v > >retrieving revision 1.235 > >diff -u -p -r1.235 rc.conf > >--- defaults/rc.conf 15 Dec 2004 12:39:28 -0000 1.235 > >+++ defaults/rc.conf 7 Jan 2005 00:30:49 -0000 > >@@ -443,6 +443,8 @@ linux_enable=3D"NO" # Linux binary compati > >svr4_enable=3D"NO" # SysVR4 emulation loaded at startup (or NO). > >osf1_enable=3D"NO" # Alpha OSF/1 emulation loaded at startup (or NO). > >clear_tmp_enable=3D"NO" # Clear /tmp at startup. > >+clear_tmp_xdirs=3D"/tmp/.X11-unix /tmp/.font-unix /tmp/.ICE-unix" > >+ # Directories needed by X11 > >ldconfig_insecure=3D"NO" # Set to YES to disable ldconfig security=20 > >checks > >ldconfig_paths=3D"/usr/lib/compat /usr/X11R6/lib /usr/local/lib=20 > >/usr/local/lib/compat/pkg" > > # shared library search paths > > > >=20 > > > I recall putting this in a conf PR, try a follow-up. >=20 > But I think your patch is a little bit wrong >=20 > I never like the way X11 is taken by /etc/rc.d/cleartmp. none must be= =20 > do after the run_rc_command. That's easy enough to fix. > If we need do this from the base system (Thing that I doubt more and=20 > more), this must be implementing a new /etc/rc.d/clearx11tmp (this may=20 > be do in the main /etc/rc.d/cleartmp, like in sendmail), with all the bit= s: > clear_x11tmp_enable, clear_x11tmp_dirs, ... >=20 > But I must point that: >=20 > X11 is now mostly a ports thing, not a base system component. If this=20 > can be take from ports (I send-pr this also), this must be the path to=20 > the solution. I put a simple script from libs, but I can work and rcNG= =20 > enabled thing if prefered. >=20 > This is not what x11 really needs. x11 only needs some like this: mkdir= =20 > -p ... && chown root:wheel ... && chmod 01777 ... . this may be=20 > prefered by the x11 team. >=20 > This can be taken both from base and ports without too much problem. =20 > The only secondary effect of this I know is that you may polite /tmp=20 > entries twice, with is not a real pain to the whole boot process. My worry with using your patch is that localpkg is run quite late in the startup process, well after X may have tried to start if xdm is run from /etc/ttys. If package startup scripts could run anywhere in the order, a script in the port would be obvious solution, but that's not the case. -- Brooks --=20 Any statement of the form "X is the one, true Y" is FALSE. PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4 --KFztAG8eRSV9hGtP Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) iD8DBQFB3tLnXY6L6fI4GtQRAkigAJ4l01FAKaCjWsHMn82qhZhwChR+UACgu/nv UFxzjSplFIUxUraQbqznXoI= =ivxQ -----END PGP SIGNATURE----- --KFztAG8eRSV9hGtP--