Xorg ICE vs. Xfce4 (4.2-RC3) needs fixing /etc/rc.d/cleartmp

[Rob]

Brooks Davis wrote:
> On Fri, Jan 07, 2005 at 08:08:34AM +0900, Rob wrote:
> 
>>Hi,
>>
>>I'm testing Xfce 4.2-RC3, but it has following problems at startup:
>>
>>---------------- .xsession-errors
>>_IceTransmkdir: ERROR: euid != 0,directory /tmp/.ICE-unix will not be 
>>created.
>>_IceTransSocketUNIXCreateListener: mkdir(/tmp/.ICE-unix) failed, errno = 2
>>_IceTransMakeAllCOTSServerListeners: failed to create listener for local
>>xfce4-session: Unable to establish ICE listeners: Cannot establish any 
>>listening sockets
> 
> 
>>The patch below from Pawel Worach solves the problem.
> 
> 
> 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

Brooks,

Inode paraniods are not concerned about *removing* these tmp_xdirs, but
more about creating them, right? Moreover, if these tmp_xdirs are already
there, and at some point ${clear_tmp_xdirs} is cleared, then the tmp_xdirs
will never be removed at a next reboot!
That won't make the paranoids happy....

So I'd suggest to always remove them, but check whether you should create them;
something like this:

  rm -f /tmp/.X[0-9]-lock
-rm -fr /tmp/.X11-unix
-mkdir -m 1777 /tmp/.X11-unix
+rm -fr /tmp/.X11-unix /tmp/.font-unix /tmp/.ICE-unix
+case ${create_tmp_xdirs} in
+[Yy][Ee][Ss])
+	mkdir -m 1777 /tmp/.X11-unix /tmp/.font-unix /tmp/.ICE-unix
+	;;
+esac



And add to /etc/defaults/rc.conf:

create_tmp_xdirs="YES"  # Create directories needed by X11


I will check this today or tomorrow on my 5.3-Stable PC.
Rob.