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

Re: perl-5.37.1 breaks texinfo-6.8

Thread Previous | Thread Next
From:
Uwe Düffert
Date:
July 14, 2022 21:15
Subject:
Re: perl-5.37.1 breaks texinfo-6.8
Message ID:
alpine.DEB.2.21.2207142227520.2384@uwe-dueffert.de
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:

https://uwe-dueffert.de/code/perl5.37.1/142_reduced_valgrind.sh.log
->
https://uwe-dueffert.de/code/perl5.37.1/142_reduced+p98k_valgrind.sh.log

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:
https://uwe-dueffert.de/code/perl5.37.1/142_reduced+p98k_output.sh.log

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):
https://uwe-dueffert.de/code/perl5.37.1/142_reduced+p98k+DEBUGGING_output.sh.log

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:
https://uwe-dueffert.de/code/perl5.37.1/142_reduced+p98k+DEBUGGING_DLv.sh.log

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:
https://uwe-dueffert.de/code/perl5.37.1/texinfo-6.8_reduced_problem+p98k.tar.xz
https://uwe-dueffert.de/code/perl5.37.1/texinfo-6.8_reduced_problem+p98k+DEBUGGING.tar.xz


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


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About