Re: [PATCH] Use mingw.h declarations for gmtime_r/localtime_r on msys2

From
Carlo Arenas <hidden>
Date
2021-11-19 09:37:03
Reply to
Mike Hommey, 2021-11-19
On Thu, Nov 18, 2021 at 11:24 PM Mike Hommey [off-list ref] wrote:
The ifdef around gmtime_r and localtime_r in mingw.h is for __MINGW64_VERSION_MAJOR.
The ifdef around _POSIX_C_SOURCE in git-compat-util.h is for
__MINGW64__.
I'd imagine that plays a role.
Must be, and indeed you are right that my 32-bit compiler doesn't set
__MINGW64__ (only __MINGW32__) and I am not hitting that codepath, but
I have no problem building and it seems it is because
_POSIX_THREAD_SAFE_FUNCTIONS it is defined unconditionally at the end
of pthread_unistd.h, unlike what is done for x86_64.

somehow your version of headers might had that removed in both, and
that was "fixed" later.  I have 6346 in i386 and 6306 in x86_64.

Carlo
Thread (11 messages) 11 messages, 2 authors, last reply 2021-11-19
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help