The patch attached.

On 10/10/07, Maria Fedotova <[EMAIL PROTECTED]> wrote:
> 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?
>
> Thanks, M.
>
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