[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xorg-devel
Subject:    Re: [PATCH xserver] configure: Enable glamor when building just Xwayland
From:       Eric Anholt <eric () anholt ! net>
Date:       2016-11-23 19:47:16
Message-ID: 87shqirnaz.fsf () eliezer ! anholt ! net
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Alan Coopersmith <alan.coopersmith@oracle.com> writes:

> On 11/22/16 04:58 PM, Eric Anholt wrote:
> > Adam Jackson <ajax@redhat.com> writes:
> > 
> > > Signed-off-by: Adam Jackson <ajax@redhat.com>
> > > ---
> > > configure.ac | 5 +----
> > > 1 file changed, 1 insertion(+), 4 deletions(-)
> > > 
> > > diff --git a/configure.ac b/configure.ac
> > > index 6fd853b..93c81a6 100644
> > > --- a/configure.ac
> > > +++ b/configure.ac
> > > @@ -2138,10 +2138,7 @@ AM_CONDITIONAL([XORG_DRIVER_MODESETTING], [test \
> > > "x$XORG_DRIVER_MODESETTING" = xy 
> > > dnl glamor
> > > if test "x$GLAMOR" = xauto; then
> > > -	if test "x$XORG" = xyes; then
> > > -		GLAMOR=yes
> > > -	fi
> > > -	if test "x$XEPHYR" = xyes; then
> > > +	if echo "$XORG" "$XEPHYR" "$XWAYLAND" | grep -q yes ; then
> > > 		GLAMOR=yes
> > > 	fi
> > > fi
> > 
> > Doesn't that want a "> /dev/null" to not spit the 'yes' into the
> > configure output?  Other than that, this is great.
> 
> Isn't that what the -q option to grep does?

Missed that.  In that case,

Reviewed-by: Eric Anholt <eric@anholt.net>


["signature.asc" (application/pgp-signature)]
[Attachment #6 (text/plain)]

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

[prev in list] [next in list] [prev in thread] [next in thread]