Hi Collin, > From what I could understand, it seems that upon seeing that the mode > is "import", the specified_modules is set with the unsorted list of > modules [1]. Then that variable is sorted before it is used [2]. Could > you confirm that I am understanding this correctly?
Yes, I confirm. So, in the Python code, the corresponding place to do the canonicalization (sorting and removing duplicates) would be in GLImport.py line 258. > If so, I feel that this patch should be the correct way to handle it. > My initial ideas where to sort the modules in main() before the > GLConfig object was created, or to modify the GLConfig class's methods > which are used to set the modules in use. Well, GLConfig applies to all modes (not just 'import', but also 'create-testdir' etc.). Since on the bash side, you found that the sorting is specifically in the func_import(), the right place to do it is in GLImport.py, not GLConfig.py. The second part of your patch (removal of sorting from GLImport.actioncmd) is good, but I guess you will want to have it combined with another patch to GLImport.py. Bruno
