Ludovic Courtès transcribed 1.0K bytes:
> Hi,
> 
> The first patch here cleans up ‘xorg-configuration-file’ et al. such
> that users can pass a list of modules to use (until now the list of
> modules was hard-coded in ‘xorg-configuration-file’.)
> 
> The second one removes WindowMaker as the default fallback session.
> Instead, the default is to look for a .desktop file at run time in the
> user and system profiles.
> 
> The downside is that if no package provides a .dekstop file, then login
> fails.  I think that’s acceptable though, because people expect to
> specify a list of window managers in the ‘packages’ field anyway, as
> show in the lightweight-desktop example.
> 
> Thoughts?

Overall, good.
What I think is lacking is that we add in the Documentation that
adding a WM/DM is necessary to get started.
It might sound like a little too much of a Cpt. Obvious, but
we can't assume everyone understand the implications of not
defining a WM/DM in the config.

> Ludo’.
> 
> Ludovic Courtès (2):
>   services: xorg: Allow users to specify a list of modules.
>   services: xorg: Remove WindowMaker as a default fallback.
> 
>  doc/guix.texi         |  26 ++++--
>  gnu/services/xorg.scm | 244 +++++++++++++++++++++++++++++++++++---------------
>  2 files changed, 189 insertions(+), 81 deletions(-)
> 
> -- 
> 2.15.0
> 
> 
> 
> 
> 

-- 
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://d.n0.is/dist/keys/
  WWW: https://n0.is