Bug 294999 – sys-apps/baselayout-1 has xsession variable but it is not used in xinit

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Stephane 2009-11-28 19:25:52 UTC]

If you look at the cvs diff between rev 1.5 and rev 1.4 of chooser.sh ( http://sources.gentoo.org/viewcvs.py/gentoo-x86/x11-apps/xinit/files/chooser.sh?r1=1.4&r2=1.5) then you see that the code block responsible for getting the XSESSION variable from rc.conf doesn't exist anymore.

However, the rc.conf file still has comments inviting the user to set his/her XSESSION variable there.

Reproducible: Always

Steps to Reproduce:
1. set XSESSION to whatever you like in rc.conf
2. run /etc/init.d/xdm start and log in


Actual Results:  
whatever you've set for a XSESSION, you end up with 3 terms in twm...

Expected Results:  
Have the WM specified in rc.conf/XSESSION start when logging in X

Workaround : Set the XSESSION variable in /etc/conf.d/xdm

[Comment 1 Tomáš Chvátal (RETIRED) 2009-11-28 22:41:03 UTC]

thats not workaround, thats expected behaviour.
WE want people to actualy use the /etc/conf.d/xdm

[Comment 2 Tomáš Chvátal (RETIRED) 2009-11-28 22:41:38 UTC]

@Remi:
i would go with invalid but i aint sure what are your feelings :]

[Comment 3 Rafał Mużyło 2009-11-28 22:54:32 UTC]

Well, yes and no.
The reason for your problem is the delay
in baselayout 2 getting stabled - the migration guide
tells, that the place for this var has changed.

[Comment 4 Rémi Cardona (RETIRED) 2009-11-28 22:59:11 UTC]

Taken from xinit's pkg_postinst() :

    ewarn "If you use startx to start X instead of a login manager like gdm/kdm,"
    ewarn "you can set the XSESSION variable to anything in /etc/X11/Sessions/ or"
    ewarn "any executable. When you run startx, it will run this as the login session."
    ewarn "You can set this in a file in /etc/env.d/ for the entire system,"
    ewarn "or set it per-user in ~/.bash_profile (or similar for other shells)."
    ewarn "Here's an example of setting it for the whole system:"
    ewarn "    echo XSESSION=\"Gnome\" > /etc/env.d/90xsession"
    ewarn "    env-update && source /etc/profile"

That's what we expect people to follow. If baselayout's rc.conf still mentions XSESSION, then it's a bug for them.

Nothing for us to fix.

Thanks

[Comment 5 Stephane 2009-11-29 16:30:17 UTC]

Based on the comments made, I move this bug to baselayout component, and I re-open it.

I hope it is how I'm supposed to do, if not please tell me :-)

Thanks in advance

Stephane

[Comment 6 Rémi Cardona (RETIRED) 2009-11-29 16:36:32 UTC]

Reassigning (that's what really decides who gets the bug mail, the fields are just extra info for filing)

Thanks

[Comment 7 Rafał Mużyło 2009-12-08 11:58:52 UTC]

*** Bug 296165 has been marked as a duplicate of this bug. ***

[Comment 8 SpanKY 2009-12-14 05:05:13 UTC]

*** Bug 296497 has been marked as a duplicate of this bug. ***

[Comment 9 SpanKY 2009-12-14 05:08:20 UTC]

the X11 team really should have mentioned this obviously intentional team to the people who maintain rc.conf file

http://sources.gentoo.org/baselayout/branches/baselayout-1_12/etc/rc.conf?r1=3164&r2=3165

[Comment 10 Rémi Cardona (RETIRED) 2009-12-14 07:46:52 UTC]

@Mike, sorry for the screw up, we'll do better next time.

Thanks