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: Dancer2 #16454
Comments
From carlos@carlosguevara.comThis is a bug report for perl from "Carlos Guevara" <carlos@carlosguevara.com>, It looks like blead broke Dancer2: Flags: Site configuration information for perl 5.27.10: Configured by root at Thu Mar 8 00:53:46 CST 2018. Summary of my perl5 (revision 5 version 27 subversion 10) configuration: @INC for perl 5.27.10: Environment for perl 5.27.10: |
From @dur-randirOn Wed, 07 Mar 2018 23:21:55 -0800, carlos@carlosguevara.com wrote:
I rather suspect a bug in Dancer2 itself. For me, version 0.201000 passes tests on blead and version 0.202000 fails t/request.t as far back as 5.18. I didn't dig further. |
The RT System itself - Status changed from 'new' to 'open' |
From @jkeenanOn Thu, 08 Mar 2018 09:45:55 GMT, randir wrote:
I agree that this is more likely a case where, rather than "blead breaks CPAN", blead exposes sub-optimal code in CPAN. The relevant code in Dancer2 is this: ##### If $value is undefined within a given cookie -- as it explicitly is in one case (used 3 times) in t/request.t -- then it is not surprising that Dancer2::Core::Cookie->new() would fail. I have submitted this pull request: The p.r. clears up the exception and permits t/request.t to run to conclusion. Three unit tests fail -- but I think that's due to how the tests are written and I'm leaving it up to the Dancer2 community to figure out how to address that. I'll leave this ticket open, but I myself am *not* going to mark this as a blocker for 5.28.0. Thank you very much. |
From peter@sysnix.comThis is a known issue in Dancer2 PerlDancer/Dancer2#1435 and not a BBC so feel free to close this issue. |
@jkeenan - Status changed from 'open' to 'rejected' |
Migrated from rt.perl.org#132950 (status was 'rejected')
Searchable as RT132950$
The text was updated successfully, but these errors were encountered: