-
Notifications
You must be signed in to change notification settings - Fork 20
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
io->catfile( $io, $str ) fails on windows #74
Comments
|
|
adding use overload 'eq' => '_overload_stringify'; to IO::All fixes it, haarg suggested adding a fallback to _overload_stringify. |
…:Spec::Win32::catfile/catdir
|
Would'nt it be better to add a cmp overload? |
|
@xlat as long as it fixes the 'Operation "eq": no method found' I'm happy. The answer to your question as far as I'm concerned is if overloading cmp overloads eq by proxy then sure, that would be better. |
|
@ediblenergy yes, it looks like to be the case if you take a look at the first test I have added to overload.t ;) |
|
Ah cool now I see it, looks good to me :) I'll ping in #io-all because github notifications are terrible. |
|
I'll try to get a fix released for this soonish; thanks for the research all! |
|
I have this issue as well on Windows using |
|
Wes told me he'd write a test; I'll incorporate one of the fixes above, have him ensure that the test passes on win32 (and ofc I'll run on linux) and then I'll cut a rel. |
|
This is resolved in the 0.87 release and can be closed. |
Also see: http://www.cpantesters.org/cpan/report/608781a5-9264-1014-8cee-01022b825c07
For when that bug shows up from another package.
Googling around I found: http://www.nntp.perl.org/group/perl.beginners/2005/04/msg74371.html
Which somewhat suggests that overloading "" is not enough, you may have to overload 'eq' as well. That seems odd to me, but maybe.
The text was updated successfully, but these errors were encountered: