Project

General

Profile

Backport #3882

REGRESSION: File.extname() broken (at least >= 1.8.7)

Added by AndiM (Andreas Mohr) almost 9 years ago. Updated almost 7 years ago.

Status:
Assigned
Priority:
Normal
[ruby-core:32599]

Description

=begin
Hi,

#!/usr/bin/ruby

require 'pathname'

mytest_filename = ARGV.shift

#mytest_filename_cleaned = Pathname.new(mytest_filename).cleanpath
mytest_filename_cleaned = mytest_filename

puts "mytest_filename #{mytest_filename} mytest_filename_cleaned #{mytest_filename_cleaned}"

myext = File.extname(mytest_filename_cleaned)
puts "mytest_filename_cleaned #{mytest_filename_cleaned} myext #{myext}"

Try .//test.out
Won't do what it should, at least on u10.04 ruby1.8 1.8.7.249-2 and ruby1.9.1 1.9.1.378-1,
as opposed to correct behaviour on ruby-1.8.5-5.el5_4.8.

Since http://ruby-doc.org/core/classes/File.html#M002543 doesn't really specify extname() argument composition constraints
(other than it being a path), it should accept duplicate slashes since this syntax is treated as a valid path pretty much everywhere.

Do you happen to have a test suite for these core API things? Since extname() also had some other issues (spaces before extension etc.),
this would be useful.

Other than this rather disturbing behaviour, I'm VERY impressed with Ruby.

Thanks,

Andreas Mohr
=end

History

#1

Updated by usa (Usaku NAKAMURA) almost 9 years ago

  • Status changed from Open to Assigned
  • Assignee set to wyhaines (Kirk Haines)
  • Priority changed from 5 to Normal

=begin
It's already fixed at 1.8.7-p302.

BTW, ruby_1_8_6 branch has this problem yet.
=end

#2

Updated by nobu (Nobuyoshi Nakada) almost 9 years ago

=begin
Backport r27404.
=end

#3

Updated by nobu (Nobuyoshi Nakada) almost 9 years ago

  • Category set to core

=begin

=end

Also available in: Atom PDF