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

feat: Only load plugins if builtin python is available. #1012

Merged
merged 5 commits into from
Dec 20, 2024

Conversation

melund
Copy link
Member

@melund melund commented Dec 19, 2024

This pull request introduces several changes to improve Python integration and update dependencies in the project. The most important changes include adding conditional switch for Python availability, defining and using the ANYBODY_BUILTIN_PYTHON_AVAILABLE macro, and updating various dependencies in the pixi.toml file.

Python Integration:

  • Added conditional compilation for AnyGUIPlugin definitions in BM_Plugin.any and ANSUR_Plugin.any based on the ANYBODY_BUILTIN_PYTHON_AVAILABLE macro. [1] [2] [3] [4]
  • Defined the ANYBODY_BUILTIN_PYTHON_AVAILABLE macro and added a message to notify users if Python is not available. [1] [2]

Dependency Updates:

  • Updated Python and various package dependencies in the pixi.toml file to newer versions. [1] [2]

@melund melund requested a review from bkje14 December 19, 2024 11:45
bkje14
bkje14 previously approved these changes Dec 20, 2024
Copy link
Contributor

@bkje14 bkje14 left a comment

Choose a reason for hiding this comment

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

Very nice addition, I ran into this issue before without knowing how to fix it :D

pixi.toml Outdated Show resolved Hide resolved
Co-authored-by: bkje14 <[email protected]>
Signed-off-by: Morten Enemark Lund <[email protected]>
@melund melund merged commit d74b005 into master Dec 20, 2024
26 checks passed
@melund melund deleted the only-plugin-if-python branch December 20, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants