Re: "cannot parse date" for a seemingly valid .Dd parameter

[Ingo Schwarze <schwarze@usta.de>]

Hi Daniel,

LÉVAI Dániel wrote on Thu, Aug 28, 2014 at 10:39:00AM +0200:
> On cs, aug 28, 2014 at 02:43:42 +0200, Ingo Schwarze wrote:

>> After some code reading and some experimentation, i'm still quite
>> unsure where the problem might lie, i fail to reproduce but need
>> additional information.  Please apply the debugging patch appended
>> below (not intended for commit) to anoncvs HEAD, and run:
>> 
>>  $ ./configure
>>  $ make

> With the cvs checkout, mandoc(1) accepts the given date for me too.
> 
> config.h:
> #define HAVE_WCHAR 0

That's the reason you don't see the problem in HEAD:
wide character support is not detected.

>> Then please show me the output of the following sequence of commands:
>> 
>>  $ locale
>>  $ ./mandoc -Wall test.1 > /dev/null
>>  $ ./mandoc -Wall -Tlocale test.1 > /dev/null
 
> $ locale
> LC_TIME=hu_HU.UTF-8

Without wide character sipport, that's effectively disabled, see here:

> $ ./mandoc -Wall -Tlocale test.1 > /dev/null
> main: LC_TIME = C
> a2time: LC_TIME = C
> a2time: calling strptime()
> a2time: LC_TIME = C
> a2time: calling strptime()
> a2time: LC_TIME = C
> a2time: calling strptime()

No setlocale() here whatsoever, it doesn't even get compiled in.

> TBH, I'm a bit confused. You see, I've used 1.12.3 before,
> and I've just upgraded to 1.13.1 recently.

Sure, that would raise the question: Why did you not see the problem
in 1.12.3, and why do you see it in 1.13.1?  But maybe we don't
even have to answer that.  I just committed a small patch to improve
wide character support detection.  Can you do a CVS update, re-run
./configure and make, and re-send the output of the same commands
as before?  Just leave the debugging patches in your checkout
directory while doing all that, they are still needed.  I hope your
lost problem comes back, because i'm still unable to reproduce it,
even on 1.13.1.

> And because 1.12.3 did not have a ./configure script,
> furthermore with 1.13.1 I was not supposed to run
> ./configure directly,

Yeah, i admit that wiggled a bit back and forth until i got the
automatic configuration beat into shape.

> it seems to me, something has changed throughout the development
> that it seems has been fixed in HEAD, but exists (for me) in 1.13.1.
> Does this make sense?

No, what you have compiled from HEAD so far does not work for
you.  Your LC_* settings show you want UTF-8 support, but you
don't actually get that.  Small consolation it would be that
since you lost the feature, you lost the bug contained in it
as well.

Yours,
  Ingo
--
 To unsubscribe send an email to discuss+unsubscribe@mdocml.bsd.lv