ludo@gnu.org (Ludovic Courtès) writes:

> I think that’d be OK.
>
> Though libXcursor contains a few ‘getenv’ calls, notably for
> XCURSOR_PATH, which looks like it’s what we’re looking for.
>
> I tested the attached path in a VM and it seems to fix the bug: I can
> drag a window in the overview area on the right and it no longer crashes
> (*and* I get a hand-shaped cursor when hovering that area :-)).  And we
> don’t need to modify libXcursor.
>
> (We could do slightly better by using the absolute file name of the
> actual icon theme instead of /run/current-system, but it’s complicated
> because I guess it doesn’t have to be Adwaita.)
>
> Thoughts?

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.

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).

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)?

cheers!

Thomas