enable download_dep_fail, use_pip, sanity_pip_check by default in PythonPackage easyblock

easybuild/easyblocks/generic/pythonbundle.py

@@ -69,9 +69,6 @@ def __init__(self, *args, **kwargs):
                 if key not in self.cfg['exts_default_options']:
                     self.cfg['exts_default_options'][key] = self.cfg[key]
 
-            self.cfg['exts_default_options']['download_dep_fail'] = True
-            self.log.info("Detection of downloaded extension dependencies is enabled")
-
             self.log.info("exts_default_options: %s", self.cfg['exts_default_options'])
 
         self.pylibdir = None

easybuild/easyblocks/generic/pythonpackage.py

@@ -330,7 +330,7 @@ def extra_options(extra_vars=None):
                                "Otherwise it will be used as-is. A value of None then skips the build step. "
                                "The template %(python)s will be replace by the currently used Python binary.", CUSTOM],
             'check_ldshared': [None, 'Check Python value of $LDSHARED, correct if needed to "$CC -shared"', CUSTOM],
-            'download_dep_fail': [None, "Fail if downloaded dependencies are detected", CUSTOM],
+            'download_dep_fail': [True, "Fail if downloaded dependencies are detected", CUSTOM],
             'install_src': [None, "Source path to pass to the install command (e.g. a whl file)."
                                   "Defaults to '.' for unpacked sources or the first source file specified", CUSTOM],
             'install_target': ['install', "Option to pass to setup.py", CUSTOM],
@@ -339,8 +339,8 @@ def extra_options(extra_vars=None):
                                    "the pip version check. Enabled by default when pip_ignore_installed=True", CUSTOM],
             'req_py_majver': [None, "Required major Python version (only relevant when using system Python)", CUSTOM],
             'req_py_minver': [None, "Required minor Python version (only relevant when using system Python)", CUSTOM],
-            'sanity_pip_check': [False, "Run 'python -m pip check' to ensure all required Python packages are "
-                                        "installed and check for any package with an invalid (0.0.0) version.", CUSTOM],
+            'sanity_pip_check': [True, "Run 'python -m pip check' to ensure all required Python packages are "
+                                       "installed and check for any package with an invalid (0.0.0) version.", CUSTOM],
             'runtest': [True, "Run unit tests.", CUSTOM],  # overrides default
             'testinstall': [False, "Install into temporary directory prior to running the tests.", CUSTOM],
             'unpack_sources': [None, "Unpack sources prior to build/install. Defaults to 'True' except for whl files",
@@ -349,7 +349,7 @@ def extra_options(extra_vars=None):
             # version. Those would fail the (extended) sanity_pip_check. So as a last resort they can be added here
             # and will be excluded from that check. Note that the display name is required, i.e. from `pip list`.
             'unversioned_packages': [[], "List of packages that don't have a version at all, i.e. show 0.0.0", CUSTOM],
-            'use_pip': [None, "Install using '%s'" % PIP_INSTALL_CMD, CUSTOM],
+            'use_pip': [True, "Install using '%s'" % PIP_INSTALL_CMD, CUSTOM],
             'use_pip_editable': [False, "Install using 'pip install --editable'", CUSTOM],
             # see https://packaging.python.org/tutorials/installing-packages/#installing-setuptools-extras
             'use_pip_extras': [None, "String with comma-separated list of 'extras' to install via pip", CUSTOM],
@@ -417,7 +417,7 @@ def determine_install_command(self):
         """
         Determine install command to use.
         """
-        if self.cfg.get('use_pip', False) or self.cfg.get('use_pip_editable', False):
+        if self.cfg.get('use_pip', True) or self.cfg.get('use_pip_editable', False):
             self.install_cmd = PIP_INSTALL_CMD
 
             if build_option('debug'):
@@ -439,7 +439,7 @@ def determine_install_command(self):
                 self.cfg.update('installopts', '--egg')
 
             pip_no_index = self.cfg.get('pip_no_index', None)
-            if pip_no_index or (pip_no_index is None and self.cfg.get('download_dep_fail')):
+            if pip_no_index or (pip_no_index is None and self.cfg.get('download_dep_fail', True)):
                 self.cfg.update('installopts', '--no-index')
 
             # avoid that pip (ab)uses $HOME/.cache/pip
@@ -958,7 +958,7 @@ def sanity_check_step(self, *args, **kwargs):
         # see also https://github.com/easybuilders/easybuild-easyblocks/issues/1877
         env.setvar('PYTHONNOUSERSITE', '1', verbose=False)
 
-        if self.cfg.get('download_dep_fail', False):
+        if self.cfg.get('download_dep_fail', True):
             self.log.info("Detection of downloaded depenencies enabled, checking output of installation command...")
             patterns = [
                 'Downloading .*/packages/.*',  # setuptools
@@ -1002,7 +1002,7 @@ def sanity_check_step(self, *args, **kwargs):
                 exts_filter = (orig_exts_filter[0].replace('python', self.python_cmd), orig_exts_filter[1])
                 kwargs.update({'exts_filter': exts_filter})
 
-        if self.cfg.get('sanity_pip_check', False):
+        if self.cfg.get('sanity_pip_check', True):
             pip_version = det_pip_version(python_cmd=python_cmd)
 
             if pip_version:

easybuild/easyblocks/j/jaxlib.py

@@ -51,7 +51,6 @@ def extra_options():
         """Custom easyconfig parameters specific to jaxlib."""
         extra_vars = PythonPackage.extra_options()
 
-        extra_vars['use_pip'][0] = True
         # Run custom build script and install the generated whl file
         extra_vars['buildcmd'][0] = '%(python)s build/build.py'
         extra_vars['install_src'][0] = 'dist/*.whl'

easybuild/easyblocks/n/numexpr.py

@@ -37,16 +37,6 @@
 class EB_numexpr(PythonPackage):
     """Support for building/installing numexpr."""
 
-    @staticmethod
-    def extra_options():
-        """Override some custom easyconfig parameters specifically for numexpr."""
-        extra_vars = PythonPackage.extra_options()
-
-        extra_vars['download_dep_fail'][0] = True
-        extra_vars['use_pip'][0] = True
-
-        return extra_vars
-
     def __init__(self, *args, **kwargs):
         """Initialisation of custom class variables for numexpr."""
         super(EB_numexpr, self).__init__(*args, **kwargs)

easybuild/easyblocks/p/pybind11.py

@@ -45,15 +45,6 @@ class EB_pybind11(CMakePythonPackage):
     Python packages using `import pybind11`
     Hence we need to install PyBind11 twice: Once with CMake and once with pip
     """
-    @staticmethod
-    def extra_options(extra_vars=None):
-        """Easyconfig parameters specific to PyBind11: Set defaults"""
-        extra_vars = PythonPackage.extra_options(extra_vars=extra_vars)
-        extra_vars = CMakeMake.extra_options(extra_vars=extra_vars)
-        extra_vars['use_pip'][0] = True
-        extra_vars['sanity_pip_check'][0] = True
-        extra_vars['download_dep_fail'][0] = True
-        return extra_vars
 
     def configure_step(self):
         """Avoid that a system Python is picked up when a Python module is loaded"""

easybuild/easyblocks/p/pytorch.py

@@ -54,8 +54,6 @@ def extra_options():
             'excluded_tests': [{}, "Mapping of architecture strings to list of tests to be excluded", CUSTOM],
             'max_failed_tests': [0, "Maximum number of failing tests", CUSTOM],
         })
-        extra_vars['download_dep_fail'][0] = True
-        extra_vars['sanity_pip_check'][0] = True
 
         return extra_vars
 

easybuild/easyblocks/s/sympy.py

@@ -38,15 +38,6 @@
 class EB_sympy(PythonPackage):
     """Custom easyblock for installing the sympy Python package."""
 
-    @staticmethod
-    def extra_options(extra_vars=None):
-        """Customize default value for easyconfig parameters for sympy"""
-        extra_vars = PythonPackage.extra_options(extra_vars=extra_vars)
-        extra_vars['use_pip'][0] = True
-        extra_vars['sanity_pip_check'][0] = True
-        extra_vars['download_dep_fail'][0] = True
-        return extra_vars
-
     def test_step(self):
         """Custom test step for sympy"""
 

easybuild/easyblocks/t/tensorflow.py

@@ -251,8 +251,6 @@ def __init__(self, *args, **kwargs):
         with self.cfg.disable_templating():
             self.cfg['exts_defaultclass'] = 'PythonPackage'
 
-            self.cfg['exts_default_options']['download_dep_fail'] = True
-            self.cfg['exts_default_options']['use_pip'] = True
             self.cfg['exts_filter'] = EXTS_FILTER_PYTHON_PACKAGES
 
         self.system_libs_info = None

easybuild/easyblocks/t/tensorrt.py

@@ -67,11 +67,6 @@ def __init__(self, *args, **kwargs):
         # Setup for the extensions step
         self.cfg['exts_defaultclass'] = 'PythonPackage'
 
-        self.cfg['exts_default_options'] = {
-            'download_dep_fail': True,
-            'use_pip': True,
-        }
-
     def configure_step(self):
         """Custom configuration procedure for TensorRT."""
         pass

easybuild/easyblocks/t/torchvision.py

@@ -38,15 +38,6 @@
 class EB_torchvision(PythonPackage):
     """Support for building/installing TorchVison."""
 
-    @staticmethod
-    def extra_options():
-        """Change some defaults for easyconfig parameters."""
-        extra_vars = PythonPackage.extra_options()
-        extra_vars['use_pip'][0] = True
-        extra_vars['download_dep_fail'][0] = True
-        extra_vars['sanity_pip_check'][0] = True
-        return extra_vars
-
     def __init__(self, *args, **kwargs):
         """Initialize torchvision easyblock."""
         super(EB_torchvision, self).__init__(*args, **kwargs)