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
Fixes #10066: Scary message about OpenSSL on SLES when running rudder agent update #1272
Conversation
rudder-agent/SPECS/rudder-agent.spec
Outdated
@@ -189,7 +189,7 @@ Requires: dmidecode | |||
### SLES 11 OSes come with OpenSSL 0.9.8h which is recent enough. | |||
## | |||
# Reference for suse_version : https://en.opensuse.org/openSUSE:Build_Service_cross_distribution_howto | |||
%if 0%{?suse_version} && 0%{?suse_version} < 1100 | |||
%if 0%{?suse_version} && 0%{?suse_version} < 1140 |
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.
Is the openssl version higher on SP4 ? It seems to me the actual condition is < 1200
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 seems logical, it's just that we didn't do that for the other tests
Commit modified |
fb11107
to
fbcab40
Compare
rudder-agent/SPECS/rudder-agent.spec
Outdated
@@ -144,12 +144,12 @@ Requires: dmidecode | |||
|
|||
## 3 - SLES: No LMDB yet | |||
# Reference for suse_version : https://en.opensuse.org/openSUSE:Build_Service_cross_distribution_howto | |||
%if 0%{?suse_version} && 0%{?suse_version} < 1140 |
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.
This one is a change in OpenSuSE 11.4
fbcab40
to
690d090
Compare
Commit modified |
rudder-agent/SPECS/rudder-agent.spec
Outdated
@@ -149,7 +149,7 @@ Requires: pmtools | |||
%define use_system_lmdb false | |||
%endif | |||
|
|||
%if 0%{?suse_version} && 0%{?suse_version} >= 1140 |
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.
This was was also OpenSuSE 11.4 specific (it is the other part of the first one)
Commit modified |
690d090
to
c181a78
Compare
OK, merging this PR |
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/10066