Howdy! Thomas Danckaert <p...@thomasdanckaert.be> skribis:
> Ha, I was just going to reply the same (see attached patch). I've > tested it on my system as well, and it works. > > Indeed encoding the absolute file name of the icon theme seems hard, > unless we set the icon theme inside the system configuration? Not sure > how we want to handle icon themes installed in user profiles. Yeah, let’s just keep /run/current-system & co. > In my patch, I included both system and user profile, in case users want > to install icon themes in their profile, as well as the default ~/.icons > (which seems to be a kind of standard as well, so thought we could keep > it by default). Yes, it’s a good idea to include ~/.icons and ~/.guix-profile/share/icons (I thought that XCURSOR_PATH did not take override the default, which includes ~/.icons.) > Also, I'm not sure if we need to set both paths in /etc/profile, or if > we can just have icon theme packages prepend to the current XCURSOR_PATH > when installed into a user profile (I'm not familiar with the details of > http://bugs.gnu.org/20255)? 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? 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. Does that make sense? Anyway, glad we have a fix. :-) Ludo’.
