[PATCH] compositor-drm: Add environment variable, WESTON_FORCE_SW_CURSORS to force software cursors

[nerdopolis]

Hi.

I figured that it would make getting this fallback behavior easier to script, or for a login manager to export the variable, for a 'fallback' session (along with other variables, such as LIBGL_ALWAYS_SOFTWARE), or just to temporarily enable the fallback behavior...

Thanks.

On Thursday, May 14, 2015 09:36:17 AM Derek Foreman wrote:
> On 13/05/15 09:25 PM, nerdopolis wrote:
> > For some video cards, hardware rendered cursors fail to change properly.
> > 
> > Add a variable that users can use for these cards when starting Weston.
> 
> Why an env var and not a weston.ini option?  I don't think this is a bad
> idea, but I think it'd be better done in the ini file?
> 
> 
> > Also document in the man page
> > 
> > This time with all the closing parenthesis, and the correct commit message.
> > ---
> >  man/weston.man       | 4 ++++
> >  src/compositor-drm.c | 5 ++++-
> >  2 files changed, 8 insertions(+), 1 deletion(-)
> > 
> > diff --git a/man/weston.man b/man/weston.man
> > index c5dc2f2..fd12938 100644
> > --- a/man/weston.man
> > +++ b/man/weston.man
> > @@ -267,6 +267,10 @@ This allows launching Weston as a nested server.
> >  For Wayland clients, holds the file descriptor of an open local socket
> >  to a Wayland server.
> >  .TP
> > +.B WESTON_FORCE_SW_CURSORS
> > +Set this variable to force Weston to use software cursors when under the 
> > +compositor-drm backend
> > +.TP
> >  .B WESTON_CONFIG_FILE
> >  Weston sets this variable to the absolute path of the configuration file
> >  it loads, or to the empty string if no file is used. Programs that use
> > diff --git a/src/compositor-drm.c b/src/compositor-drm.c
> > index 0cdb8f4..4e5e2fc 100644
> > --- a/src/compositor-drm.c
> > +++ b/src/compositor-drm.c
> > @@ -1641,7 +1641,10 @@ drm_output_init_egl(struct drm_output *output, struct drm_compositor *ec)
> >  		weston_log("cursor buffers unavailable, using gl cursors\n");
> >  		ec->cursors_are_broken = 1;
> >  	}
> > -
> > +	if (getenv("WESTON_FORCE_SW_CURSORS"))
> > +	{
> > +		ec->cursors_are_broken = 1;
> > +	}
> >  	return 0;
> >  }
> >  
> > 
>