On Wed, Feb 15, 2012 at 07:45:14AM +1100, Brett wrote: > Hi, > While installing xcursor-themes I noticed that the config instructions > at are out-of-date. The path specified does not > exist when this port is built.
${PREFIX} is supposed to be the path where ports are installed (/usr/local by default). The policy is to not hard-code /usr/local in port files. ~/.icons doesn't need to exist, but it's the freedesktop standard location inside ${HOME} where applications will put additionnal cursors when not installed globally. > Also the instructions say to use the > XCURSOR_SIZE environment variable to change the pointer size, but (at > least when using jwm) it does nothing. Changing Xcursor.size in > ~/.Xdefaults does work though. XCURSOR_SIZE only affects the size of the cursor in applications started with the new value. It doesn't change the size of existing cursors. But when used correctly it definatly does work. So I've nothing against a patch that adds the fact that the Xcursor.size resource can be used, but I don't want to remove the env var documentation. > > > Index: ports/x11/xcursor-themes/pkg/DESCR > =================================================================== > RCS file: /usr/cvsync/ports/x11/xcursor-themes/pkg/DESCR,v > retrieving revision 1.2 > diff -u -r1.2 DESCR > --- ports/x11/xcursor-themes/pkg/DESCR 15 Apr 2010 14:34:12 -0000 > 1.2 > +++ ports/x11/xcursor-themes/pkg/DESCR 14 Feb 2012 20:38:32 -0000 > @@ -6,7 +6,7 @@ > > To use the new cursors > - set the environment variable XCURSOR_PATH to > - "~/.icons:${PREFIX}/lib/X11/icons" > + "/usr/local/lib/X11/icons" > > - either : > a) set the XCURSOR_THEME environment variable to one of the theme names > @@ -18,5 +18,5 @@ > Inherits=whiteglass > -- Cut -- > > -The XCURSOR_SIZE environment variable can be used to set the size of > -the cursors. > +The size of the cursors can be changed by adding the line "Xcursor.size: > +value" to ~/.Xdefaults, where value is a pixel size (eg 32 or 48). -- Matthieu Herrb