Adding sort=true keyword to readdir()

NEWS.md

@@ -136,6 +136,9 @@ Language changes
     backslashes and the end of the literal while 2n+1 backslashes followed by a quote encodes n
     backslashes followed by a quote character ([#22926]).
 
+  * `readdir()` output is now guaranteed to be sorted on any OS. The keyword `sort` allows to
+    optionally turn off explicit sorting ([#24626]).
+
 Breaking changes
 ----------------
 

base/file.jl

@@ -401,11 +401,12 @@ struct uv_dirent_t
 end
 
 """
-    readdir(dir::AbstractString=".") -> Vector{String}
+    readdir(dir::AbstractString="."; sort::Bool=true) -> Vector{String}
 
 Returns the files and directories in the directory `dir` (or the current working directory if not given).
+If `sort=false` the resulting list won't necessarily be sorted.
 """
-function readdir(path::AbstractString)
+function readdir(path::AbstractString; sort::Bool=true)
     # Allocate space for uv_fs_t struct
     uv_readdir_req = zeros(UInt8, ccall(:jl_sizeof_uv_fs_t, Int32, ()))
 
@@ -425,6 +426,9 @@ function readdir(path::AbstractString)
     # Clean up the request string
     ccall(:jl_uv_fs_req_cleanup, Void, (Ptr{UInt8},), uv_readdir_req)
 
+    # sort entries if not requested otherwise
+    sort && sort!(entries)
+
     return entries
 end
 

test/file.jl

@@ -959,6 +959,19 @@ cd(dirwalk) do
 end
 rm(dirwalk, recursive=true)
 
+###################
+#     readdir     #
+###################
+
+mktempdir() do path
+    cd(path) do
+        for k in 1:10
+            touch(randstring())
+        end
+        @test issorted(readdir())
+    end
+end
+
 ############
 # Clean up #
 ############