Path-Tiny-0.028 Path::Tiny on win32 turns C:/ or C:\ into C: but those aren't equivalent #49

Closed
ghost opened this Issue Aug 18, 2013 · 3 comments

Comments

Projects
None yet
1 participant
@ghost

ghost commented Aug 18, 2013

http://perlmonks.org/?node_id=1049892 # Path-Tiny-0.028 Path::Tiny on win32 turns C:/ or C:\ into C: / absolute / realpath are broken

Editing sub path to comment out the following line seems to fix it :)

$path =~ s{/$}{} if $path ne "/"; # hack to make splitpath give us a basename

@ghost

ghost commented Aug 19, 2013

Thanks for the report. What version of Perl? What version of File::Spec and Cwd?

@ghost

ghost commented Aug 19, 2013

(as asked on Perlmonks)

Is there any reason to allow a raw volume like "C:" (no trailing slash) or "\server\share" as a Path::Tiny "path"? Or should I be fixing those to have a trailing slash so they are properly understood as referencing the root directory of those volumes?

@ghost

ghost commented Aug 19, 2013

Fixed and released to CPAN as 0.029

@ghost ghost closed this Aug 19, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment