Re: [PATCH] gnu: xorg-server: Enable Xephyr.

[Eric Bavier]

On Mon, 22 Aug 2016 22:23:02 +0200
David Craven <address@hidden> wrote:

> * gnu/packages/xorg.scm (xorg-server)[inputs]: Add xcb-util, xcb-util-image,
> xcb-util-keysyms, xcb-util-renderutil, xcb-util-wm.
> [arguments]: Add '--enable-kdrive' and '--enable-xephyr' configure-flags.
> ---
>  gnu/packages/xorg.scm | 13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm
> index b24e437..9c87602 100644
> --- a/gnu/packages/xorg.scm
> +++ b/gnu/packages/xorg.scm
> @@ -5088,7 +5088,13 @@ over Xlib, including:
>          ("xkbcomp" ,xkbcomp)
>          ("xkeyboard-config" ,xkeyboard-config)
>          ("xtrans" ,xtrans)
> -        ("zlib" ,zlib)))
> +        ("zlib" ,zlib)
> +        ;; Inputs for Xephyr
> +        ("xcb-util" ,xcb-util)
> +        ("xcb-util-image" ,xcb-util-image)
> +        ("xcb-util-keysyms" ,xcb-util-keysyms)
> +        ("xcb-util-renderutil" ,xcb-util-renderutil)
> +        ("xcb-util-wm" ,xcb-util-wm)))
>      (native-inputs
>         `(("python" ,python-minimal-wrapper)
>           ("pkg-config" ,pkg-config)))
> @@ -5106,7 +5112,10 @@ over Xlib, including:
>                              "/bin")
>  
>               ;; For the log file, etc.
> -             "--localstatedir=/var")
> +             "--localstatedir=/var"
> +             ;; For sddm
> +             "--enable-kdrive"
> +             "--enable-xephyr")
>  
>         #:phases (alist-cons-before
>                   'configure 'pre-configure

This seems alright, and a welcome addition, to me.  It builds fine
for me, and `guix size xorg-server` reports <5MiB size increase
(226MiB to 231MiB).

`guix refresh -l xorg-server` says 369 dependent packages. Should this
be put on core-updates?  What do others think?

`~Eric