Project

General

Profile

Actions

Bug #13515

closed

Pathname#join doesn't add separator on UNC paths

Added by p0deje (Alex Rodionov) almost 7 years ago. Updated over 6 years ago.

Status:
Closed
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.

Actions #1

Updated by nobu (Nobuyoshi Nakada) almost 7 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]
Actions #2

Updated by usa (Usaku NAKAMURA) over 6 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) over 6 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) over 6 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.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0