-
Notifications
You must be signed in to change notification settings - Fork 148
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 facter to 4.0.51 #6348
Update facter to 4.0.51 #6348
Conversation
@@ -88,6 +88,9 @@ cp -a .%{_bindir}/* \ | |||
%doc %{gem_docdir} | |||
|
|||
%changelog | |||
* Wed Feb 24 2021 Lukas Zapletal <lzap+rpm@redhat.com> 4.0.51-1 | |||
- Update to 5.0.51 |
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.
i guess here is a typo. that should be 4.0.51
and not 5.0.51
i assume.
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.
For what it's worth, ./bump_rpm.sh packages/foreman/rubygem-facter
should generally do the right thing for you already. With gems it even knows how to pull the latest gem version.
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.
Damn thanks, why I keep forgetting this script. In the past we had nothing like this and I am still used to the old workflow. Rebased.
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.
It's been there for 3 years now, time to adjust your habits :)
6fbf915
to
a4a7667
Compare
This should fix https://projects.theforeman.org/issues/31943
For plugin updates, please indicate which repos this should be built into:
According to testing of the problem, I might actually file 2.4 PR. But first things first - nightly and I need to test this.