New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
locale.t on Win32 failing "Verify that strings with embedded NUL collate" and "Verify that strings with embedded NUL and extra trailing NUL collate" #15444
Comments
From @bulk88Created by @bulk88------------------------------------------------------ Test Summary Report PERL_DEBUG_FULL_TEST run attached. Perl Info
|
From @bulk88 |
From @khwilliamsonOn 07/15/2016 04:18 PM, bulk88 (via RT) wrote:
620f73f may have fixed this. Please |
The RT System itself - Status changed from 'new' to 'open' |
From @bulk88[log 9648eab Add epigraphs for 5.22.3-RC2 and 5.24.1-RC2.txt](https://rt-archive.perl.org/perl5/Ticket/Attachment/1412499/763073/log 9648eab Add epigraphs for 5.22.3-RC2 and 5.24.1-RC2.txt) |
From @khwilliamsonOn 07/25/2016 05:57 PM, bulk88 via RT wrote:
Thanks for this. Please rerun it, adding -DLv |
From @bulk88On Mon Jul 25 17:06:42 2016, public@khwilliamson.com wrote:
Recompiled perl with debugging and did -DLv see attached. -- |
From @bulk88[log 9648eab Add epigraphs for 5.22.3-RC2 and 5.24.1-RC2.txt](https://rt-archive.perl.org/perl5/Ticket/Attachment/1412529/763098/log 9648eab Add epigraphs for 5.22.3-RC2 and 5.24.1-RC2.txt) |
From @bulk88On Mon Jul 25 18:01:39 2016, bulk88 wrote:
7.8 MB of text in my attachment, yeah, that post was rejected from the ML... -- |
From @khwilliamsonOn 07/25/2016 07:22 PM, bulk88 via RT wrote:
Thanks. This showed up two problems. One is that there was unintended The other is that your platform's strxfrm() is returning a length that I'm not sure if this will fix the problem. It would be helpful if you |
From @khwilliamson |
From @bulk88On Tue Jul 26 15:32:16 2016, public@khwilliamson.com wrote:
I'll build it later, for right now, i googled up this https://github.com/apache/stdcxx/blob/4.1.3/src/collate.cpp#L477 but I can't find the ticket associated with "29935" for that project. MSVC 2003 calls itself 1310, so based on "#if defined _MSC_VER && _MSC_VER < 1400" the bug was fixed in VC 2005 and newer. -- |
From @bulk88On Tue Jul 26 15:32:16 2016, public@khwilliamson.com wrote:
Your khw-locale branch fixes the test fail. -- |
From @khwilliamsonOn Tue Jul 26 19:41:54 2016, bulk88 wrote:
Ok, but I'll need to look at the log, now that the other issues (should) have been cleaned up |
From @bulk88On Tue Jul 26 21:37:02 2016, khw wrote:
The one already in this ticket or do I have to make a new one? -- |
From @khwilliamsonOn 07/27/2016 02:42 PM, bulk88 via RT wrote:
Make a new one, and to shorten its length, you can use the modified |
From @khwilliamsonBlead has several changes in it that have fixed similar failures on other platforms. Let me know if that fixes your example. If not please rerun the tests cd t and attach the output to this ticket. -- |
From @cpansproutOn Tue Aug 02 18:46:18 2016, khw wrote:
Surely you mean ../perl, not myperl? (Or just ./perl after ‘make test’.)
-- Father Chrysostomos |
From @khwilliamsonOn 08/02/2016 08:56 PM, Father Chrysostomos via RT wrote:
Darn. This is the 3rd time in 2 days I made this mistake. s!myperl!./perl -I../lib!
|
From @bulk88On Tue Aug 02 18:46:18 2016, khw wrote:
Blead at Revision: c7202de This also involves moving some complicated debugging statements to a passes for me in locale.t Attaching the file you wanted. -- |
From @khwilliamsonThe OP says this is now fixed |
@khwilliamson - Status changed from 'open' to 'pending release' |
From @khwilliamsonThank you for filing this report. You have helped make Perl better. With the release today of Perl 5.26.0, this and 210 other issues have been Perl 5.26.0 may be downloaded via: If you find that the problem persists, feel free to reopen this ticket. |
@khwilliamson - Status changed from 'pending release' to 'resolved' |
Migrated from rt.perl.org#128629 (status was 'resolved')
Searchable as RT128629$
The text was updated successfully, but these errors were encountered: