-
Notifications
You must be signed in to change notification settings - Fork 24
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 #10469: Make SLES12 RPMs depend on java8 headless #1305
Fixes #10469: Make SLES12 RPMs depend on java8 headless #1305
Conversation
rudder-jetty/SPECS/rudder-jetty.spec
Outdated
@@ -88,10 +88,13 @@ Requires: jre >= 1.8 | |||
Requires: java-1.8.0-openjdk-headless | |||
%endif | |||
|
|||
%if 0%{?suse_version} && 0%{?suse_version} == 1315 && 0%{?sle_version} && 0%{?sle_version} >= 120100 |
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 think we can remove the "suse_version" checks.
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.
Apparently SLES12 has:
# rpm --eval '%{suse_version}'
1315
SLES11 has:
# rpm --eval '%{suse_version}'
1110
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.
But no sle_version:
SLE11
# rpm --eval '%{sle_version}'
%{sle_version}
SLES12
# rpm --eval '%{sle_version}'
120200
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 is consistent with what is documented there: https://en.opensuse.org/openSUSE:Build_Service_cross_distribution_howto
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.
Here we are targeting SLES12SP1+, so 0%{?sle_version} && 0%{?sle_version} >= 120100
should be ok.
Commit modified |
caf00c3
to
e186ba5
Compare
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/10469