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


[This was posted directly to [EMAIL PROTECTED]  So that it  
enters our tracking system, I am re-posting excerpts via parrotbug.   
I will be reviewing it shortly. --Jim Keenan]

Maria Fedotova writes:

Hello folks,
There was a problem in config/gen/makefiles.pm:
     $targets = $conf->options->get('targets');
while the option name was 'target'.

I've restored correct access and slightly improved the option
description. Could anyone review and possibly commit the patch? Is the
[PATCH] mail formed correctly?
Hello folks,
There was a problem in config/gen/makefiles.pm:
     $targets = $conf->options->get('targets');
while the option name was 'target'.

I've restored correct access and slightly improved the option
description. Could anyone review and possibly commit the patch? Is the
[PATCH] mail formed correctly?



Index: lib/Parrot/Configure/Options/Reconf.pm
===================================================================
--- lib/Parrot/Configure/Options/Reconf.pm      (revision 22005)
+++ lib/Parrot/Configure/Options/Reconf.pm      (working copy)
@@ -48,9 +48,9 @@ General Options:
 
    --help               Show this text
    --verbose            Output extra information
-   --step=(gen::languages)
+   --step=(gen::languages|gen::makefiles)
                         Execute a single configure step
-   --target=(Makefile)  Needed if --step=gen::makefiles
+   --target=(Makefile)  Regenerate a specific makefile
    --languages="list of languages"
                         Specify a list of languages to process
 
Index: config/gen/makefiles.pm
===================================================================
--- config/gen/makefiles.pm     (revision 22005)
+++ config/gen/makefiles.pm     (working copy)
@@ -103,7 +103,7 @@ EOF
 sub makefiles {
     my ( $self, $conf ) = @_;
 
-    my $targets = $conf->options->get('targets');
+    my $targets = $conf->options->get('target');
     my @targets =
         defined $targets
         ? split ' ', $targets

Reply via email to