Comment 10 for bug 818864

Revision history for this message
Yves-Alexis Perez (corsac) wrote : Re: [Bug 818864] Re: add support foran“Xclient”fallbacksession

On mar., 2011-08-16 at 22:11 +0000, Andreas Rottmann wrote:
> Yves-Alexis Perez <email address hidden> writes:
>
> > But note that *preventing* people to chose what they're used to might
> > not be the wisest.
> >
> Indeed. I don't really care if I have to tweak configs to enable my
> ~/.xsession script to be used, but with the Debian package, before this
> issue had been fixed (assuming the .desktop file in question would have
> been included as an example instead), I'd have had to:
>
> # cp -a /usr/share/xsessions /etc
> # cp /usr/share/doc/lightdm/examples/default.desktop /etc/xessions
> # $EDITOR /etc/lightdm/lightdm.conf # to change the directory lightdm
> # looks for .desktop files in

Why copy stuff in /etc? You only need to put a .desktop file
in /usr/share/xsessions.
>
> Additionally, I would have had the burden to update my copy of the
> desktop files in /etc/xessions, should they change in the package.
> IMHO, for my usecase, that's quite more inconvinient than for some user
> not caring about ~/.xsession just to ignore the corresponding dropdown
> box entry.

Again, not necessary, you can still use the ones from /u/s/xsessions.
>
> > I don't really agree with the “confuse normal users in 99% of the
> > case” (but again, it might depend which users we are considering).
> >
> If it's really decided, that for the Debian package, the default
> Xsession should not be present in the menu, I'd at least expect
> prominent documentation on how to enable this feature in
> /usr/share/doc/lightdm/README.Debian (or somewhere similiar). Of
> course, *I* prefer having that feature enabled by default (and thanks
> for the fix, BTW).

For Debian package, it really depends on how the fix is done upstream,
but I might keep shipping the lightdm-xsession.desktop file even then,
because that's what Debian users are used to (at least that's what is
done in GDM 2).

Regards,
--
Yves-Alexis