-
-
Notifications
You must be signed in to change notification settings - Fork 650
Added backport fix for reposync #3227
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks for using the issue template. That makes this a lot easier. I have some comments that I would love if we address them before we merge this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@c4t3l Could you sign the commits please? (And if possible squash the fix and revert fix into the main commit) |
@SchoolGuy, sure thing. I will do that once I get back home. |
fb6488a
to
7d5e0bb
Compare
Please let me know if there is anything else required. |
@c4t3l Nope all was fine now. This PR just slipped under my radar. |
Linked Items
Fixes #2865 for v3.2.2 which is the current supported stable branch in epel8.
Description
This PR backports some of the changes presented in #2942. Namely, the librepo "Cannot create output directory" error and the missing noarch package downloads referenced in this comment
Behaviour changes
Old: Reposyncs fail on consecutive runs and noarch packages are ignored in repos.
New: Reposyncs pass on consecutive runs and noarch packages are downloaded.
Category
This is related to a:
Tests