Project

General

Profile

Actions

Bug #5859

closed

incorrect return value of Pathname.realdirpath of Pathname objects created by Pathname.entries

Added by tommylommykins (Thomas Green) almost 13 years ago. Updated almost 13 years ago.

Status:
Closed
Assignee:
-
Target version:
ruby -v:
ruby 1.9.3p0 (2011-10-30 revision 33570) [x86_64-linux]
Backport:
[ruby-core:41959]

Description

It appears when calling realdirpath on a Pathname object returned by Pathname#entries, the returned value is always the current working directory of the ruby process, instead of the location of the file in the filesystem
This disagrees with the rdoc description of Pathname#realdirpath, which does state that realdirpath will return the absolute name of the file.

Tho following code demonstrates the issue:

mkdir /tmp/thing
touch /tmp/thing/a
cd ~
ruby-1.9.3-p0 -ve "require 'pathname'; puts Pathname.new('/tmp/thing').entries.sort[2].realdirpath"

This is the output from the ruby process:

ruby 1.9.3p0 (2011-10-30 revision 33570) [x86_64-linux]
/home/t/a

According to my interpretation of the description in rdoc, the correct output should be "/tmp/thing/a"
http://www.ruby-doc.org/stdlib-1.9.3/libdoc/pathname/rdoc/Pathname.html#method-i-realdirpath

Updated by kosaki (Motohiro KOSAKI) almost 13 years ago

A following modified script tell us why the script don't work as you expect. Pathname#entries return directly entrie, not absolute path. then, it forgot /tmp/thing.

% ./ruby--trunk -e "require 'pathname'; p Pathname.new('/tmp/thing').entries.sort[2]"
#Pathname:a

It is not a bug. just misleading spec.

Updated by Eregon (Benoit Daloze) almost 13 years ago

I was going to answer but kosaki just did it.
Anyway, #entries always return filenames.
#children return what you would expect by default(with_directory=true):
ruby -e "require 'pathname'; puts Pathname.new('/tmp/thing').children.first.realdirpath" # => /tmp/thing/a

Although it would not work either if you used a relative path and chdir'd, in which case you should #expand_path before.

I think it would be worth to add an example to Pathname#entries and I'm unsure if such low-level method is useful at all in Pathname.

Updated by ayumin (Ayumu AIZAWA) almost 13 years ago

  • Category set to doc
  • Priority changed from Normal to 3
  • Target version set to 2.0.0
Actions #4

Updated by akr (Akira Tanaka) almost 13 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r34246.
Thomas, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


Updated by Eregon (Benoit Daloze) almost 13 years ago

Thank you Akira for improving the documentation.

I would have gladly helped as I made the suggestion, but I'm unfortunately pretty busy right now.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0