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
install-deps.sh: use DTS on centos if GCC is too old #19398
Conversation
062f266
to
3c84eea
Compare
@shinobu-x could you help test this? the goal here is to make sure that |
a89c585
to
987d06f
Compare
@tchaikov ack |
e5953c8
to
71873f7
Compare
|
|
c673062
to
decf048
Compare
please note, run-make-check.sh sources install-deps.sh here to import the $PATH and other environmental variables, which could be changed by the the DTS "enable" script. Signed-off-by: Kefu Chai <kchai@redhat.com>
decf048
to
e68a3a2
Compare
@shinobu-x thanks for testing. fixed and repushed. |
835ede6
to
97dbcdb
Compare
the DTS-6 is only available in the testing repo Signed-off-by: Kefu Chai <kchai@redhat.com>
97dbcdb
to
1b24952
Compare
. /opt/rh/devtoolset-6/enable | ||
%else | ||
. /opt/rh/devtoolset-7/enable |
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.
@ktdreyer this reflects the different support of DTS from RHEL and CentOS.
lgtm! |
1b24952
to
54030a3
Compare
devtoolset-7 is available for RHEL, but not for centos. only dts-6 is available for centos. Signed-off-by: Kefu Chai <kchai@redhat.com>
54030a3
to
b38621f
Compare
@@ -50,6 +50,38 @@ function ensure_decent_gcc { | |||
$SUDO ln -nsf /usr/bin/g++ /usr/bin/x86_64-linux-gnu-g++ | |||
} | |||
|
|||
function version_lt { | |||
test $1 != $(echo -e "$1\n$2" | sort -rV | head -n 1) |
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.
Nice!
Signed-off-by: Kefu Chai kchai@redhat.com