Browse Source

Merge pull request #39125 from Xrayez/py-modules-order-4.0

SCons: use `OrderedDict` to ensure insertion order of modules
Rémi Verschelde 5 years ago
parent
commit
105bef19ff
2 changed files with 6 additions and 4 deletions
  1. 4 3
      SConstruct
  2. 2 1
      methods.py

+ 4 - 3
SConstruct

@@ -8,6 +8,7 @@ import glob
 import os
 import pickle
 import sys
+from collections import OrderedDict
 
 # Local
 import methods
@@ -181,7 +182,7 @@ for k in platform_opts.keys():
         opts.Add(o)
 
 # Detect modules.
-modules_detected = {}
+modules_detected = OrderedDict()
 module_search_paths = ["modules"]  # Built-in path.
 
 if ARGUMENTS.get("custom_modules"):
@@ -523,11 +524,11 @@ if selected_platform in platform_list:
     sys.path.remove(tmppath)
     sys.modules.pop("detect")
 
-    modules_enabled = {}
+    modules_enabled = OrderedDict()
     env.module_icons_paths = []
     env.doc_class_path = {}
 
-    for name, path in sorted(modules_detected.items()):
+    for name, path in modules_detected.items():
         if not env["module_" + name + "_enabled"]:
             continue
         sys.path.insert(0, path)

+ 2 - 1
methods.py

@@ -2,6 +2,7 @@ import os
 import re
 import glob
 import subprocess
+from collections import OrderedDict
 
 
 def add_source_files(self, sources, files, warn_duplicates=True):
@@ -138,7 +139,7 @@ def parse_cg_file(fname, uniforms, sizes, conditionals):
 
 
 def detect_modules(at_path):
-    module_list = {}  # name : path
+    module_list = OrderedDict()  # name : path
 
     modules_glob = os.path.join(at_path, "*")
     files = glob.glob(modules_glob)