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

Reconfigure the RPM URLs if checksum changed. #65

Merged
merged 1 commit into from
Oct 9, 2023
Merged

Conversation

pomodorox
Copy link
Contributor

Currently if for some reason the RPM entry changed, bazeldnf won't reconfigure the URLs if they are already set. While this behavior aims to cover cases like mirror urls getting added out of band, there is a case where the URLs are messed up and do match the checksum. Therefore with this change it reconfigures the RPM URLs when the checksum changed.

Currently if for some reason the RPM entry changed,
bazeldnf won't reconfigure the URLs if they are already
set. While this behavior aims to cover cases like mirror
urls getting added out of band, there is a case where the
URLs are messed up and do match the checksum. Therefore
with this change it reconfigures the RPM URLs when the
checksum changed.

Signed-off-by: Yufeng Duan <55268016+pomodorox@users.noreply.github.com>
Signed-off-by: Yufeng Duan <55268016+didovesei@users.noreply.github.com>
Copy link
Owner

@rmohr rmohr left a comment

Choose a reason for hiding this comment

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

LGTM.

Thanks!

@rmohr rmohr merged commit bf38400 into rmohr:main Oct 9, 2023
1 check passed
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.

None yet

2 participants