Re: [RFC 1/4] Fix i18n -o option in msvc engine.pl
From: "Johannes Schindelin" <Johannes.Schindelin@gmx.de>
Show 10 quoted lines
> Hi Philip,
>
> On Thu, 20 Nov 2014, Philip Oakley wrote:
>
>> The i18n 5e9637c6 introduced an extra '-o' option
>> into the make file,
>
> I take it you are referring to
>
> https://github.com/git/git/commit/5e9637c6#diff-b67911656ef5d18c4ae36cb6741b7965R2195
Yes, Makefile#L2195 (of the new version)
Show 19 quoted lines
>
>> diff --git a/contrib/buildsystems/engine.pl
>> b/contrib/buildsystems/engine.pl
>> index 23da787..9144ea7 100755
>> --- a/contrib/buildsystems/engine.pl
>> +++ b/contrib/buildsystems/engine.pl
>> @@ -140,6 +140,18 @@ sub parseMakeOutput
>> next;
>> }
>>
>> + if ($text =~ /^mkdir /) {
>> + # options to the Portable Object translations in the
>> line
>> + # mkdir -p po/... && msgfmt ... (eg -o) may be mistaken
>> for linker options
>
> Maybe better
>
> # the line "mkdir ... && msgfmt ..." contains no linker optionsShow 15 quoted lines
>
>> + next;
>> + }
>> +
>> + if ($text =~ /^msgfmt /) {
>> + # options to the Portable Object translations in the
>> line
>> + # mkdir -p po/... && msgfmt ... (eg -o) may be mistaken
>> for linker options
>> + next;
>> + }
>
> These two if clauses do the same, maybe call it
>
> if ($test =~ /^(mkdir|msgfmt) /)Show 8 quoted lines
>
> Ciao,
> Johannes
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
--
*** Please reply-to-all at all times ***
*** (do not pretend to know who is subscribed and who is not) ***
*** Please avoid top-posting. ***
The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.
You received this message because you are subscribed to the Google
Groups "msysGit" group.
To post to this group, send email to msysgit@googlegroups.com
To unsubscribe from this group, send email to
msysgit+unsubscribe@googlegroups.com
For more options, and view previous threads, visit this group at
http://groups.google.com/group/msysgit?hl=en_US?hl=en
---
You received this message because you are subscribed to the Google Groups "Git for Windows" group.
To unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
Philip Oakley <philipoakley@iee.org>