From: Paul RASCLE <paul.rascle@edf.fr>
Date: Wed, 12 Dec 2018 15:09:43 +0000 (+0100)
Subject: Module separator for variable SALOME_MODULES_ORDER is changed from ',' to ':' because... 
X-Git-Tag: V9_3_0a1~7
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=e28c954f8038a6ce9409509b93e958d2e375ef10;p=modules%2Fkernel.git

Module separator for variable SALOME_MODULES_ORDER is changed from ',' to ':' because it is processed in parseConfigFile.py (KERNEL)
and ',' is a separator: duplicates are removed with a sequence "split, set, join" and the set let the variables in a random order.
When we have SALOME_MODULES_ORDER=SHAPER,GEOM,SMESH we can get SMESH,GEOM,SHAPER as a resulting order and the dump is wrong.
We must now write:
SALOME_MODULES_ORDER="SHAPER:GEOM:SMESH"
---

diff --git a/src/SALOMEDSImpl/SALOMEDSImpl_Study.cxx b/src/SALOMEDSImpl/SALOMEDSImpl_Study.cxx
index 56718e43f..c8d893cf7 100644
--- a/src/SALOMEDSImpl/SALOMEDSImpl_Study.cxx
+++ b/src/SALOMEDSImpl/SALOMEDSImpl_Study.cxx
@@ -2065,7 +2065,7 @@ bool SALOMEDSImpl_Study::DumpStudy(const std::string& thePath,
     std::string order = getenv("SALOME_MODULES_ORDER");
     std::vector<std::string> mlist;
     while ( !order.empty() ) {
-      size_t idx = order.find( "," );
+      size_t idx = order.find( ":" );
       std::string m = order.substr(0, idx);
       order = order.substr( ( idx == std::string::npos ) ? order.size() : idx+1 );
       if ( m.empty() || std::find( mlist.begin(), mlist.end(), m ) != mlist.end() ) continue;