options: tiny simplification in dealing with erroneous opts
authorStefano Lattarini <stefano.lattarini@gmail.com>
Fri, 10 May 2013 21:01:27 +0000 (23:01 +0200)
committerStefano Lattarini <stefano.lattarini@gmail.com>
Fri, 10 May 2013 21:01:27 +0000 (23:01 +0200)
* lib/Automake/Options.pm (_process_option_list): Here, when an
invalid option is detected, there's no need to call &error with
the "uniq_scope => US_GLOBAL" switch.  In fact, if the same
erroneous option is specified in AUTOMAKE_OPTIONS in both (say)
'Makefile.am' and 'sub/Makefile.am', we want each such erroneous
usage reported separately, rather than just the first time it is
encountered (as happens when "uniq_scope => US_GLOBAL" is used).

Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>
lib/Automake/Options.pm

index d578c9b..dcdc119 100644 (file)
@@ -384,8 +384,7 @@ sub _process_option_list (\%@)
         }
       elsif (! _is_valid_easy_option $_)
         {
-          error ($where, "option '$_' not recognized",
-                 uniq_scope => US_GLOBAL);
+          error ($where, "option '$_' not recognized");
           return 1;
         }
     }