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

Reply via email to