Thread (2 messages) flat view 2 messages, 2 authors, 2016-06-15

Re: git on Cygwin: Not a valid object name HEAD

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:43:28

Linus Torvalds [off-list ref] writes:
On Tue, 7 Aug 2007, Steffen Prohaska wrote:
quoted
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.
quoted hunk
@@ -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 ;-).
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help