Re: git on Cygwin: Not a valid object name HEAD
Linus Torvalds <torvalds@linux-foundation.org> writes:
Show 16 quoted lines
> On Tue, 7 Aug 2007, Steffen Prohaska wrote:
>>
>> Is there any chance that patches would be accepted that try to
>> do so? Even if they add "b" to fopen and O_BINARY to open, which
>> both are useless on Unix?
>
> I certainly don't think it would be wrong to add O_BINARY to the open()
> parameters (and "b" to fopen() and friends), if it makes a difference.
>
> Add a
>
> #ifndef O_BINARY
> #define O_BINARY 0
> #endif
>
> and it should be harmless anywhere else.
Heh, you beat me to this --- I like it.
Show 8 quoted lines
> @@ -2023,6 +2027,12 @@ int move_temp_to_file(const char *tmpfile, const char *filename)
> ...
> +static void close_or_die(int fd, const char *file)
> +{
> + if (close(fd))
> + die("unable to close %s (%s)", file, strerror(errno));
> +}
> +And I like this even better ;-).
Junio C Hamano <gitster@pobox.com>