commit fd4af860d476ccaff6f5c7202f2b25ffea1860ab
author Kengo Seki <sekikn@apache.org> Fri Jan 29 10:46:06 2021 +0900
committer GitHub <noreply@github.com> Fri Jan 29 10:46:06 2021 +0900
tree 522a9f438208bf493b77e9b30813c56849334388
parent e952272ed7aa28d7e2c36b26d81093ca01aa46a8

BIGTOP-3482. Fix toolchain not to break cmake by creating unnecessary symlink. (#719)

bigtop_toolchain/manifests/packages.pp

diff --git a/bigtop_toolchain/manifests/packages.pp b/bigtop_toolchain/manifests/packages.pp
index c42c549..b0bbb59 100644
--- a/bigtop_toolchain/manifests/packages.pp
+++ b/bigtop_toolchain/manifests/packages.pp
@@ -260,16 +260,18 @@
     command => "/usr/bin/pip3 freeze --all; /usr/bin/pip3 --version; /usr/bin/pip3 install -q flake8 wheel",
   }
 
-  if ($operatingsystem == 'Fedora' and versioncmp($operatingsystemmajrelease, '31') >= 0) or
-     ($osfamily == 'RedHat' and $operatingsystem != 'Fedora' and versioncmp($operatingsystemmajrelease, '8') >= 0) {
-    file { '/usr/bin/python':
-      ensure => 'link',
-      target => '/usr/bin/python2',
-    }
-  } else {
-    file { '/usr/bin/cmake':
-      ensure => 'link',
-      target => '/usr/bin/cmake3',
+  if $osfamily == 'RedHat' {
+    if ($operatingsystem == 'Fedora' and versioncmp($operatingsystemmajrelease, '31') >= 0) or
+       ($operatingsystem != 'Fedora' and versioncmp($operatingsystemmajrelease, '8' ) >= 0) {
+      file { '/usr/bin/python':
+        ensure => 'link',
+        target => '/usr/bin/python2',
+      }
+    } else {
+      file { '/usr/bin/cmake':
+        ensure => 'link',
+        target => '/usr/bin/cmake3',
+      }
     }
   }
 }