develooper Front page | perl.perl5.porters | Postings from July 2022

Re: perl-5.37.1 breaks texinfo-6.8

Thread Previous | Thread Next
Uwe Düffert
July 14, 2022 21:15
Re: perl-5.37.1 breaks texinfo-6.8
Message ID:
On Wed, 13 Jul 2022, Karl Williamson wrote:

> I am unable to reproduce your problem.  I have gone back to the earliest 
> 5.37.1 and used gcc 12, and nothing.  valgrind shows it is failing in 
> initialization.  I have put together a group of patches some of which fix 
> issues with initialization.
> Attached is a patch of all of the things since 5.37.1, so it should apply 
> cleanly to that source.
Yes, it does apply cleanly - but does not fix the problem either.

At first glance, I thought it got at least better, because valgrind on the 
reduced texinfo testcase only reports 4 out of 5 Invalid reads:

But that's only because the reduced test case does not run through 
anymore, perl-5.37.1 with that 98kb patch already panics before reaching 
the line that would like to output the broken saved_LC_ALL2:

On the other hand, that might bring us closer to the problem. So I 
recompiled perl-5.37.1 with that 98kb patch with -DDEBUGGING and rerun the 
reduced texinfo testcase. As before, running it with -DDEBUGGING as well 
running it with valgrind makes the testcase run through (without any 
invalid char panic):

I notice that 1) either texinfos perl code or the compiling perl did 
change LC_NUMERIC inbetween and 2) that the output or rerunning it with 
-DLv is pretty short:

In there, there is not much happening between saved_LC_ALL1 and 
saved_LC_ALL2, and without -DDEBUGGING the patched perl is already 
panicing before reaching saved_LC_ALL2.

With a newly built perl, the texinfo testcase is modified each time, i.e. 
the tp folder content differs and would complain about mismatch, so for 
above builds the regenerated reduced tastcase is:

I will most likely be away from keyboard for the next 10 days, so do not 
expect further response in that time.
Good luck!

Thread Previous | Thread Next Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at | Group listing | About