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

[tycho-4.0.x] Remove no longer valid checksum properties | Add Integration Test for issue #2875 #3927

Merged
merged 2 commits into from
Jun 7, 2024

Conversation

eclipse-tycho-bot
Copy link

Backport

This will backport the following commits from main to tycho-4.0.x:

Questions ?

Please refer to the Backport tool documentation

laeubi and others added 2 commits June 7, 2024 17:19
Currently Tycho/P2 only *adds* checksums but as some are now disabled
for publish this can lead to the case where an existing checksum is kept
as-is.

This now performs and additional check if there are old checksums and
remove them when recreate the repository.

Fix #2875

(cherry picked from commit ae2aae1)
This integration test shows that `download.md5` and
`download.checksum.md5` are present after running the
`fix-artifacts-metadata` goal when they shouldn't.

If these are not removed and the content of the artifact changed then,
after the recalculation done by this goal, new p2 libs will see an
outdated value for these old checksum algorithms and will fail the
installation due to a checksum mismatch.

(cherry picked from commit b88a16e)
Copy link

github-actions bot commented Jun 7, 2024

Test Results

  594 files  +3    594 suites  +3   3h 55m 26s ⏱️ + 9m 21s
  415 tests +1    409 ✅ +2   6 💤 ±0  0 ❌  - 1 
1 245 runs  +3  1 226 ✅ +4  19 💤 ±0  0 ❌  - 1 

Results for commit 1faa0da. ± Comparison against base commit 3b7a9f4.

@eclipse-tycho-bot eclipse-tycho-bot merged commit 0ecd63b into tycho-4.0.x Jun 7, 2024
11 checks passed
@laeubi laeubi deleted the backport/tycho-4.0.x/pr-3902_pr-3902 branch June 16, 2024 15:05
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.

3 participants