Re: [PATCH] mingw: abort on invalid strftime formats

2018-03-20 Thread Johannes Schindelin
Hi Junio, On Mon, 19 Mar 2018, Junio C Hamano wrote: > Johannes Schindelin writes: > > > On Windows, strftime() does not silently ignore invalid formats, but > > warns about them and then returns 0 and sets errno to EINVAL. > > > > Unfortunately, Git does not expect

Re: [PATCH] mingw: abort on invalid strftime formats

2018-03-19 Thread Junio C Hamano
Johannes Schindelin writes: > On Windows, strftime() does not silently ignore invalid formats, but > warns about them and then returns 0 and sets errno to EINVAL. > > Unfortunately, Git does not expect such a behavior, as it disagrees > with strftime()'s semantics on

[PATCH] mingw: abort on invalid strftime formats

2018-03-19 Thread Johannes Schindelin
On Windows, strftime() does not silently ignore invalid formats, but warns about them and then returns 0 and sets errno to EINVAL. Unfortunately, Git does not expect such a behavior, as it disagrees with strftime()'s semantics on Linux. As a consequence, Git misinterprets the return value 0 as "I