diff options
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | dir.c | 22 | ||||
-rw-r--r-- | test/ruby/test_dir.rb | 21 |
3 files changed, 47 insertions, 2 deletions
@@ -1,3 +1,9 @@ +Mon Feb 3 10:49:48 2014 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * dir.c (glob_helper): return the filename with actual cases on + the filesystem if it is case-insensitive. [ruby-core:42469] + [Feature #5994] + Sun Feb 2 05:48:42 2014 Eric Wong <e@80x24.org> * io.c (rb_io_syswrite): add RB_GC_GUARD @@ -1397,14 +1397,29 @@ glob_helper( if (exist == NO || isdir == NO) return 0; - if (magical || recursive) { + if (magical || recursive || ((FNM_SYSCASE || HAVE_HFS) && plain)) { struct dirent *dp; DIR *dirp; IF_HAVE_HFS(int hfs_p); dirp = do_opendir(*path ? path : ".", flags, enc); - if (dirp == NULL) return 0; + if (dirp == NULL) { +# if FNM_SYSCASE || HAVE_HFS + if (!(magical || recursive) && (errno == EACCES)) { + /* no read permission, fallback */ + goto literally; + } +# endif + return 0; + } IF_HAVE_HFS(hfs_p = is_hfs(dirp)); +# if HAVE_HFS + if (!(hfs_p || magical || recursive)) { + closedir(dirp); + goto literally; + } + flags |= FNM_CASEFOLD; +# endif while ((dp = READDIR(dirp, enc)) != NULL) { char *buf; enum answer new_isdir = UNKNOWN; @@ -1486,6 +1501,9 @@ glob_helper( else if (plain) { struct glob_pattern **copy_beg, **copy_end, **cur2; +# if FNM_SYSCASE || HAVE_HFS + literally: +# endif copy_beg = copy_end = GLOB_ALLOC_N(struct glob_pattern *, end - beg); if (!copy_beg) return -1; for (cur = beg; cur < end; ++cur) diff --git a/test/ruby/test_dir.rb b/test/ruby/test_dir.rb index 6054b8f7fe..b0d848c83f 100644 --- a/test/ruby/test_dir.rb +++ b/test/ruby/test_dir.rb @@ -228,6 +228,27 @@ class TestDir < Test::Unit::TestCase assert_empty(Dir.glob(File.join(@root, "<")), bug8597) end + def test_glob_cases + feature5994 = "[ruby-core:42469] [Feature #5994]" + feature5994 << "\nDir.glob should return the filename with actual cases on the filesystem" + Dir.chdir(File.join(@root, "a")) do + open("FileWithCases", "w") {} + return unless File.exist?("filewithcases") + assert_equal(%w"FileWithCases", Dir.glob("filewithcases"), feature5994) + end + Dir.chdir(File.join(@root, "c")) do + open("FileWithCases", "w") {} + mode = File.stat(".").mode + begin + File.chmod(mode & ~0444, ".") + return if mode == File.stat(".").mode + assert_equal(%w"filewithcases", Dir.glob("filewithcases"), feature5994) + ensure + File.chmod(mode, ".") + end + end + end + def test_home env_home = ENV["HOME"] env_logdir = ENV["LOGDIR"] |