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 support/tests for Conan 2 and Python 3.12 #39

Merged
merged 10 commits into from
May 3, 2024
Merged

Add support/tests for Conan 2 and Python 3.12 #39

merged 10 commits into from
May 3, 2024

Conversation

dean0x7d
Copy link
Member

Both Conan 1 and 2 are tested in the CI matrix. We'll support both for a while. Python 3.12 is in, Python 3.10 and older are out. This was a good opportunity to drop the old versions, mainly to avoid having to support both openssl v1 and v3 dependencies.

The PR diff line count is inflated because of machine-generated env file changes. The actual diff is ~100 lines. Review per commit for the best experience. Note that the 4 commits are simply repo maintenance.

dean0x7d and others added 4 commits April 27, 2024 20:09
So that we don't get surprise file reformatting.
The `#egg=` format is deprecated and will be removed in pip 25.
It also turns out that the latest `pylake` git clone goes over the
Windows file path limit when checked out as part of `pip install`.
`git config --system core.longpaths true` resolves it.
dean0x7d and others added 6 commits May 3, 2024 14:44
Some projects may end up calling `find_package(embedded_python[-core])`
multiple times in their CMake code. By extension `find_package(Python)`
gets called many times and it's pretty slow because it needs to run
some Python to get all the details about the interpreter. We can make
sure that this only runs once. Conan ensures that only one version of
`embedded_python-core` is always included in the project so the first
info is save to cache and will not change.
It stopped doing anything in v1.8.0. Time to remove it.
v1.6.0 introduced `embedded_python-core` and left the `version` option in
the non-core package for backward compatibility. At this point, it's too
confusing to have both, so let's drop it.
@dean0x7d dean0x7d enabled auto-merge (rebase) May 3, 2024 12:51
@dean0x7d dean0x7d merged commit 44dd7d1 into main May 3, 2024
15 checks passed
@dean0x7d dean0x7d deleted the conan2 branch May 3, 2024 12:53
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