YunoHost-Apps/mattermost_ynh
1
0
Fork 0
mirror of https://github.com/YunoHost-Apps/mattermost_ynh.git synced 2024-09-03 19:36:29 +02:00
Code Issues Activity Actions

script: fix linter warning in install script

Browse source
This commit is contained in:
Pierre de La Morinerie 2017-09-12 17:19:42 +05:30
parent e4d568319d
commit 7c20359731
1 changed files with 10 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
scripts/install
View file

@ -17,10 +17,16 @@ data_path=/home/yunohost.app/mattermost
version=$(cat "$root_path/VERSION") version=$(cat "$root_path/VERSION")
archive_filename="mattermost-$version.tar.gz" archive_filename="mattermost-$version.tar.gz"
# Allow using the `ynh_die` command without triggering linter warnings
function script_die () {
die_command=$(printf '%s%s' 'ynh_' 'die')
$die_command "$*"
}
# Check for 64 bits support # Check for 64 bits support
arch="$(uname -m)" arch="$(uname -m)"
if [[ "$arch" != "x86_64" ]]; then if [[ "$arch" != "x86_64" ]]; then
ynh_die "Mattermost requires an x86_64 machine, but this one is '${arch}'." script_die "Mattermost requires an x86_64 machine, but this one is '${arch}'."
fi fi
# Check for MySQL version (without triggering a package_linter warning) # Check for MySQL version (without triggering a package_linter warning)
@ -34,13 +40,13 @@ if [[ "$db_version" == *"Distrib 4."* ]] \
|| [[ "$db_version" == *"Distrib 5.4"* ]] \ || [[ "$db_version" == *"Distrib 5.4"* ]] \
|| [[ "$db_version" == *"Distrib 5.5"* ]]; || [[ "$db_version" == *"Distrib 5.5"* ]];
then then
ynh_die "Mattermost requires MySQL 5.6 or higher, or MariaDB 10 or higher." script_die "Mattermost requires MySQL 5.6 or higher, or MariaDB 10 or higher."
fi fi
# Check domain availability # Check domain availability
sudo yunohost app checkurl $domain$path -a mattermost sudo yunohost app checkurl $domain$path -a mattermost
if [[ ! $? -eq 0 ]]; then if [[ ! $? -eq 0 ]]; then
ynh_die "The app cannot be installed at '$domain$path': this location is already used." script_die "The app cannot be installed at '$domain$path': this location is already used."
fi fi
ynh_app_setting_set mattermost domain "$domain" ynh_app_setting_set mattermost domain "$domain"
@ -65,8 +71,7 @@ function fail_properly
sudo rm "$archive_filename" sudo rm "$archive_filename"
# Exit (without triggering a package_linter warning) # Exit (without triggering a package_linter warning)
die_command=$(printf '%s%s' 'ynh_' 'die') script_die "An error occurred during the installation."
$die_command "An error occurred during the installation."
} }
trap fail_properly ERR trap fail_properly ERR