Re: [PATCH 1/1] Use correct /dev/null for UNIX and Windows
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Oct 31, 2018, 04:48 UTC
- Message-ID
- <xmqqy3aewwky.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <20181031035618.GA31247@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 5 quoted lines
> On Tue, Oct 30, 2018 at 11:26:36AM -0700, chris via GitGitGadget wrote: > >> From: chris <chris@webstech.net> > > You might want to adjust your user.name. :)
Yes, absolutely. We'd want to see that the From: line and one of the Signed-off-by: lines are idential.
Show 6 quoted lines
>> Use File::Spec->devnull() for output redirection to avoid messages >> when Windows version of Perl is first in path. The message 'The >> system cannot find the path specified.' is displayed each time git is >> run to get colors. > > Thanks, makes sense, and the patch looks good to me.
Yup, and we already use File::Spec everywhere anyway, so this is not a new dependency, either. Which is very good.