Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 04/15] Set _O_BINARY as default fmode for both MinGW and MSVC

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:47:25

Johannes Sixt [off-list ref] writes:
Just for the records: The reason why including git-compat-util.h was not
necessary before 04/15 is that test-genrandom is linked against libgit.a.
This way the startup code had picked up the definition of _CRT_fmode from
mingw.c that is initialized to _O_BINARY. After the original 04/15 this
symbol was not present anymore in libgit.a, and the default (text mode)
was used. And with this fix, main() is overridden to explicitly set the
mode of stdout to _O_BINARY.
Beautiful.  I think it is worth mentioning some of the above in the commit
log message.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help