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

Unpin cryptography, python, and add make to environment-dev.yml #3352

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

jaimergp
Copy link
Contributor

I'm using this in some Docker containers as part of my devcontainer config, and they don't include make. Also, cryptography<40 is not needed anymore (see conda/conda-content-trust#57).

I also unpinned python so you can use it on any solvable Python environment without overriding to 3.9.

mitmproxy and doctest are not available on linux-aarch64, so that fails, but the selector syntax only allows OS-level names, not architectures, so I can't omit those.

@@ -24,7 +25,7 @@ dependencies:
# micromamba dependencies
- cli11 >=2.2
# micromamba test dependencies
- python =3.9 # Some not too recent version
- python
Copy link
Member

Choose a reason for hiding this comment

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

I actually don't know the reason of pinning python here... Maybe this is hiding some incompatibilities. Let's see what happens with the CI.

Copy link
Member

Choose a reason for hiding this comment

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

Ok I think unpinning python makes it get fmt 11 everywhere while #3368 only handles a part of it in the codebase.
I suggest restoring python pinning for this PR and handle fmt 11 in another one (I will open an issue for it).
Is it something blocking at the moment or more like an enhancement opening for more possibilities?

That being said, that's only showing one issue of possible other ones when unpinning python, so that would be something to check within the fixing PR.

@Hind-M
Copy link
Member

Hind-M commented Jul 29, 2024

The failures seem to be related to using fmt 11. As #3368 was merged, rebasing on main should fix it.

@Hind-M Hind-M added the release::ci_docs For PRs related to CI or documentation label Jul 29, 2024
jjerphan added a commit to jjerphan/mamba that referenced this pull request Jul 30, 2024
A few other elements were required, as observed in:
mamba-org#3352

Signed-off-by: Julien Jerphanion <[email protected]>
@jjerphan
Copy link
Member

Missing elements for the support of fmt 11 are introduced in #3371.

Hind-M pushed a commit that referenced this pull request Jul 30, 2024
* build: Support fmt 11

A few other elements were required, as observed in:
#3352

Signed-off-by: Julien Jerphanion <[email protected]>

* Unpin python on 3.9

Signed-off-by: Julien Jerphanion <[email protected]>

* ci: use libcxx<18 for macOS

Signed-off-by: Julien Jerphanion <[email protected]>

* Add TODO comment

Co-authored-by: Hind-M <[email protected]>

Signed-off-by: Julien Jerphanion <[email protected]>

---------

Signed-off-by: Julien Jerphanion <[email protected]>
@Hind-M
Copy link
Member

Hind-M commented Jul 30, 2024

Now that #3371 is merged, rebasing should hopefully fix the failures this time :)

@jjerphan
Copy link
Member

jjerphan commented Aug 1, 2024

#3377 should also have resolved the other problem on the CI, @jaimergp. So this PR might be integrated easily now. :)

Copy link
Member

@Hind-M Hind-M left a comment

Choose a reason for hiding this comment

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

Thanks!

@Hind-M Hind-M merged commit d662921 into mamba-org:main Aug 1, 2024
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release::ci_docs For PRs related to CI or documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants