-
Notifications
You must be signed in to change notification settings - Fork 359
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
Opam admin cache doesn't create a symlink for sha256 to sha512 #6064
Comments
Has issue 2 been fixed in opam 2.1.5 by #5538 ? |
I'll test building an opam 2.1.6 rpm to be used in the build |
I've made a github runner to package a repo tarball, it uses ubuntu's opam package: 2.1.2. The resulting tarball contains the file in the sha256 folder, but not a file in the sha512 folder: |
I've retried building the package with 2.1.6 and issue 2 still happens. Building opam 2.2.0 requires some changes, so it will take some time to test whether it's affected |
Actually I'm getting the same problem with opam 2.2.0 while using https://github.com/kit-ty-kate/opam-health-check-ng, which means a large number of packages fail and I'm unable to check them. I'm looking into it. |
Do you have a previously populated cache or it is an new one? |
I can reproduce this locally starting from an empty cache. It's strange this only seems to happen for a single package in the repository. |
we managed to found the issue and fix it in #6068. We still need to figure out if the fix doesn't introduce any performance issues but at least it's being actively worked on. |
I've tested the tarballing with the new 2.3.0 alpha, and can confirm that it builds without issues now, thanks! |
Thanks for the update! |
I've run into a couple issues when building xapi's opam repository as an RPM in an environment without internet access:
I'm updating the opam version for issue 2. to see if this behaviour has changed. In any case I've been able to work around issue 1 by manually creating the symlink in the sha256 folder to the sha512 file. Another workaround is removing the sha512 hash from the checksum list: it makes opam create the file created correctly in the sha256 folder.
I don't quite understand why this is the only package in the whole repository with this behaviour, but it looks like a bug.
The text was updated successfully, but these errors were encountered: