bug#39248: format-time-string ignores user's preferred locale

[Alan Third]

On Sun, Jan 26, 2020 at 12:35:31AM -0800, Paul Eggert wrote:
> On 1/25/20 3:18 AM, Alan Third wrote:
> > On Thu, Jan 23, 2020 at 11:35:20PM -0800, Paul Eggert wrote:
> > >   #ifdef HAVE_NS
> > >     ns_pool = ns_alloc_autorelease_pool ();
> > > -#ifdef NS_IMPL_GNUSTEP
> > > -  /* GNUstep stupidly resets our locale settings after we made them.  */
> > > -  fixup_locale ();
> > > -#endif
> > 
> > Hi Paul, this LGTM, but I’m just curious if you’re sure we don’t need
> > that code above any more?
> 
> Although I don't use GNUstep and haven't tested the code, as I recall
> GNUstep initializes itself on its first call (which is why the fixup_locale
> call was there, as ns_alloc_autorelease_pool was the first call to GNUstep),
> and if so then we don't need fixup_locale there anymore since we now call
> fixup_locale earlier, just after calling ns_init_locale which calls NSLocale
> currentlocale should cause GNUstep to initialize itself.
> 
> If I'm wrong feel free to put that code back in, but in the meantime I
> installed the patch as-is, as Paul R. reports that it works for him.

No, sounds reasonable. Thanks for the explanation.
-- 
Alan Third