portability: use interpreters from /usr/bin/env

default.nix

@@ -24,24 +24,14 @@ stdenv.mkDerivation {
     ];
   };
 
-  prePatch = ''
-    substituteInPlace scripts/auto-install-hpp.sh \
-      --replace-fail /bin/bash ${stdenv.shell}
-    substituteInPlace scripts/install-tar-on-remote \
-      --replace-fail /bin/bash ${stdenv.shell}
-    substituteInPlace scripts/generate-tar-doc \
-      --replace-fail /bin/sh ${stdenv.shell}
-    substituteInPlace scripts/packageDep \
-      --replace-fail /usr/bin/python ${python3Packages.python.interpreter}
-  '';
-
   strictDeps = true;
 
   nativeBuildInputs = [
     cmake
     doxygen
     libsForQt5.wrapQtAppsHook
     pkg-config
+    python3Packages.python
   ];
   buildInputs = [ libsForQt5.qtbase ];
   propagatedBuildInputs = [

scripts/auto-install-hpp.sh

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 
 # Exit on error
 set -e

scripts/create-tags.sh

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 
 set -e
 

scripts/generate-tar-doc

@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/usr/bin/env bash
 
 # generate-tar-doc $DEVEL_HPP_DIR/install/share/doc http://gepettoweb.laas.fr/hpp
 # then copy /tmp/hpp.tar.gz on server and inflate archive.

scripts/install-tar-on-remote

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 
 VERSION="v4.6.0"
 HOST=gepettoweb.laas.fr

scripts/packageDep

@@ -1,44 +1,53 @@
-#!/usr/bin/python
+#!/usr/bin/env python3
 
 from __future__ import print_function, with_statement
 
-from optparse import OptionParser
+import os
 import re
-import sys, os
+import sys
+from optparse import OptionParser
 
 try:  # python2
     from commands import getstatusoutput
 except:  # python3
     from subprocess import getstatusoutput
 
 pkgConfigPath = os.environ.get("PKG_CONFIG_PATH")
-pathList = re.split(':', pkgConfigPath)
+pathList = re.split(":", pkgConfigPath)
 
 usage = "usage: %prog [options] package-1 ... package-n"
 parser = OptionParser(version="packageDep 0.1", usage=usage)
 
-parser.add_option("-o", "--output", dest="filename",
-                  help="write graph to FILE", metavar="FILE")
-parser.add_option("-q", "--quiet", action="store_true", dest="quiet", default=False,
-                  help="Do not print messages to stdout")
+parser.add_option(
+    "-o", "--output", dest="filename", help="write graph to FILE", metavar="FILE"
+)
+parser.add_option(
+    "-q",
+    "--quiet",
+    action="store_true",
+    dest="quiet",
+    default=False,
+    help="Do not print messages to stdout",
+)
 
 (options, args) = parser.parse_args()
 
 if options.filename is None:
     parser.print_help()
 
 if options.quiet:
-    sys.stdout = open(os.devnull, 'w')
+    sys.stdout = open(os.devnull, "w")
 
 packageList = args
 filename = options.filename
 
-requireExpr = re.compile('(\s*Requires:\s+)(.+)')
-pkgStdName = re.compile('\w+[\w\d\._-]*');
+requireExpr = re.compile("(\s*Requires:\s+)(.+)")
+pkgStdName = re.compile("\w+[\w\d\._-]*")
 
 packageSet = set()
 dependencySet = set()
 
+
 def extractDep(inPkg):
 
     packageSet.add(inPkg)
@@ -48,22 +57,22 @@ def extractDep(inPkg):
 
     for path in pathList:
         found = False
-        filePc = path+"/"+inPkg+".pc"
+        filePc = path + "/" + inPkg + ".pc"
 
         try:
             with open(filePc) as f:
                 for line in f:
                     matchLine = requireExpr.match(line)
                     if matchLine:
-                        requireList = re.split(':', matchLine.group(0))
+                        requireList = re.split(":", matchLine.group(0))
                         depLine = requireList[1]
-                        depFullList = re.split(',', depLine)
+                        depFullList = re.split(",", depLine)
                         for depFull in depFullList:
                             dep = pkgStdName.search(depFull)
                             if dep:
                                 depName = dep.group(0)
                                 depSet.add(depName)
-                                depString = "\"" + depName + "\" -> \"" + inPkg + "\""
+                                depString = '"' + depName + '" -> "' + inPkg + '"'
                                 dependencySet.add(depString)
 
         except IOError:
@@ -76,22 +85,22 @@ def extractDep(inPkg):
 
 def getDocLink(package):
     cmd = "pkg-config --variable=doxygendocdir " + package
-    status,output = getstatusoutput(cmd)
+    status, output = getstatusoutput(cmd)
 
     if status != 0:
-        return ''
+        return ""
 
-    if output != '':
+    if output != "":
         return output
 
     cmd = "pkg-config --variable=docdir " + package
-    status,output = getstatusoutput(cmd)
+    status, output = getstatusoutput(cmd)
 
     if status != 0:
-        return ''
+        return ""
 
-    if output != '':
-        return output + '/html'
+    if output != "":
+        return output + "/html"
     return output
 
 
@@ -103,24 +112,24 @@ for pkg in packageList:
 #
 
 try:
-    f = open(filename, 'w')
+    f = open(filename, "w")
 except IOError:
     print("cannot open ", filename)
 
 f.write("digraph CD  {\n")
-f.write("\tsize = \"12,15\"\n")
+f.write('\tsize = "12,15"\n')
 f.write("\trankdir = BT\n")
 f.write("\tcompound=true\n")
 f.write("\n")
 
 for package in packageSet:
     docLink = getDocLink(package)
-    if docLink != '':
-        docLink = docLink + '/index.html'
-    f.write("\t\"" + package + "\" [shape = box, href = \"" + docLink + "\"]\n")
+    if docLink != "":
+        docLink = docLink + "/index.html"
+    f.write('\t"' + package + '" [shape = box, href = "' + docLink + '"]\n')
 
 
 for dependency in dependencySet:
-    f.write("\t" + dependency + "\n");
+    f.write("\t" + dependency + "\n")
 
 f.write("}\n")