xcat-core icon indicating copy to clipboard operation
xcat-core copied to clipboard

Fix permissions from 0755 to 0644

Open viniciusferrao opened this issue 3 years ago • 2 comments

The PR is to fix issue #6910

The modification include

modified:   xCAT-server/share/xcat/ib/netboot/sles/ib.postinstall
modified:   xCAT-server/share/xcat/install/scripts/post.ubuntu
modified:   xCAT-server/share/xcat/install/scripts/post.xcat
modified:   xCAT-server/share/xcat/install/scripts/pre.pkvm3
modified:   xCAT-server/share/xcat/install/scripts/pre.rh.rhels7
modified:   xCAT-server/share/xcat/install/windows/enterprise.win7.x86_64.tmpl
modified:   xCAT-server/share/xcat/install/windows/enterprise.win8.x86_64.tmpl
modified:   xCAT-server/share/xcat/install/windows/ultimate.win7.x86_64.tmpl
modified:   xCAT-server/share/xcat/mypostscript/mypostscript.tmpl
modified:   xCAT-server/share/xcat/netboot/sles/compute.sles11.ppc64.exlist
modified:   xCAT-server/share/xcat/netboot/sles/compute.sles11.x86_64.exlist
modified:   xCAT-server/share/xcat/netboot/sles/compute.sles12.ppc64le.exlist
modified:   xCAT-server/share/xcat/netboot/sles/compute.sles12.x86_64.exlist
modified:   xCAT-server/share/xcat/netboot/ubuntu/compute.exlist
modified:   xCAT-server/share/xcat/netboot/windows/detectefi.cpp
modified:   xCAT-server/share/xcat/netboot/windows/getnextserver.cpp
modified:   xCAT-server/share/xcat/netboot/windows/xCAT.psd1
modified:   xCAT-server/share/xcat/scripts/createdb.sql
modified:   xCAT-server/share/xcat/scripts/setup-server-cert.sh
modified:   xCAT-server/share/xcat/scripts/setup-xcat-ca.sh
modified:   xCAT-server/share/xcat/scripts/updateDBM.sql
modified:   xCAT-server/share/xcat/tools/powervm/vios.conf
modified:   xCAT/postscripts/disableconsistentNICrename
modified:   xCAT/postscripts/serialconsole
modified:   xCAT/postscripts/setupesx
modified:   xCAT/postscripts/xcatlib.sh
modified:   xCAT/postscripts/xcatpostinit.service

viniciusferrao avatar Mar 03 '21 20:03 viniciusferrao

Well perhaps this patch really broke something. I'll try to understand the errors on Travis. But as I can see this is tested against Ubuntu right? Is there any information about the variant that's tested?

viniciusferrao avatar Mar 05 '21 20:03 viniciusferrao

Looking at the "View config" tab it appears to be xenial

gurevichmark avatar Mar 05 '21 20:03 gurevichmark

Thank you for your contribution! There are some issues with this PR because it removes executable bit from some scripts with missing shebang lines. This will be fixed in #7441.

Obihoernchen avatar May 07 '24 14:05 Obihoernchen