Merge pull request #626 from amatsuda/sort_manifest_entries

Sort manifest entries alphabetically before choosing one
rails · Sep 17, 2019 · cbf3b82 · cbf3b82
2 parents 1bcee8e + b416821
commit cbf3b82
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/lib/sprockets/manifest_utils.rb b/lib/sprockets/manifest_utils.rb
@@ -38,6 +38,7 @@ def find_directory_manifest(dirname, logger = Logger.new($stderr))
       entries = File.directory?(dirname) ? Dir.entries(dirname) : []
       manifest_entries = entries.select { |e| e =~ MANIFEST_RE }
       if manifest_entries.length > 1
+        manifest_entries.sort!
         logger.warn("Found multiple manifests: #{manifest_entries}. Choosing the first alphabetically: #{manifest_entries.first}")
       end
       entry = manifest_entries.first || generate_manifest_path