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

Add option to print path to K's Python dependency #92

Closed
Baltoli opened this issue Nov 20, 2023 · 1 comment · Fixed by runtimeverification/k#3823
Closed

Add option to print path to K's Python dependency #92

Baltoli opened this issue Nov 20, 2023 · 1 comment · Fixed by runtimeverification/k#3823

Comments

@Baltoli
Copy link
Contributor

Baltoli commented Nov 20, 2023

When using kup-installed K to run Poetry-based projects, we need to point the Poetry virtual environment at a version of Python that's been installed by Nix; pyenv and the system Python don't have the correct headers / include paths to work with the Nix C++ compiler that builds the Python binding modules.

We should implement a kup command that just prints out the path to the Python derivation that K depends on. For example:

$ kup which-k-python
/nix/store/bc45k1n0pkrdkr3xa6w84w1xhkl1kkyp-python3-3.10.12/bin/python3

Then, projects that want to use kup for a Python-based semantics can run something like:

$ poetry env use --no-cache $(kup which-k-python)

to make sure that their project uses the right version of Python.

Related:

@Baltoli Baltoli linked a pull request Nov 20, 2023 that will close this issue
rv-jenkins added a commit to runtimeverification/k that referenced this issue Nov 20, 2023
Needed to implement a fix for
runtimeverification/kup#92

---------

Co-authored-by: Bruce Collie <[email protected]>
Co-authored-by: rv-jenkins <[email protected]>
@Baltoli
Copy link
Contributor Author

Baltoli commented Nov 29, 2023

Fixed by: runtimeverification/k#3823

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

Successfully merging a pull request may close this issue.

1 participant