Skip to content

Mamba split #36183

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

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Mamba split #36183

wants to merge 4 commits into from

Conversation

dannf
Copy link
Member

@dannf dannf commented Dec 9, 2024

We've been using upstream date-based tags for our versions, but that's apparently not what those tags are for. Currently that has CI wanting to move us backwards. We should use the semver tags instead.

Since libmamba, libmambapy and micromamba are all tagged separately, it's not clear which tag prefix we should be using. My thought was that we should treat them as separate sources even though they are from the same repo, and even though their releases always seem to point to the same hash.

If we did that, we'd build libmamba and libmamba-dev from libmamba.yaml, and then have micromamba.yaml, py3-libmambapy.yaml, and mamba-package.yaml files that build those binaries against the existing libmamba-dev.

But there are some problems here.

  1. I don't think we have a defined way to migrate from 2024.10.02 to 2.0.4 version strings, since they sort lesser. We discussed options here, and withdrawing the date-versioned apks seems like the way to go.
  2. mamba-package doesn't have a tag stream. I'm not sure which tag stream we should be using to build it.
  3. micromamba is supposed to be statically linked, according to https://github.com/mamba-org/mamba?tab=readme-ov-file#micromamba. Ours is not, and looks like it will require some CMake work to do so.

@dannf dannf force-pushed the mamba-split branch 2 times, most recently from e81290a to 74b8acb Compare December 10, 2024 18:47
@dannf dannf force-pushed the mamba-split branch 2 times, most recently from f552075 to 41cedf2 Compare December 12, 2024 20:52
@dannf dannf added the approved-to-run A repo member has approved this external contribution label Dec 12, 2024
…2.0.5

We were tracking the date-based tags, but these tags are not
always forward-moving. Switch to the semver tags instead.

Now, there isn't a single stream of semver tags in this project, rather
there are tags per binary - e.g. libmamba-2.0.5, libmambapy-2.0.5 and
micromamba-2.0.5. These tags appear to always point to the same commit
hash for each release, but I'm not sure we can assume that will always
be the case. Let's just treat them as separate projects.

Signed-off-by: dann frazier <[email protected]>
Signed-off-by: dann frazier <[email protected]>
@philroche
Copy link
Member

Latest package update PR @ #36642

@philroche
Copy link
Member

Also see slack thread (internal to chainguard) RE discussion static vs dynamic https://chainguard-dev.slack.com/archives/C0627BW5GLF/p1730889041303439

@debasishbsws
Copy link
Member

There is a Latest update PR #38633

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-to-run A repo member has approved this external contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants