diff options
| author | Dan Nicholson <dbn.lists@gmail.com> | 2009-02-05 09:17:17 -0800 |
|---|---|---|
| committer | Dan Nicholson <dbn.lists@gmail.com> | 2009-02-05 09:21:24 -0800 |
| commit | f05148fa57a1ebdb75135f026b7c5f4678a3b293 (patch) | |
| tree | 2842eaee0ed9b119144b3e7c2677df7a59b187f4 | |
| parent | 96585612d17d31fa9dd0f5c260fc33f72be9a75f (diff) | |
Fix comment in XORG_MACROS_VERSION on usage
The comment recommending checking XORG_MACROS_VERSION with m4_ifndef was
flawed for two reasons:
* m4_fatal should be used if the macro is not defined. There's no use
generating configure if the macro doesn't exist. Just fail during
autoconf.
* There's no reason to mention the version needed in the message. That
check does not do version verification. That's what the actual macro
is for.
Signed-off-by: Dan Nicholson <dbn.lists@gmail.com>
| -rw-r--r-- | xorg-macros.m4.in | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in index 2a5e59c..56ec104 100644 --- a/xorg-macros.m4.in +++ b/xorg-macros.m4.in @@ -35,8 +35,9 @@ dnl of the copyright holder. # your configure.ac with the minimum required version, such as: # XORG_MACROS_VERSION(1.1) # -# To force at least a version with this macro defined, also add: -# m4_ifndef([XORG_MACROS_VERSION], [AC_FATAL([must install xorg-macros 1.1 or later before running autoconf/autogen])]) +# To ensure that this macro is defined, also add: +# m4_ifndef([XORG_MACROS_VERSION], +# [m4_fatal([must install xorg-macros before running autoconf/autogen])]) # # # See the "minimum version" comment for each macro you use to see what |