search

kenahoo / Path-Class

Cross-platform path specification manipulation
http://search.cpan.org/dist/Path-Class/
15 stars 28 forks source link

fix tests failing on Windows #12

Closed szabgab closed 11 years ago

kenahoo commented 11 years ago

In general, it should be possible to write tests that don't care what platform they're on - if not, it indicates some failing of the module, and suggests that people using the module will have to write similar special-case code. I wonder if there's some other fix?

szabgab commented 11 years ago

I agree with you, but I am not familiar with the module - it is only a dependency for half of CPAN I was trying to install. Maybe all the path separators need to be internally converted to forward slash?

szabgab commented 11 years ago

See new change. Maybe you like that better.

kenahoo commented 11 years ago

We can't do that internal change, the native syntax must be used when stringifying.

I just committed a different fix that should fix the issue, and it adds another method to the API.