Skip to content

Fix md5 checks for onedir conda-standalone #1007

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Fix md5 checks for onedir conda-standalone #1007

wants to merge 2 commits into from

Conversation

jaimergp
Copy link
Contributor

@jaimergp jaimergp commented Jun 24, 2025 •
edited
Loading

Description

Noticed this warning while debugging other PR locally:

PREFIX=/Users/jrodriguez/condabin
WARNING: md5sum mismatch of tar archive
expected: be440733d74e4252c543459b65442660
     got: ecdfe394654062976a7fcc9a70f589eb
Unpacking bootstrapper...
Unpacking payload...

I forgot to hash the extra _internal/ files in #993.

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jun 24, 2025
@github-project-automation github-project-automation bot moved this to 🆕 New in 🔎 Review Jun 24, 2025
@marcoesters
Copy link
Contributor

We could add to the SH installer tests that they check stdout/stderr for warnings like these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
🔎 Review
Status: 🆕 New
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jaimergp @marcoesters @conda-bot