Skip to content
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

Fix permissions from 0755 to 0644 #6911

Closed

Conversation

viniciusferrao
Copy link
Member

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
Copy link
Member Author

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?

@gurevichmark
Copy link
Contributor

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

@besawn besawn closed this Feb 17, 2023
@besawn besawn reopened this Feb 17, 2023
@Obihoernchen Obihoernchen added this to the 2.17 milestone Apr 30, 2024
@Obihoernchen
Copy link
Member

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.

@viniciusferrao viniciusferrao deleted the cleanup-compile-warnings branch May 7, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants