Merge pull request #3998 from boegel/eb_import_check

change 'eb' command to import easybuild.framework to check if EasyBuild framework is available

Author: bartoldeman

.github/workflows/eb_command.yml

@@ -73,15 +73,15 @@ jobs:
           pymajver=$(python -c 'import sys; print(sys.version_info[0])')
           pymajminver=$(python -c 'import sys; print(".".join(str(x) for x in sys.version_info[:2]))')
           # check patterns in verbose output
-          for pattern in "^>> Considering .python.\.\.\." "^>> .python. version: ${pymajminver}\.[0-9]\+, which matches Python ${pymajver} version requirement" "^>> 'python' is able to import 'easybuild', so retaining it" "^>> Selected Python command: python \(.*/bin/python\)" "^This is EasyBuild 4\.[0-9.]\+"; do
+          for pattern in "^>> Considering .python.\.\.\." "^>> .python. version: ${pymajminver}\.[0-9]\+, which matches Python ${pymajver} version requirement" "^>> 'python' is able to import 'easybuild.framework', so retaining it" "^>> Selected Python command: python \(.*/bin/python\)" "^This is EasyBuild 4\.[0-9.]\+"; do
               echo "Looking for pattern \"${pattern}\" in eb_version.out..."
               grep "$pattern" eb_version.out
           done
           # also check when specifying Python command via $EB_PYTHON
           for eb_python in "python${pymajver}" "python${pymajminver}"; do
               export EB_PYTHON="${eb_python}"
               eb --version | tee eb_version.out 2>&1
-              for pattern in "^>> Considering .${eb_python}.\.\.\." "^>> .${eb_python}. version: ${pymajminver}\.[0-9]\+, which matches Python ${pymajver} version requirement" "^>> '${eb_python}' is able to import 'easybuild', so retaining it" "^>> Selected Python command: ${eb_python} \(.*/bin/${eb_python}\)" "^This is EasyBuild 4\.[0-9.]\+"; do
+              for pattern in "^>> Considering .${eb_python}.\.\.\." "^>> .${eb_python}. version: ${pymajminver}\.[0-9]\+, which matches Python ${pymajver} version requirement" "^>> '${eb_python}' is able to import 'easybuild.framework', so retaining it" "^>> Selected Python command: ${eb_python} \(.*/bin/${eb_python}\)" "^This is EasyBuild 4\.[0-9.]\+"; do
                   echo "Looking for pattern \"${pattern}\" in eb_version.out..."
                   grep "$pattern" eb_version.out
               done

eb

@@ -46,6 +46,10 @@ REQ_MIN_PY3VER=5
 
 EASYBUILD_MAIN='easybuild.main'
 
+# easybuild module to import to check whether EasyBuild framework is available;
+# don't use easybuild.main here, since that's a very expensive module to import (it makes the 'eb' command slow)
+EASYBUILD_IMPORT_TEST='easybuild.framework'
+
 function verbose() {
     if [ ! -z ${EB_VERBOSE} ]; then echo ">> $1"; fi
 }
@@ -79,13 +83,13 @@ for python_cmd in ${EB_PYTHON} ${EB_INSTALLPYTHON} 'python' 'python3' 'python2';
         fi
 
         if [ ! -z $PYTHON ]; then
-            # check whether easybuild is available for selected python command
-            $PYTHON -c "import easybuild" 2> /dev/null
+            # check whether EasyBuild framework is available for selected python command
+            $PYTHON -c "import $EASYBUILD_IMPORT_TEST" 2> /dev/null
             if [ $? -eq 0 ]; then
-                verbose "'$python_cmd' is able to import 'easybuild', so retaining it"
+                verbose "'$python_cmd' is able to import '$EASYBUILD_IMPORT_TEST', so retaining it"
             else
-                # if easybuild is not available, don't use this python command, keep searching...
-                verbose "'$python_cmd' is NOT able to import 'easybuild' so NOT retaining it"
+                # if EasyBuild framework is not available, don't use this python command, keep searching...
+                verbose "'$python_cmd' is NOT able to import '$EASYBUILD_IMPORT_TEST' so NOT retaining it"
                 unset PYTHON
             fi
         fi