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

Fixed repodata creation for multiple rpms #13

Merged
merged 1 commit into from
May 9, 2017
Merged

Fixed repodata creation for multiple rpms #13

merged 1 commit into from
May 9, 2017

Conversation

berndwarmuth
Copy link

@berndwarmuth berndwarmuth commented May 9, 2017

The creation of repodata files was broken when more than one rpm were added to the yum repository. More specifically, the resulting metadata *.xml files have been overwritten in each loop iteration, resulting in having only one package defined in the repodata files. Thus, the yum-repository goal was not usable when intending to add more than one rpm to the repository.

This PR fixes #12 by making sure that the context variable to add rpm packages gets reused in each loop iteration instead of recreated.

@ctron ctron merged commit 40bfaa7 into ctron:master May 9, 2017
@ctron
Copy link
Owner

ctron commented May 9, 2017

Big thanks! Let me know when I should make a new release of the plugin.

@berndwarmuth
Copy link
Author

Thanks for responding so fast :) It would be really cool to release a new version right away.

@ctron
Copy link
Owner

ctron commented May 9, 2017

The release is done. It may take a while into it is synced with Maven central.

Many thanks for the PR!

@ctron
Copy link
Owner

ctron commented May 11, 2017

@triggetry Do you have a Twitter handle for mentioning you?

@berndwarmuth
Copy link
Author

@ctron
Copy link
Owner

ctron commented May 12, 2017

That was easy 😁

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.

yum-repository goal creates wrong repodata for multiple RPMs
3 participants