Merge pull request #3539 from boegel/pythonpackage_fix_builcmd_python…

…_template inject custom '`%(python)s`' template value before getting value of '`buildcmd`' custom easyconfig parameter in `PythonPackage` easyblock
easybuilders · Dec 19, 2024 · 90c14fe · 90c14fe
2 parents 47fe3fa + 51ce299
commit 90c14fe
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/easybuild/easyblocks/generic/pythonpackage.py b/easybuild/easyblocks/generic/pythonpackage.py
@@ -818,7 +818,11 @@ def configure_step(self):
 
     def build_step(self):
         """Build Python package using setup.py"""
-        build_cmd = self.cfg.get_ref('buildcmd')
+
+        # inject extra '%(python)s' template value before getting value of 'buildcmd' custom easyconfig parameter
+        self.cfg.template_values['python'] = self.python_cmd
+        build_cmd = self.cfg['buildcmd']
+
         if self.use_setup_py:
 
             if get_software_root('CMake'):
@@ -829,10 +833,9 @@ def build_step(self):
 
             if not build_cmd:
                 build_cmd = 'build'  # Default value for setup.py
-            build_cmd = '%(python)s setup.py ' + build_cmd
+            build_cmd = f"{self.python_cmd} setup.py {build_cmd}"
 
         if build_cmd:
-            build_cmd = build_cmd % {'python': self.python_cmd}
             cmd = ' '.join([self.cfg['prebuildopts'], build_cmd, self.cfg['buildopts']])
             res = run_shell_cmd(cmd)