Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 330355 - x11-apps/xinit: add LXDM support
Summary: x11-apps/xinit: add LXDM support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo X packagers
URL: http://gitweb.sabayon.org/?p=overlay....
Whiteboard:
Keywords:
Depends on:
Blocks: 342661 346833
  Show dependency tree
 
Reported: 2010-07-29 13:38 UTC by Fabio Erculiani (RETIRED)
Modified: 2011-11-16 01:57 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
lxde-base/lxdm-0.2.0 ebuild (lxdm-0.2.0.ebuild,1.76 KB, text/plain)
2010-08-18 15:45 UTC, Mitch Harder
Details
/etc/conf.d/lxdm to start lxdm (lxdm.conf.d,412 bytes, text/plain)
2010-08-18 15:52 UTC, Mitch Harder
Details
/etc/init.d/lxdm to start lxdm (lxdm.init.d,5.98 KB, text/plain)
2010-08-18 15:53 UTC, Mitch Harder
Details
/etc/lxdm/xinitrc configuration file (xinitrc,1.51 KB, text/plain)
2010-08-18 15:54 UTC, Mitch Harder
Details
Patch to disable pam_console (lxdm-0.2.0-pam_console-disable.patch,387 bytes, patch)
2010-08-18 15:55 UTC, Mitch Harder
Details | Diff
The init.d patch (0001-x11-base-xorg-server-Support-lxdm-within-the-xdm-ini.patch,4.41 KB, patch)
2010-10-02 17:11 UTC, Michał Górny
Details | Diff
The startDM.sh patch (0002-x11-apps-xinit-startDM.sh-support-backgrounding-with.patch,3.36 KB, patch)
2010-10-02 17:11 UTC, Michał Górny
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fabio Erculiani (RETIRED) gentoo-dev 2010-07-29 13:38:48 UTC
Beside it's not in Portage yet (but in overlay), the commit at URL introduces early lxdm support (LXDE login manager). Give it a spin.

Reproducible: Always
Comment 1 Fabio Erculiani (RETIRED) gentoo-dev 2010-07-29 13:58:42 UTC
Please also see:
http://gitweb.sabayon.org/?p=overlay.git;a=commit;h=03eb0310651527ae94971a2413dcc58ca0111dc4

We have working lxdm ebuilds in the sabayon overlay too. Maybe lxdm team is interested.
Comment 2 Fabio Erculiani (RETIRED) gentoo-dev 2010-08-14 07:12:23 UTC
Please also see:
http://gitweb.sabayon.org/?p=overlay.git;a=commit;h=c8881ba64adc19e31f24f95a1bc3f239319b4d65

for startDM.sh changes required in order to make lxdm-binary working with /etc/init.d/xdm
Comment 3 Mitch Harder 2010-08-18 15:45:51 UTC
Created attachment 243455 [details]
lxde-base/lxdm-0.2.0 ebuild
Comment 4 Mitch Harder 2010-08-18 15:52:37 UTC
Created attachment 243457 [details]
/etc/conf.d/lxdm to start lxdm
Comment 5 Mitch Harder 2010-08-18 15:53:42 UTC
Created attachment 243459 [details]
/etc/init.d/lxdm to start lxdm
Comment 6 Mitch Harder 2010-08-18 15:54:53 UTC
Created attachment 243461 [details]
/etc/lxdm/xinitrc configuration file

Utilize global and local xinitrc and xprofile files (which had been ignored).
Comment 7 Mitch Harder 2010-08-18 15:55:52 UTC
Created attachment 243463 [details, diff]
Patch to disable pam_console

Patch to disable pam_console in /etc/pam.d/lxdm.  Portage will complain profusely when sys-libs/pam is built if the file exists as included in the upstream.
Comment 8 Mitch Harder 2010-08-18 15:56:11 UTC
I am attaching an ebuild for lxde-base/lxdm-0.2.0.

This ebuild is very similar to the ebuild currently in the sabayon overlay; however, I have stripped out the Sabayon customizations for artwork, etc.

This ebuild was based on an original lxde-base/lxdm-9999 ebuild from the lxde overlay.

I have customized this ebuild to include an /etc/init.d/lxdm and /etc/conf.d/lxdm file to allow starting lxdm using the rc scripts.

Lxdm currently has some incompatibilities with the /etc/init.d/xdm script and the /etc/X11/StartDM.sh script.  I will be working to iron out these incompatibilities in different bug submittals.

Also, this ebuild adds an /etc/lxdm/xinitrc file to utilize global and local xinitrc and xprofile files (which had been ignored).  This file was largely based on the portion of kdm's Xsession script that provides this functionality for kdm.

Lastly, the ebuild utilizes a patch to disable pam_console in /etc/pam.d/lxdm.  Portage will complain profusely when sys-libs/pam is built if the file exists as included in the upstream.
Comment 9 Rémi Cardona (RETIRED) gentoo-dev 2010-08-18 17:41:36 UTC
All those files and patches are beyond the scope of this bug. Unless the lxde herd wants to "take over" this bug report.

As for the xdm/xinit changes Fabio posted in comment #2, those look fine to me but I'd rather let Tomáš (*ping*!!!) comment on them since he knows those bits better than me.

Cheers
Comment 10 Mitch Harder 2010-08-19 11:44:53 UTC
My apologies.

I got turned around as to which bug was filed.
Comment 11 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2010-10-02 16:42:57 UTC
Now I start thinking that we seriously need to consider rewriting that script. I really dislike the idea of adding more and more variables to the old skel... but for now, I'll clean the referenced changes up and attach a patch in a while.
Comment 12 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2010-10-02 17:11:00 UTC
Created attachment 249343 [details, diff]
The init.d patch
Comment 13 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2010-10-02 17:11:49 UTC
Created attachment 249345 [details, diff]
The startDM.sh patch

I dislike my work but I think that's the best we can do without rewriting it.
Comment 14 Fabio Erculiani (RETIRED) gentoo-dev 2010-10-02 17:26:43 UTC
Many people died trying to fix the mess. So, make sure you get things working with LXDM before even thinking of dying ;)
Comment 15 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2010-10-02 17:33:50 UTC
(In reply to comment #14)
> Many people died trying to fix the mess. So, make sure you get things working
> with LXDM before even thinking of dying ;)

BTW someone should suggest upstream to let lxdm background on its own.
Comment 16 Fabio Erculiani (RETIRED) gentoo-dev 2010-10-02 17:43:50 UTC
I agree, even because it seems to be the de-facto default among other DMs.
Comment 17 Fabio Erculiani (RETIRED) gentoo-dev 2011-01-11 21:48:45 UTC
So, ladies and gentlemen, what's the status here? Can we just add the missing bits that introduce lxdm support in init.d/xdm and startDM.sh, and then, think about merging some good enough lxdm ebuild to tree?

Please please pretty please let's kill this bug ;)
Comment 18 Fabio Erculiani (RETIRED) gentoo-dev 2011-01-15 18:15:53 UTC
Packages requiring minimal changes to get lxdm supported are:
- x11-base/xorg-server (init.d/xdm)
- x11-apps/xinit (startDM.sh)

If nobody has objections, I'd go ahead and add the missing bits.
Comment 19 Fabio Erculiani (RETIRED) gentoo-dev 2011-01-22 18:09:06 UTC
I knocked myself out, and updated xinit and xorg-server accordingly.