Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Python executable in installed files should be supplied by the caller #758

Open
yurivict opened this issue Aug 26, 2022 · 2 comments
Open
Assignees

Comments

@yurivict
Copy link

For example in apps/upf/upf_to_json python should be replaced with cmake's variable PYTHON3_COMMAND or similar.

@simonpintarelli
Copy link
Collaborator

At the moment SIRIUS only depends explicitly on python when the python module is built (cf https://github.com/spack/spack/blob/develop/var/spack/repos/builtin/packages/sirius/package.py). In most cases the python module isn't required. Spack has become the preferred way for installing SIRIUS. On HPC systems this would then often require to build python from source, if we would make it a hard dependency.
The typical user will run the conversion script locally (input file creation) and later run the computation on a cluster. The typical end user will most likely not compile sirius locally, but run the conversion script directly from the source repo.

Is there no python exectuable called python on freebsd?

We have the conversion script in a separate repository. aiida needs it to be installable via pip. The release on pypi is currently a python module only https://pypi.org/project/upf-to-json, without an executable.
I've added a python click interface https://github.com/simonpintarelli/upf_to_json/tree/click, I can make a release of this one on pypi if that would help. Then we could also add an option to cmake, to turn off the installation of the upf_to_json script (which seems to be broken on freebsd due to name of the python exectuable).

@yurivict
Copy link
Author

Is there no python exectuable called python on freebsd?

No. Multiple Python versions can be installed concurrently, each having its unique name/path.
FreeBSD port has some Python version as default and it has the PYTHON_CMD variable containing the path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants