search

YunoHost-Apps / lutim_ynh

Lutim package for YunoHost
https://lut.im
GNU General Public License v3.0
5 stars 7 forks source link

Level 5 forcé #14

Closed maniackcrudelis closed 7 years ago

maniackcrudelis commented 7 years ago

INSTALL SCRIPT ✘ At line 136 'ynh_die' or 'exit' command is executed with system modification before. ... Oui je pourrais le mettre avant, mais pour la clarté du script, je le préfère ici ! ✘ set -eu is missing at beginning of file. Remplacé par ynh_abort_if_errors

REMOVE SCRIPT ✘ set -u is missing at beginning of file. Usage de SECURE_REMOVE

UPGRADE SCRIPT ✘ set -eu is missing at beginning of file. Remplacé par ynh_abort_if_errors

BACKUP SCRIPT ✘ set -eu is missing at beginning of file. Il est bien présent au début du script !

RESTORE SCRIPT ✘ At line 51 'ynh_die' or 'exit' command is executed with system modification before. Aucune modification avant. ✘ set -eu is missing at beginning of file. Il est bien présent au début du script !

.FONCTIONS SCRIPT ✘ Line 79 you should add "sudo" before this command line: rm -r $temp_dir sudo n'est pas nécessaire. Et c'est mieux sans ! ✘ Line 244/332/333: 'exit' command shouldn't be used. Use 'ynh_die' helper instead. Variable exit_code ✘ Line 331/357: 'exit' command shouldn't be used. Use 'ynh_die' helper instead. Fonction ynh_exit_properly ✘ Line 334: 'exit' command shouldn't be used. Use 'ynh_die' helper instead.*** exit 0, ce n'est pas un exit sur une erreur.