search

kenahoo / Path-Class

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

New version of Path::Class breaks windows paths including volume #29

Open frioux opened 10 years ago

frioux commented 10 years ago

We upgraded from Path::Class 0.26 to Path::Class 0.33. In the old version this test would pass

 is(dir('C:', 'foo')->stringify, 'C:\foo');

but now it gets

 'C:foo'

which is incorrect.

kenahoo commented 9 years ago

I'm not really sure that's incorrect. Doesn't C:foo mean foo in whatever the current working directory on the C: volume is?

frioux commented 9 years ago

Well it might, but path class didn't do that before...

-frew (sent from my HTC 6+ Galaxy bend, please pardon my brevity) On Nov 10, 2014 8:52 PM, "Ken Williams" notifications@github.com wrote:

I'm not really sure that's incorrect. Doesn't C:foo mean foo in whatever the current working directory on the C: volume is?

— Reply to this email directly or view it on GitHub https://github.com/kenahoo/Path-Class/issues/29#issuecomment-62494439.