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

omd archive link update #24642

Closed
wants to merge 2 commits into from
Closed

Conversation

haochenx
Copy link
Member

@haochenx haochenx commented Oct 18, 2023

@haochenx haochenx marked this pull request as draft October 18, 2023 14:25
src: "http://pw374.github.io/distrib/omd/omd-0.5.tar.gz"
checksum: "md5=d343deacc270674828260b0fdc55f74c"
src: "https://github.com/ocaml/omd/archive/refs/tags/0.5.tar.gz"
checksum: "sha256=fc462bf7b8f896f924fef610da436d9152168676adca2d606233101eb67e9432"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
checksum: "sha256=fc462bf7b8f896f924fef610da436d9152168676adca2d606233101eb67e9432"
checksum: [
"sha256=fc462bf7b8f896f924fef610da436d9152168676adca2d606233101eb67e9432"
"md5=d343deacc270674828260b0fdc55f74c"
]

Copy link
Member Author

Choose a reason for hiding this comment

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

if the archives are not the same, do we still want the md5 hash?

src: "http://pw374.github.io/distrib/omd/omd-1.2.0.tar.gz"
checksum: "md5=2a1aa5144466a466e33b34de855bcd1c"
src: "https://github.com/ocaml/omd/archive/refs/tags/1.2.0.tar.gz"
checksum: "sha256=180465a930f27be1bf82bb4709f900cad0d7c69820c0c4efa98573242287637e"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
checksum: "sha256=180465a930f27be1bf82bb4709f900cad0d7c69820c0c4efa98573242287637e"
checksum: [
"sha256=180465a930f27be1bf82bb4709f900cad0d7c69820c0c4efa98573242287637e"
"md5=2a1aa5144466a466e33b34de855bcd1c"
]

And the same for all of them. Are the tarballs identical? If now, we should try to find them in the cache and upload them on the opam-source-archives repo

Copy link
Member Author

Choose a reason for hiding this comment

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

I hoped them to be the same, but they are different (at least for the ones that I have checked.) That's why I removed the old checksum.

If now, we should try to find them in the cache and upload them on the opam-source-archives repo

Do you mean if they are not different, we should cache and upload?

I was not able to find archives that have the same hash. I tried https://github.com/ocaml/omd, https://github.com/pw374/omd, and Software Heritage, and they are all different from the recorded hash

Copy link
Member

Choose a reason for hiding this comment

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

The cache that I mean is either of

there is a similar URL for the other hashes.

In this case, in the past we marked the package as unavailable since we could not compare the tarball contents with the original ones. I think we should discuss this specific case at the next maintainers meeting

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for clarification. Let's discuss this further next week.

Copy link
Member Author

Choose a reason for hiding this comment

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

@mseri do you mind to add it as an item to https://github.com/ocaml/opam-repository/wiki/Meeting-notes? I don't have the permission to modify the wiki page

Copy link
Member

Choose a reason for hiding this comment

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

@mseri mseri closed this Nov 8, 2023
@mseri
Copy link
Member

mseri commented Nov 8, 2023

This cannot be built as is, one needs to regenerate the setup.ml.
WE will open a new PR to mark them unavailable

mseri added a commit to mseri/opam-repository that referenced this pull request Nov 8, 2023
See ocaml#24642

Signed-off-by: Marcello Seri <marcello.seri@gmail.com>
@rjbou rjbou mentioned this pull request Nov 13, 2023
@haochenx haochenx deleted the omd-archive-link-update branch November 25, 2023 12:46
nberth pushed a commit to nberth/opam-repository that referenced this pull request Jun 18, 2024
See ocaml#24642

Signed-off-by: Marcello Seri <marcello.seri@gmail.com>
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