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
Blead Breaks CPAN: JETTERO/Games-RolePlay-MapGen-1.5008.tar.gz #16405
Comments
From @eserteThis is a bug report for perl from slaven@rezic.de, The test suite of Games-RolePlay-MapGen-1.5008 started to fail And it still fails with perl5.27.8-157-gef80cd9 Flags: Site configuration information for perl 5.27.9: Configured by eserte at Tue Feb 6 19:16:51 CET 2018. Summary of my perl5 (revision 5 version 27 subversion 9) configuration: @INC for perl 5.27.9: Environment for perl 5.27.9: |
From zefram@fysh.orgBisects to commit a155eb0 "(perl #131895) The underlying issue is that the Games::RolePlay::MapGen module has a Prior to 5.27.6, a string containing a nul (as a Storable data string When a string containing nul is rejected as a pathname Perl emits There is nothing for the core to change here. It is for the module to -zefram |
The RT System itself - Status changed from 'new' to 'open' |
From @jkeenanI started to look at this and was tracking down the problem when I saw Zefram's post. I agree with his diagnosis. In particular, commenting out 'use common::sense;' throughout enables all tests to pass on Perl 5 blead. I have opened a ticket in the module's bug tracker. Resolving ticket as Rejected. Thank you very much. -- |
@jkeenan - Status changed from 'open' to 'rejected' |
From @xsawyerxOn 02/06/2018 10:27 PM, Zefram wrote:
Any chance you could provide a suggestion on how to cleanly resolve it |
From zefram@fysh.orgSawyer X wrote:
There are many options: * check for nul in the string before -f, always treating nul-containing * discriminate between pathnames and data entirely based on looking like * suppress that category of warning in some narrow scope around -f; * drop the dual-use interface in favour of separate interfaces for -zefram |
From @xsawyerxOn 02/08/2018 02:25 PM, Zefram wrote:
This is helpful. Thank you. I've added them to the ticket upstream. |
Migrated from rt.perl.org#132822 (status was 'rejected')
Searchable as RT132822$
The text was updated successfully, but these errors were encountered: