Hey Thomas, Thomas Danckaert <p...@thomasdanckaert.be> skribis:
> From: l...@gnu.org (Ludovic Courtès) > Subject: Re: bug#24445: GNOME desktop session crash when re-arranging > dock > Date: Sun, 08 Oct 2017 21:18:28 +0200 > >> I think we should set XCURSOR_PATH via >> ‘session-environment-service-type’ like in the patch I posted. I >> find >> it marginally nicer than setting it in /etc/profile because that way >> it >> remains close to ‘gnome-service-type’; we need to do the same in >> ‘xfce-service-type’ though. WDYT? > > Sure, I agree. My choice of /etc/profile was not really motivated: I > just looked for the first place I could set an environment variable, > and used that, thinking there would be someone to tell me if this was > a good place to put it, or not. Silly me: we cannot have $HOME/.guix-profile/share/icons and $HOME/.icons in ‘XCURSOR_PATH’ if we do it via ‘session-environment-service-type’. So I guess we have to go with /etc/profile as in your patch. Can you push the patch (with a comment linking to this issue)? >> Then, eventually, we can add XCURSOR_PATH as a search path >> definition of >> ‘libxcursor’, but due to bug 20255, it won’t be effective here; so >> that’ll be a mostly cosmetic change. > > Adding it as a “cosmetic change” is perhaps useful as a reminder that > we can improve this in the future. Also (bug 22138), I believe search > paths of dependencies are not propagated, so users would also have to > explicitly install libXcursor itself in a profile to have this take > effect? No strong opinion here :) Right. In the patch, I think you can add the extra ‘search-paths’ entry, comment it out, and add a TODO comment. > One less bug for the 0.14 release (come and switch back already, > Mohammed... :-) ) Definitely. :-) Ludo’.
