Alan Coopersmith <alan.coopersm...@oracle.com> writes: > On 11/22/16 04:58 PM, Eric Anholt wrote: >> Adam Jackson <a...@redhat.com> writes: >> >>> Signed-off-by: Adam Jackson <a...@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 <e...@anholt.net>
signature.asc
Description: PGP signature
_______________________________________________ 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