From owner-freebsd-x11@FreeBSD.ORG Sun Jan 9 17:35:01 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 07E7116A4CE; Sun, 9 Jan 2005 17:35:01 +0000 (GMT) Received: from 212.106.238.57.adsl.jazztel.es (212.106.238.57.adsl.jazztel.es [212.106.238.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9143B43D48; Sun, 9 Jan 2005 17:34:59 +0000 (GMT) (envelope-from josemi@freebsd.jazztel.es) Received: from [192.168.254.16] (orion.redesjm.local [192.168.254.16]) j09HYqtq000750; Sun, 9 Jan 2005 18:34:55 +0100 (CET) (envelope-from freebsd@redesjm.local) Message-ID: <41E16B3C.30302@redesjm.local> Date: Sun, 09 Jan 2005 18:34:52 +0100 From: Jose M Rodriguez User-Agent: Mozilla Thunderbird 1.0 (X11/20050106) X-Accept-Language: es-es, es MIME-Version: 1.0 To: Brooks Davis References: <41DDC4F2.5090709@yahoo.com> <20050107003806.GA14003@odin.ac.hmc.edu> <41DE5242.4030606@redesjm.local> <20050107182024.GB30931@odin.ac.hmc.edu> In-Reply-To: <20050107182024.GB30931@odin.ac.hmc.edu> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 8bit X-AntiVirus: checked by AntiVir Milter (version: 1.1.0-3; AVE: 6.29.0.5; VDF: 6.29.0.31; host: antares.redesjm.local) 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: Sun, 09 Jan 2005 17:35:01 -0000 Brooks Davis escribió: >On Fri, Jan 07, 2005 at 10:11:30AM +0100, Jose M Rodriguez wrote: > > >>Brooks Davis escribió: >> >> >> >>>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 >>>=================================================================== >>>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 >>>=================================================================== >>>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="NO" # Linux binary compati >>>svr4_enable="NO" # SysVR4 emulation loaded at startup (or NO). >>>osf1_enable="NO" # Alpha OSF/1 emulation loaded at startup (or NO). >>>clear_tmp_enable="NO" # Clear /tmp at startup. >>>+clear_tmp_xdirs="/tmp/.X11-unix /tmp/.font-unix /tmp/.ICE-unix" >>>+ # Directories needed by X11 >>>ldconfig_insecure="NO" # Set to YES to disable ldconfig security >>>checks >>>ldconfig_paths="/usr/lib/compat /usr/X11R6/lib /usr/local/lib >>>/usr/local/lib/compat/pkg" >>> # shared library search paths >>> >>> >>> >>> >>> >>I recall putting this in a conf PR, try a follow-up. >> >>But I think your patch is a little bit wrong >> >>I never like the way X11 is taken by /etc/rc.d/cleartmp. none must be >>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 >>more), this must be implementing a new /etc/rc.d/clearx11tmp (this may >>be do in the main /etc/rc.d/cleartmp, like in sendmail), with all the bits: >>clear_x11tmp_enable, clear_x11tmp_dirs, ... >> >>But I must point that: >> >>X11 is now mostly a ports thing, not a base system component. If this >>can be take from ports (I send-pr this also), this must be the path to >>the solution. I put a simple script from libs, but I can work and rcNG >>enabled thing if prefered. >> >>This is not what x11 really needs. x11 only needs some like this: mkdir >>-p ... && chown root:wheel ... && chmod 01777 ... . this may be >>prefered by the x11 team. >> >>This can be taken both from base and ports without too much problem. >>The only secondary effect of this I know is that you may polite /tmp >>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 > > > You're molsty wrong. init launch gettys after rc has finished (and also localpkg). -- josemi