simplify smpirun script (and kill some perl)

[simgrid.git] / src / smpi / smpirun.in

diff --git a/src/smpi/smpirun.in b/src/smpi/smpirun.in
index 8d6d8ad..72d3819 100755 (executable)
--- a/src/smpi/smpirun.in
+++ b/src/smpi/smpirun.in
@@ -299,15 +299,16 @@ fi
 if [ -z "${HOSTFILE}" ] ; then
     HOSTFILETMP=1
     HOSTFILE="$(mktemp smpitmp-hostfXXXXXX)"
-    perl -ne 'print "$1\n" if /.*<host.*?id="(.*?)".*?\/>.*/' ${PLATFORM} > ${HOSTFILE}
-    # put all <cluster tag on its own line.
-    python -c '
+    @PYTHON_EXECUTABLE@ -c '
 import xml.etree.ElementTree as ET
 import sys
 import re
 
 tree = ET.parse(sys.stdin)
 
+for elem in tree.findall(".//host"):
+    print(elem.attrib["id"])
+
 for elem in tree.findall(".//cluster"):
     prefix = elem.attrib["prefix"]
     radical = elem.attrib["radical"]
@@ -316,19 +317,29 @@ for elem in tree.findall(".//cluster"):
         m = re.match("^([^-]*?)-([^-]*)$", r)
         if m:
             for i in range(int(m.group(1)), int(m.group(2))):
-                print "{}{}{}".format(prefix, i, suffix)
+                print(prefix + str(i) + suffix)
         else:
-            print "{}{}{}".format(prefix, r, suffix)
-            ' < ${PLATFORM} >> ${HOSTFILE}
+            print(prefix + r + suffix)
+            ' < ${PLATFORM} > ${HOSTFILE}
 fi
 UNROLLEDHOSTFILETMP=0
 
-#parse if our lines are terminated by :num_process
-multiple_processes=$(grep -c ":" $HOSTFILE)
-if [ "${multiple_processes}" -gt 0 ] ; then
+# parse if our lines are terminated by :num_process
+if grep -q ':' $HOSTFILE ; then
     UNROLLEDHOSTFILETMP=1
     UNROLLEDHOSTFILE="$(mktemp smpitmp-hostfXXXXXX)"
-    perl -ne ' do{ for ( 1 .. $2 ) { print "$1\n" } } if /(.*?):(\d+).*/'  ${HOSTFILE}  > ${UNROLLEDHOSTFILE}
+    @PYTHON_EXECUTABLE@ -c '
+import sys
+import re
+
+for line in sys.stdin:
+    m = re.match("(.*):(.*)", line)
+    if m:
+        for i in range(0, int(m.group(2))):
+            print(m.group(1))
+    else:
+        print(line.strip())
+' < ${HOSTFILE}  > ${UNROLLEDHOSTFILE}
     if [ ${HOSTFILETMP} = 1 ] ; then
         rm ${HOSTFILE}
         HOSTFILETMP=0