Use current directory's root for Dir.glob("/...") on Windows (cryst…

…al-lang#13628)
Blacksmoke16 · Dec 11, 2023 · d89ce7b · d89ce7b
1 parent 0db5037
commit d89ce7b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/spec/std/dir_spec.cr b/spec/std/dir_spec.cr
@@ -428,6 +428,7 @@ describe "Dir" do
     it "root pattern" do
       {% if flag?(:windows) %}
         Dir["C:/"].should eq ["C:\\"]
+        Dir["/"].should eq [Path[Dir.current].anchor.not_nil!.to_s]
       {% else %}
         Dir["/"].should eq ["/"]
       {% end %}

diff --git a/src/dir/glob.cr b/src/dir/glob.cr
@@ -375,12 +375,7 @@ class Dir
     end
 
     private def self.root
-      # TODO: better implementation for windows?
-      {% if flag?(:windows) %}
-        "C:\\"
-      {% else %}
-        File::SEPARATOR_STRING
-      {% end %}
+      Path[Dir.current].anchor.not_nil!.to_s
     end
 
     private def self.dir?(path, follow_symlinks)