Takao:

>> Note that /etc/X11/gdm/Xsession should only be run if the user
>> is using the GDM display manager.  Perhaps the comment should
>> highlight this?
> 
> I thought Indiana Live session from the bootable CD.
> It runs dtstart but now xinitrc.d is loaded by /etc/X11/gdm/Xsession so 
> I'd like to avoid to run dtstart directly.
> I think if Indiana Live session calls /etc/X11/gdm/Xsession instead of 
> dtstart, this problem can be resolved.

Yes, as a part of this fix, the Indiana Live session would need to be
changed to either call /etc/X11/gdm/Xsession or do the things that
/etc/X11/gdm/Xsession does directly, like sourcing the files in
/etc/X11/xinit/xinitrc.d before starting gnome-session.

>>> If it's not designed to be run directly, it shouldn't be in /usr/bin
>>
>> The dtstart script now no longer does seems to do anything except
>> translate "jds" to "/usr/bin/gnome-session".  I think it be better to
>> just get rid of the dtstart script altogether and also get rid of the
>> gnome-session-06-Xsession-dtstart.diff and the
>> gnome-session-07-Xsession.diff patches and just let gnome-session be
>> run directly from the /etc/X11/gdm/Xsession script when using GDM.
>> What do you think?  Since CDE login doesn't use the
>> /usr/share/xsessions/gnome.desktop file, this shouldn't have any impact
>> on CDE login.
> 
> I think it's good to remove dtstart if we can. GDM don't have to call 
> dtstart.

I agree.  It doesn't seem to do anything significant anymore.  We've
moved all the code into /etc/X11/xinit/xinitrc.d, which is a more
appropriate place for such setup.

Brian

Reply via email to