Hi Ludo and Danny, Ludovic Courtès <l...@gnu.org> writes:
> Hello, > > Timothy Sample <samp...@ngyro.com> skribis: > >> There are two workarounds. The first is to keep Danny’s patch as-is, >> but add logic to “xinitrc” so that it only uses “~/.xsession” when >> passed “custom” as an argument (it’s kinda magic, but it seems to be a >> common convention). I have a slight preference for this option, because >> it makes the whole “~/.xsession” thing more explicit. However, it would >> break other DMs that don’t install a “custom.desktop”. Maybe we could >> integrate it into all DMs at the service level. >> >> The quicker workaround is to keep the current logic, and install the >> following with GDM: >> >> [Desktop Entry] >> Encoding=UTF-8 >> Type=Application >> Name=Fail >> Comment=This session always fails. >> NoDisplay=true >> Exec=false >> >> This way, GDM fails cleanly when there are no “.desktop” files. It >> doesn’t show up in the list, either (“NoDisplay=true”), so everything >> just kinda works as expected without any visible changes. > > I have a slight preference for the hack you propose Timothy, at least in > the short term, because it seems less likely to break anything than > fiddling with “Exec=custom” and all that. > > Anyway, it’d be great if one of you could push a fix by 1.0. :-) Pushed as 8caa458953eeac783c73a0e5aaa72842fe3914c9. I’m closing the bug and the patch. If we want to change the “xinitrc” script to understand “custom”, we can start a new discussion to figure out the best way to do it after 1.0. -- Tim