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

Disable pyenv cache for linux because maturin is bugged as hell #681

Closed
wants to merge 1 commit into from

Conversation

evandrocoan
Copy link
Contributor

@evandrocoan evandrocoan commented Jul 9, 2020

  1. df319c7 - is:review should include relearning cards
  2. c50fa2c - invalidate cache

https://github.com/ankitects/anki/runs/852260154#step:25:768

FTL_TEMPLATE_DIRS="../qt/ftl" FTL_LOCALE_DIRS="../qt/ftl/repo/desktop" 
CARGO_TARGET_DIR="/home/runner/work/anki/anki/target" \
	maturin develop 
🔗 Found pyo3 bindings
🐍 Found CPython 3.7m at python
💥 maturin failed
  Caused by: Expected `python` to be a python interpreter inside a virtualenv ಠ_ಠ

@dae
Copy link
Member

dae commented Jul 11, 2020

Busting the cache again seems to have fixed it for now - maybe GitHub are just moving files around?

@evandrocoan
Copy link
Contributor Author

evandrocoan commented Jul 11, 2020

It could be. Although looking at the cache restoring it does not show the cache was not fully restored (as it may happen).

But, without closely debugging why maturing is thinking he is not inside a virtual env, is hard to tell.

I know that was quite cumbersome to have this maturing detection working on Windows:

  1. Expected python to be a python interpreter inside a virtualenv PyO3/maturin#283 - Expected python to be a python interpreter inside a virtualenv

They do not seem to do a robust detection and keep relying on fragile strategy.

@dae
Copy link
Member

dae commented Jul 13, 2020

Let's keep this open for a while and see if the problem pops up again - I'd rather not lose the cache if we can avoid it.

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 this pull request may close these issues.

2 participants