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 rubygem-foreman_bootdisk to 19.0.6 #8311

Closed

Conversation

stejskalleos
Copy link

@stejskalleos stejskalleos commented Aug 9, 2022

For Foreman 3.3

@stejskalleos stejskalleos changed the base branch from rpm/develop to rpm/3.3 August 9, 2022 08:48
@stejskalleos
Copy link
Author

@theforeman/packaging Hi can we merge the PR?

@evgeni
Copy link
Member

evgeni commented Sep 1, 2022

Not if it's not building? ;)

Requires: %{?scl_prefix_ruby}ruby(release)
Requires: %{?scl_prefix_ruby}ruby >= 2.5
Requires: %{?scl_prefix_ruby}ruby(rubygems)
Requires: ruby >= 2.5
Copy link
Member

Choose a reason for hiding this comment

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

You're dropping SCL macros, but 3.3 us still building for EL7 with SCL

BuildRequires: npm(@babel/core) >= 7.7.0
BuildRequires: npm(@babel/core) < 8.0.0
BuildRequires: npm(@theforeman/builder) >= 8.15.0
BuildRequires: npm(@theforeman/builder) < 9.0.0
Copy link
Member

Choose a reason for hiding this comment

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

I bet this one can't be satisfied (we're using v10) and should not be here

@theforeman-bot theforeman-bot added Waiting on contributor RPM Stable branch PRs that are opened against a stable branch. Usually a cherry pick and removed Not yet reviewed labels Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs testing RPM Stable branch PRs that are opened against a stable branch. Usually a cherry pick Waiting on contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants