-
Notifications
You must be signed in to change notification settings - Fork 11
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
autodie breaks the open set by utf8::all #7
Comments
Thank you for this! Do you have any idea what the underlying problem is (or the solution)? |
That was actually reported for autodie before, but apparently didn't draw enough interest. And there's an explanation, plus maybe a solution, but i actually do not understand it: https://rt.cpan.org/Ticket/Display.html?id=54777 |
I don't understand the solution provided there either, but that looks like it is actually an issue with autodie, so I'm closing this and pinging the autodie ticket. |
Fine with me. I hope something comes out of that. :) |
autodie 2.12 fixes the interaction with open - we should require that version as a minimum. |
autodie obliterated the IO layers set by the open pragma prior to version 2.12, so now we refuse to run with autodie before that bug was fixed. RT #54777: autodie loses open pragma GH #7: autodie breaks the open set by utf8::all
autodie obliterated the IO layers set by the open pragma prior to version 2.12, so now we refuse to run with autodie before that bug was fixed. RT #54777: autodie loses open pragma GH #7: autodie breaks the open set by utf8::all
Fixed in 0.005. |
I've written a test to demonstrate the problem: https://gist.github.com/1103863
utf8::all works fine, but when autodie is invoked, the encoding is stripped from open().
In order to avoid browser shenanigans it's probably best to get the test via git:
git clone git://gist.github.com/1103863.git gist-1103863
The text was updated successfully, but these errors were encountered: