Whilst populating the xorg_cv_ cache entries, and the subsequent xorg_testset_, we always include the FLAG_PREFIX in the name, to distinguish between C and C++ iterations. However, in the body of the test function, we dropped the FLAG_PREFIX expansion and so always used the empty string and hence did not correctly detect incorrect flags.
The bug was introduced with commit 51f69514e65637f575df901ef942515e70314f6a Author: Chase Douglas <[email protected]> Date: Mon Mar 12 11:49:28 2012 -0700 Separate unknown warning options by language Reported-by: Sedat Dilek <[email protected]> Signed-off-by: Chris Wilson <[email protected]> Cc: Sedat Dilek <[email protected]> Cc: Chase Douglas <[email protected]> Cc: Gaetan Nadon<[email protected]> --- xorg-macros.m4.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in index d589ebd..cd76fca 100644 --- a/xorg-macros.m4.in +++ b/xorg-macros.m4.in @@ -1543,11 +1543,11 @@ fi found="no" m4_foreach([flag], m4_cdr($@), [ if test $found = "no" ; then - if test "x$xorg_testset_unknown_warning_option" = "xyes" ; then + if test "x$xorg_testset_]CACHE_PREFIX[_unknown_warning_option" = "xyes" ; then PREFIX[FLAGS]="$PREFIX[FLAGS] -Werror=unknown-warning-option" fi - if test "x$xorg_testset_unused_command_line_argument" = "xyes" ; then + if test "x$xorg_testset_]CACHE_PREFIX[_unused_command_line_argument" = "xyes" ; then PREFIX[FLAGS]="$PREFIX[FLAGS] -Werror=unused-command-line-argument" fi -- 1.8.4.rc1 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
