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 katello package dependencies #8637

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Oct 26, 2022

No description provided.

BuildRequires: rubygem(pulp_rpm_client) >= 3.17.0
BuildRequires: rubygem(pulp_rpm_client) < 3.18.0
BuildRequires: rubygem(pulpcore_client) >= 3.21.0
BuildRequires: rubygem(pulpcore_client) < 3.22.0
Copy link
Member Author

Choose a reason for hiding this comment

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

@evgeni any thoughts on dropping the version bounds? or at least the upper bound? Since this is a proper gem we should have automatic dependencies based on the gemspec. AFAIK that's still verified, which would make the build fail, just later than it is now. It'd reduce the churn on keeping this in sync. At least until #8414.

To be clear, I think it should be a separate effort, but it can help us. Especially if we're going to build more nightly/scratch packages.

Copy link
Member

Choose a reason for hiding this comment

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

I'd be cool with that

Copy link
Member Author

Choose a reason for hiding this comment

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

Cool. Not sure I can get to it soon, but I'll keep it in mind.

@ekohl
Copy link
Member Author

ekohl commented Oct 26, 2022

[test rpm]

@ekohl
Copy link
Member Author

ekohl commented Oct 26, 2022

Repoclosure fails because it hasn't been mashed yet.

@ekohl ekohl merged commit 7c4ee00 into theforeman:rpm/develop Oct 26, 2022
@ekohl ekohl deleted the rpm/update-katello-deps branch October 26, 2022 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants