Re: toolchain/56102: libstc++ missing locale_t if _XOPEN_SOURCE defined

[Tobias Ulmer <tobiasu%tmux.org@localhost>]

The following reply was made to PR toolchain/56102; it has been noted by GNATS.

From: Tobias Ulmer <tobiasu%tmux.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: toolchain/56102: libstc++ missing locale_t if _XOPEN_SOURCE
 defined
Date: Mon, 12 Apr 2021 03:35:55 +0200

 On Mon, Apr 12, 2021 at 12:50:01AM +0000, Joerg Sonnenberger wrote:
 > The following reply was made to PR toolchain/56102; it has been noted by GNATS.
 > 
 > From: Joerg Sonnenberger <joerg%bec.de@localhost>
 > To: gnats-bugs%netbsd.org@localhost
 > Cc: 
 > Subject: Re: toolchain/56102: libstc++ missing locale_t if _XOPEN_SOURCE
 >  defined
 > Date: Mon, 12 Apr 2021 02:49:43 +0200
 > 
 >  On Mon, Apr 12, 2021 at 12:30:01AM +0000, tobiasu%tmux.org@localhost wrote:
 >  > In pkgsrc this is fixed by removing _XOPEN_SOURCE: http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/devel/abseil/patches/patch-absl_time_internal_cctz_src_time__zone__format.cc?rev=1.1&content-type=text/x-cvsweb-markup
 >  > 
 >  > But come on guys, that's not a fix. Let me paste you the error message:
 >  
 >  Source is definitely wrong. locale_t shouldn't be present for
 >  _XOPEN_SOURCE<700. Now likely should care about it for
 >  _XOPEN_SOURCE=700, but then you can just use _POSIX_C_SOURCE=200809L in
 >  that case...
 >  
 >  Joerg
 >  
 
 
 I'm not buying it. This is an internal method of libstdc++.
 It's an implementation detail that I should not have to care about even
 one bit.
 
 The reproducer doesn't even use or call anything. C++ has locales
 regardless of what C or crusty old unix standards say.
 
 But in the end it's your decision and I'm not attached enough to this
 OS to fight for it.