# 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;