From: L p R n d n <guix@lprndn.info>
To: Brice Waegeneire <brice@waegenei.re>
Cc: 35305@debbugs.gnu.org
Subject: [bug#35305] LightDM service
Date: Tue, 14 Apr 2020 11:38:18 +0200 [thread overview]
Message-ID: <874ktmjtkl.fsf@lprndn.info> (raw)
In-Reply-To: <2d2d720d1fd6a6357a171e13953a593b@waegenei.re> (Brice Waegeneire's message of "Sun, 12 Apr 2020 09:53:41 +0000")
Hello,
Thanks again for the feedbacks.
Brice Waegeneire <brice@waegenei.re> writes:
[...]
> There is an error on vt1 at startup about some dbus stuff related to
> “accountservice”. The cursor theme isn't set as well as the icon theme,
> it
> seems; it may come from some missing environment variable as NixOS set a
> few of them in their service[0].
For the cursor, setting "XCURSOR_PATH" seems to solve the problem.
Preparing a patch.
As for dbus, from what I found at NixOS[0][1], it's a accountsservice
bug. I'll send a bug report. In the meantime, it doesn't appear to
affect LightDM too much so it might be ok...?
> After digging in the new service file I keep wondering if it's the right
> way to go about implementing it. I tried to set a simple setting
> for `lightdm-gtk-greeter` and I ended up with the following which
> doesn't
> look intuitive:
>
> (service lightdm-service-type
> (lightdm-configuration
> (greeter-configuration-file
> (lightdm-gtk-greeter-configuration-file
> (lightdm-gtk-greeter-configuration
> (extra-config "hide-user-image=true"))))))
Yeah, far from elegant. Did it that way in case someone wanted to
provide its own file but dropping
lightdm-gtk-greeter-configuration-file gives one level less of nesting.
> Furthermore `lightdm-service-type` only set a single seat while lightdm
> can
> have many of them. So maybe there should be a lightdm-seat` record used
> as
> a list of seats in `lightdm-service-type`. Each greater will be able to
> expand `lightdm-service-type` by appending a seat to it. Then the
> configuration could look like this:
>
> (service lightdm-service-type)
> (service lightdm-gtk-greeter-service-type
> (lightdm-gtk-greeter-configuration
> (assets (list guix-artwork))
> (theme-name "Guix")
> (extra-config "hide-user-image=true")))
I didn't know one could use a different greeter for each seat. Indeed,
it would be nice to have that feature!
The only thing that bothers me is that you don't get a working
default LightDM service by default. I feel that just adding (service
lightdm-service-type) to your configuration should get you a fully
working LightDM, specially for newcomers. Here one would lack a greeter.
:/
I believe there is a solution that would give us the best of both
worlds. If someone has a clue, an idea or even a patch, please share it!
:)
Also, merging as is and improving the service later is possible too.
>
> Using this approch would also avoid populating “/etc”, at least for
> `lightdm-grk-greeter`, since each greeter would be defined as a service
> and
> could expand “/etc” if it really needs it. For `lightdm-gtk-greeter`
> specifically avoiding putting config in “/etc/” can be done by compiling
> it
> with `-DCONFIG_FILE` pointing to `/run/current-system` And `lightdm` can
> be
> started with `--config`.
I will make a patch to use --config for lightdm.
For lightdm-gtk-greeter, how would one put the config file in
"/run/current-system"? Do we need to make a package out of it?
> Here is an example in how to make a conjuration file procedure more
> readable while keeping under the line length limit by using
> `match-record`:
>
> (define (lightdm-gtk-greeter-configuration-file config)
> (match-record config <lightdm-gtk-greeter-configuration>
> (theme-name icon-theme-name cursor-theme-name cursor-size background
> extra-config)
> (mixed-text-file "lightdm-gtk-greeter.conf" "
> [greeter]
> theme-name = " theme-name "
> icon-theme-name = " icon-theme-name "
> cursor-theme-name = " cursor-theme-name "
> cursor-theme-size = " (number->string cursor-size) "
> background = " background "
> " extra-config "
> ")))
Way better! Preparing a patch for that.
> BTW the default background could use one from Guix artwork repo like
> “(file-append %artwork-repository "/grub/GuixSD-fully-black-16-9.svg")”.
Yeap!
> WDYT? I may be completely off on the `lightdm-seat` part.
>
> [0]:
> https://github.com/NixOS/nixpkgs/blob/release-19.09/nixos/modules/services/x11/display-managers/lightdm-greeters/gtk.nix#L18
>
> Cheers,
> - Brice
Have a nice day!
L p R n d n
next prev parent reply other threads:[~2020-04-14 9:39 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-17 14:24 [bug#35305] [WIP] LightDM service L p R n d n
2019-04-18 11:20 ` Jonathan Brielmaier
[not found] ` <handler.35305.B.155550391014002.ack@debbugs.gnu.org>
2019-04-18 13:20 ` [bug#35305] Acknowledgement ([WIP] LightDM service) L p R n d n
2019-04-18 16:03 ` L p R n d n
2019-08-26 15:58 ` L p R n d n
2020-03-15 21:50 ` Nicolò Balzarotti
2020-03-16 7:34 ` Efraim Flashner
2020-03-16 8:36 ` L p R n d n
2020-03-19 11:54 ` [bug#35305] LightDM service L p R n d n
2019-05-23 11:04 ` [bug#35305] [PATCH] " L p R n d n
2020-04-07 17:06 ` [bug#35305] " Brice Waegeneire
2020-04-09 16:02 ` L p R n d n
2020-04-12 9:53 ` Brice Waegeneire
2020-04-14 9:38 ` L p R n d n [this message]
2020-04-14 13:17 ` L p R n d n
2020-04-22 15:26 ` L p R n d n
2020-05-06 14:05 ` L p R n d n
2020-05-08 22:18 ` Ricardo Wurmus
2020-05-09 15:09 ` L p R n d n
2020-05-10 19:21 ` Ricardo Wurmus
2020-05-11 10:14 ` L p R n d n
2020-05-12 9:59 ` L p R n d n
2020-05-20 20:51 ` Ricardo Wurmus
2020-05-21 8:28 ` L p R n d n
2020-05-21 9:23 ` Ricardo Wurmus
2020-06-08 15:35 ` L p R n d n
2022-08-04 5:09 ` [bug#35305] [WIP] " Maxim Cournoyer
2020-06-19 14:47 ` [bug#35305] " L p R n d n
2022-08-04 2:19 ` [bug#35305] [WIP] " Maxim Cournoyer
2022-08-31 7:13 ` bug#35305: " Ricardo Wurmus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=874ktmjtkl.fsf@lprndn.info \
--to=guix@lprndn.info \
--cc=35305@debbugs.gnu.org \
--cc=brice@waegenei.re \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.