Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
Make it a mere warning on Cygwin if any of the scripts in build/pkgs …
Browse files Browse the repository at this point in the history
…are executable, but a hard error on other platforms
  • Loading branch information
embray committed Jul 6, 2017
1 parent 954630f commit 1b96a66
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions build/bin/sage-spkg
Original file line number Diff line number Diff line change
Expand Up @@ -693,8 +693,17 @@ fi
for script in build install check; do
if [ -f "spkg-$script" ]; then
if [ -x "spkg-$script" ]; then
echo >&2 "WARNING: spkg-$script should not be marked executable"
echo >&2 "in the build/pkgs directory."
msg="spkg-$script should not be marked executable in the \
build/pkgs directory"
if [ "$UNAME" = "CYGWIN" ]; then
# On Cygwin we can't neecessarily rely on file permissions
# being sane, so just issue a warning; on other platforms
# this should be enforsed as an error
echo >&2 "WARNING: $msg"
else
error_msg "$msg"
exit 1
fi
fi

write_script_wrapper $(pwd)/"spkg-$script"
Expand Down

0 comments on commit 1b96a66

Please sign in to comment.