Project

General

Profile

Bug #13515

Pathname#join doesn't add separator on UNC paths

Added by p0deje (Alex Rodionov) over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
ruby -v:
ruby 2.3.1p112 (2016-04-26 revision 54768) [x64-mingw32]
[ruby-core:80900]

Description

When using Pathname#join on Windows, I have no problems when working with normal paths:

irb(main):008:0> Pathname.new('C:/selenium').join('common')
=> #<Pathname:C:/selenium/common>

However, when doing the same thing on UNC shares, the path is malformed:

irb(main):009:0> Pathname.new('//vboxsrv/selenium').join('common')
=> #<Pathname://vboxsrv/seleniumcommon>

As you can see, there is no separator prepending common.

I'm not sure how can I provide a reproducible test script, but let me know if there is anything else I can do to help with issue.

#1

Updated by nobu (Nobuyoshi Nakada) over 3 years ago

  • Status changed from Open to Closed

Applied in changeset trunk|r59102.


pathname.rb: UNC root pathname needs a separator

  • ext/pathname/lib/pathname.rb (Pathname#plus): UNC root pathname needs a separator. File.basename returns "/" on UNC root, as well as sole drive letter, even if it does not end with a separator. [ruby-core:80900] [Bug #13515]
#2

Updated by usa (Usaku NAKAMURA) about 3 years ago

  • Backport changed from 2.2: UNKNOWN, 2.3: UNKNOWN, 2.4: UNKNOWN to 2.2: WONTFIX, 2.3: REQUIRED, 2.4: REQUIRED

Updated by usa (Usaku NAKAMURA) about 3 years ago

  • Backport changed from 2.2: WONTFIX, 2.3: REQUIRED, 2.4: REQUIRED to 2.2: WONTFIX, 2.3: DONE, 2.4: REQUIRED

ruby_2_3 r59240 merged revision(s) 59102.

Updated by nagachika (Tomoyuki Chikanaga) about 3 years ago

  • Backport changed from 2.2: WONTFIX, 2.3: DONE, 2.4: REQUIRED to 2.2: WONTFIX, 2.3: DONE, 2.4: DONE

ruby_2_4 r59400 merged revision(s) 59102.

Also available in: Atom PDF