Re: [PATCH] xorg-macros.m4.in minor fix for incorrect warning message

[Dan Nicholson]

On Wed, Feb 18, 2009 at 2:52 PM, Dan danstowell+x...@gmail.com wrote:
 Hi -

 One of the warnings in xorg-macros.m4.in is wrong, it prints out the
 wrong version when it says requires version x.x. Patch pasted below
 against http://cgit.freedesktop.org/xorg/util/macros/tree/xorg-macros.m4.in
 - not a proper git patch sorry, but hope OK and sensible

Yeah, I noticed that a couple weeks ago. Applied in
72d82ed965f9cfbc310897ec17d2dc10bddcef4e . Please do try to send in
properly formatted patches if you can.

--
Dan
___
xorg mailing list
xorg@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/xorg

[PATCH] xorg-macros.m4.in minor fix for incorrect warning message

[Dan]

Hi -

One of the warnings in xorg-macros.m4.in is wrong, it prints out the
wrong version when it says requires version x.x. Patch pasted below
against http://cgit.freedesktop.org/xorg/util/macros/tree/xorg-macros.m4.in
- not a proper git patch sorry, but hope OK and sensible

Dan
-- 
http://www.mcld.co.uk
--- xorg-macros.m4.in.txt   2009-02-18 22:36:42.0 +
+++ xorg-macros.m4.in copy.txt  2009-02-18 22:37:27.0 +
@@ -51,10 +51,10 @@
XORG_MACROS_major=`echo $XORG_MACROS_version | sed 's/\..*$//'`
XORG_MACROS_minor=`echo $XORG_MACROS_version | sed -e 's/^[0-9]*\.//' 
-e 's/\..*$//'`]
if test $XORG_MACROS_major -ne $XORG_MACROS_needed_major ; then
-   AC_MSG_ERROR([configure built with incompatible version of 
xorg-macros.m4 - requires version ${XORG_MACROS_major}.x])
+   AC_MSG_ERROR([configure built with incompatible version of 
xorg-macros.m4 - requires version ${XORG_MACROS_needed_major}.x])
fi
if test $XORG_MACROS_minor -lt $XORG_MACROS_needed_minor ; then
-   AC_MSG_ERROR([configure built with too old of a version of 
xorg-macros.m4 - requires version ${XORG_MACROS_major}.${XORG_MACROS_minor}.0 
or newer])
+   AC_MSG_ERROR([configure built with too old of a version of 
xorg-macros.m4 - requires version 
${XORG_MACROS_needed_major}.${XORG_MACROS_needed_minor}.0 or newer])
fi
AC_MSG_RESULT([yes, $XORG_MACROS_version])
 ]) # XORG_MACROS_VERSION
___
xorg mailing list
xorg@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/xorg