Skip to content
New issue

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

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

Already on GitHub? # to your account

CI: build using the correct Python version #26

Merged

Conversation

Xmader
Copy link
Member

@Xmader Xmader commented May 29, 2023

https://github.com/Distributive-Network/PythonMonkey/actions/runs/5115473014/jobs/9196841495?pr=24#step:8:18 fails because cmake is not using the correct version of Python to build pythonmonkey.so.

@Xmader Xmader changed the title Chore: build with the correct Python version CI: build with the correct Python version May 29, 2023
echo $(poetry run python --version)
rm -rf ./build/
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got the correct Python version with the cached ./build deleted.
It seems falling back to the cached values in build/CMakeCache.txt.
We shouldn't cache ./build dir anyhow.

Copy link
Member Author

@Xmader Xmader May 29, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it's because the virtualenv installed by poetry doesn't have the Development components as in find_package(Python 3.9...<3.12 COMPONENTS Interpreter Development REQUIRED)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ouhh that's a weird symptom.... Why did this not happen with my branch?

@Xmader Xmader force-pushed the Xmader/chore/build-with-correct-python-version branch from 265293c to 4ff7522 Compare May 29, 2023 23:24
@Xmader Xmader changed the title CI: build with the correct Python version CI: build using the correct Python version May 29, 2023
@Xmader Xmader marked this pull request as ready for review May 30, 2023 00:22
Copy link
Contributor

@Hamada-Distributed Hamada-Distributed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Hamada-Distributed Hamada-Distributed merged commit 0f92131 into main May 30, 2023
@Xmader Xmader deleted the Xmader/chore/build-with-correct-python-version branch May 30, 2023 19:02
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants