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

Update foreman, foreman-proxy & katello dependencies #10024

Merged

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Nov 24, 2023

foreman and foreman-proxy use rich dependencies, but katello is gem based and gem2rpm doesn't understand that yet.

@ekohl ekohl force-pushed the rpm/develop-rich-deps-pkg-updates branch from 63b1d84 to c4027a2 Compare November 24, 2023 17:20
Comment on lines -34 to -35
BuildArch: noarch
Provides: foreman-plugin-%{plugin_name} = %{version}
Copy link
Member

Choose a reason for hiding this comment

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

removing those two seems wrong?

Copy link
Member Author

Choose a reason for hiding this comment

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

If you look carefully it's moved around, to lines 59/60. I debated moving it in the template to make the diff smaller.

Copy link
Member

Choose a reason for hiding this comment

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

aha! damn it diff view!

@evgeni evgeni merged commit 2423ff6 into theforeman:rpm/develop Nov 24, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants