From bed45d2b52e094786b4d6ad0de9a14b39b7bb382 Mon Sep 17 00:00:00 2001
From: Martin Quinson <martin.quinson@ens-rennes.fr>
Date: Wed, 10 Jul 2019 01:19:45 +0200
Subject: [PATCH] simplify smpirun script (and kill some perl)

---
 src/smpi/smpirun.in | 25 ++++++++++++++++++-------
 1 file changed, 18 insertions(+), 7 deletions(-)

diff --git a/src/smpi/smpirun.in b/src/smpi/smpirun.in
index d095d29778..72d381929d 100755
--- a/src/smpi/smpirun.in
+++ b/src/smpi/smpirun.in
@@ -299,8 +299,6 @@ 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_EXECUTABLE@ -c '
 import xml.etree.ElementTree as ET
 import sys
@@ -308,6 +306,9 @@ 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"]
@@ -319,16 +320,26 @@ for elem in tree.findall(".//cluster"):
                 print(prefix + str(i) + suffix)
         else:
             print(prefix + r + suffix)
-            ' < ${PLATFORM} >> ${HOSTFILE}
+            ' < ${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
-- 
2.20.1