# New Ticket Created by  James Keenan 
# Please include the string:  [perl #42082]
# in the subject line of all future correspondence about this issue. 
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=42082 >


Ongoing refactoring of Configure.pl in the 'reconfigure' branch  
suggests two bugs in Configure.pl.

1.  Configure.pl has a lexical @valid_opts (starting at line 284 for  
me).  One of the elements of this array is:  'verbose-step=N'.   
Shouldn't this really be:  'verbose-step'?

2.  The POD for Configure.pl documents two options which are not  
found in @valid_opts:  'bison_required' and 'flex_required'.  Are  
these old options that have been removed?  If so, the corresponding  
POD should also be deleted.

Here is a patch that will correct these problems in trunk.

kid51

Index: Configure.pl
===================================================================
--- Configure.pl        (revision 17743)
+++ Configure.pl        (working copy)
@@ -142,18 +142,10 @@
 
 Specify which lexer to use.
 
-=item C<--flex_required=X.Y.Z>
-
-Override the minimum acceptable flex version.
-
 =item C<--yacc=(parser)>
 
 Specify which parser to use.
 
-=item C<--bison_required=X.Y>
-
-Override the minimum acceptable bison version.
-
 =item C<--define=val1[,val2]>
 
 Generate "#define PARROT_DEF_VAL1 1" ... entries in has_header.h. Currently
@@ -295,7 +287,7 @@
     jitcapable ld ldflags lex libdir libexecdir libs link linkflags
     localstatedir m maintainer mandir miniparrot nomanicheck oldincludedir
     opcode ops optimize parrot_is_shared pmc prefix profile sbindir
-    sharedstatedir step sysconfdir verbose verbose-step=N version without-gdbm
+    sharedstatedir step sysconfdir verbose verbose-step version without-gdbm
     without-gmp without-icu yacc);
 
 my %args;

Reply via email to