git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: "Philip Oakley" <philipoakley@iee.org>
To: "Eric Sunshine" <sunshine@sunshineco.com>
Cc: "Git List" <git@vger.kernel.org>,
	"Junio C Hamano" <gitster@pobox.com>,
	"MsysGit List" <msysgit@googlegroups.com>,
	"Yue Lin Ho" <b8732003@student.nsysu.edu.tw>
Subject: Re: [PATCH v2 10/16] engine.pl: delete the captured stderr file if empty
Date: Mon, 20 Jul 2015 07:16:24 +0100	[thread overview]
Message-ID: <FDE2D068444A4350B0C2F25275CBC41D@PhilipOakley> (raw)
In-Reply-To: CAPig+cSDGUNZfc3cV=P2WUDUR+MBBjoHFxTBFaizf0NH-LmFTw@mail.gmail.com

From: "Eric Sunshine" <sunshine@sunshineco.com>
> On Sun, Jul 19, 2015 at 4:08 PM, Philip Oakley <philipoakley@iee.org> 
> wrote:
>> Keep the build clean of extraneous files if it is indeed clean.
>> Otherwise leave the msvc-build-makedryerrors.txt file both as
>> a flag for any CI system or for manual debugging.
>>
>> Note that the file will contain the new values of the GIT_VERSION
>> and GITGUI_VERSION if they were generated by the make file. They
>> are omitted if the release is tagged and indentically defined in
>> their respective GIT_VERSION_GEN file DEF_VER variables.
>>
>> Signed-off-by: Philip Oakley <philipoakley@iee.org>
>> ---
>> diff --git a/contrib/buildsystems/engine.pl 
>> b/contrib/buildsystems/engine.pl
>> index a6999b6..020776e 100755
>> --- a/contrib/buildsystems/engine.pl
>> +++ b/contrib/buildsystems/engine.pl
>> @@ -77,6 +77,8 @@ EOM
>>
>>  my $ErrsFile = "msvc-build-makedryerrors.txt";
>>  @makedry = `cd $git_dir && make -n MSVC=1 V=1 2>$ErrsFile` if 
>> !@makedry;
>> +# test for an empty Errors file and remove it
>> +for ($ErrsFile) {unlink $_ if (-f $_) && (!-s $_);}
>
> Why the 'for' loop?
>
> Also, if you're using the 'for' loop for the $_ side-effect, then why
> not the simpler:

It was cargo cult programming, with some Google searching to select 
between invocations. Most examples were looping through lists in 
scripts, hence the down select.

>
>    for ($ErrsFile) { unlink if -f && !-s; }

A lot better. Will fix.

>
> ?
>
>>
>>  # Parse the make output into usable info
>>  parseMakeOutput();
>> --
>> 2.4.2.windows.1.5.gd32afb6
>

-- 
-- 
*** 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.

  reply	other threads:[~2015-07-20  6:16 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-19 20:08 [PATCH v2 00/16] Make the msvc-build scripts work again Philip Oakley
2015-07-19 20:08 ` [PATCH v2 01/16] perl/Makefile: treat a missing PM.stamp as if empty Philip Oakley
2015-07-19 20:08 ` [PATCH v2 02/16] .gitignore: improve MSVC ignore patterns Philip Oakley
2015-07-19 20:08 ` [PATCH v2 03/16] .gitignore: ignore library directories created by MSVC VS2008 buildsystem Philip Oakley
2015-07-19 20:08 ` [PATCH v2 04/16] (msvc-build) Vcproj.pm: remove duplicate GUID Philip Oakley
2015-07-19 20:08 ` [PATCH v2 05/16] engine.pl: fix error message (lib->link) Philip Oakley
2015-07-19 20:08 ` [PATCH v2 06/16] engine.pl: Properly accept quoted spaces in filenames Philip Oakley
2015-07-19 20:08 ` [PATCH v2 07/16] engine.pl: Fix i18n -o option in msvc buildsystem generator Philip Oakley
2015-07-19 20:08 ` [PATCH v2 08/16] engine.pl: ignore invalidcontinue.obj which is known to MSVC Philip Oakley
2015-07-20  1:54   ` Eric Sunshine
2015-07-20  6:13     ` Philip Oakley
2015-07-19 20:08 ` [PATCH v2 09/16] engine.pl: name the msvc buildsystem's makedry error file Philip Oakley
2015-07-19 20:08 ` [PATCH v2 10/16] engine.pl: delete the captured stderr file if empty Philip Oakley
2015-07-20  2:01   ` Eric Sunshine
2015-07-20  6:16     ` Philip Oakley [this message]
2015-07-20  6:40       ` Eric Sunshine
2015-07-20 11:55         ` Philip Oakley
2015-07-20 18:10           ` Eric Sunshine
2015-07-19 20:08 ` [PATCH v2 11/16] engine.pl: add debug line to capture the dry-run Philip Oakley
2015-07-19 20:08 ` [PATCH v2 12/16] engine.pl: provide more debug print statements Philip Oakley
2015-07-19 20:08 ` [PATCH v2 13/16] Vcproj.pm: list git.exe first to be startup project Philip Oakley
2015-07-19 20:08 ` [PATCH v2 14/16] vcbuild/readme: Improve layout and reference msvc-build script Philip Oakley
2015-07-19 20:21   ` Philip Oakley
2015-07-19 20:08 ` [PATCH v2 14/16] vcbuild/readme: Improve layout Philip Oakley
2015-07-19 20:08 ` [PATCH v2 15/16] msvc-build: add complete Microsoft Visual C compilation script Philip Oakley
2015-07-19 20:08 ` [PATCH v2 16/16] config.mak.uname: add MSVC No_SafeExeceptionHandler option Philip Oakley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=FDE2D068444A4350B0C2F25275CBC41D@PhilipOakley \
    --to=philipoakley@iee.org \
    --cc=b8732003@student.nsysu.edu.tw \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=msysgit@googlegroups.com \
    --cc=sunshine@sunshineco.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).