Skip to content

Handle macOS permissions in install script #10488

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.

Sign up for GitHub

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

Open
Montekkundan wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Handle macOS permissions in install script #10488

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 16 additions & 4 deletions tools/install.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -117,12 +117,14 @@ mkdir -p ../smarty/templates_c
# Setting 770 permissions for templates_c
sudo chmod 770 ../smarty/templates_c

# Detecting distribution
if ! os_distro=$(hostnamectl 2>/dev/null)
# Detecting operating system/distribution
if [[ "$(uname -s)" == "Darwin" ]]; then
os_distro="macOS"
elif ! os_distro=$(hostnamectl 2>/dev/null)
then
os_distro="Other"
os_distro="Other"
else
os_distro=$(hostnamectl |awk -F: '/Operating System:/{print $2}'|cut -f2 -d ' ')
os_distro=$(hostnamectl |awk -F: '/Operating System:/{print $2}'|cut -f2 -d ' ')
fi

debian=("Debian" "Ubuntu")
Expand All @@ -140,6 +142,11 @@ elif [[ " ${redhat[*]} " == *" $os_distro "* ]]; then
# can write the config.xml file.
sudo chgrp apache ../project
sudo chmod 770 ../project
elif [[ "$os_distro" == "macOS" ]]; then
current_user=$(id -un)
sudo chown "$current_user":staff ../smarty/templates_c
sudo chgrp staff ../project
sudo chmod 770 ../project
else
echo "$os_distro Linux distribution detected. We currently do not support this. "
echo "Please manually change subdirectory ownership and permissions to ensure the web server can read *and write* in the following: "
Expand All @@ -155,6 +162,8 @@ if [ -d logs ]; then
sudo chgrp www-data logs
elif [[ " ${redhat[*]} " == *" $os_distro "* ]]; then
sudo chgrp apache logs
elif [[ "$os_distro" == "macOS" ]]; then
sudo chgrp staff logs
else
echo "$os_distro Linux distribution detected. We currently do not support this. Please manually set the permissions for the directory tools/logs/"
fi
Expand Down Expand Up @@ -244,6 +253,9 @@ while true; do
* ) echo "Please enter 'y' or 'n'."
esac
done;
elif [[ "$os_distro" == "macOS" ]]; then
echo "macOS detected."
echo "Automatic Apache configuration is not supported on macOS. Please configure Apache manually using the macOS developer install guide."
else
echo "$os_distro Linux distribution detected. We currently do not support this. Please configure Apache manually."
exit 1
Expand Down
Loading