[PATCH] winemaker: Fix --single-target option parsing.

André Hentschel nerv at dawncrow.de
Wed Mar 13 06:01:50 CDT 2019


On 12.03.19 14:41, Huw Davies wrote:
> Signed-off-by: Huw Davies <huw at codeweavers.com>
> ---
>   tools/winemaker/winemaker | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/winemaker/winemaker b/tools/winemaker/winemaker
> index ec5fb16a68..88ae78ce48 100755
> --- a/tools/winemaker/winemaker
> +++ b/tools/winemaker/winemaker
> @@ -2702,8 +2702,9 @@ binmode(STDOUT, ":utf8");
>   
>   target_init(\@global_settings);
>   
> -foreach(@ARGV) {
> -  my $arg=$_;
> +my @args = @ARGV;
> +while (@args>0) {
> +  my $arg=shift @args;
>     # General options
>     if ($arg eq "--nobanner") {
>       $opt_no_banner=1;
> @@ -2712,7 +2713,7 @@ foreach(@ARGV) {
>     } elsif ($arg eq "--nobackup") {
>       $opt_backup=0;
>     } elsif ($arg eq "--single-target") {
> -    $opt_single_target=shift @ARGV;
> +    $opt_single_target=shift @args;
>     } elsif ($arg eq "--lower-none") {
>       $opt_lower=$OPT_LOWER_NONE;
>     } elsif ($arg eq "--lower-all") {
> 

well spotted

Signed-off-by: André Hentschel <nerv at dawncrow.de>



More information about the wine-devel mailing list