Skip to content

Commit

Permalink
[bugfix] Use correct temporary directory for python installation
Browse files Browse the repository at this point in the history
There are tow problems with the way the `pythonpath` instance field is
defined:

- it uses the log directory to store a build artifact, and
- the functions that use it do not accept a full path.

We cannot use the `build_path()` function instead of `log_path()` as the
function provides a full path as well. Thus we opted to hard-code a
relative path with respect to the build directory.

Issue: easybuilders/easybuild-easyconfigs#21078
  • Loading branch information
gkaf89 committed Jul 30, 2024
1 parent 0cb2cc9 commit 8289bd5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions easybuild/easyblocks/p/python.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
from easybuild.framework.easyconfig import CUSTOM
from easybuild.framework.easyconfig.templates import TEMPLATE_CONSTANTS
from easybuild.tools.build_log import EasyBuildError, print_warning
from easybuild.tools.config import build_option, log_path
from easybuild.tools.config import build_option
from easybuild.tools.modules import get_software_libdir, get_software_root, get_software_version
from easybuild.tools.filetools import apply_regex_substitutions, change_dir, mkdir
from easybuild.tools.filetools import read_file, remove_dir, symlink, write_file
Expand Down Expand Up @@ -145,7 +145,7 @@ def __init__(self, *args, **kwargs):
self.pyshortver = '.'.join(self.version.split('.')[:2])

# Used for EBPYTHONPREFIXES handler script
self.pythonpath = os.path.join(log_path(), 'python')
self.pythonpath = os.path.join('easybuild', 'python')

ext_defaults = {
# Use PYPI_SOURCE as the default for source_urls of extensions.
Expand Down

0 comments on commit 8289bd5

Please sign in to comment.