Bonjour,
C'est plutôt bien pour un premier script !
Mais bon, on peut toujours trouver des choses améliorables :
- dans la majorité des cas "start /unix" est inutile, et a pour effet de bord qu'annuler une installation ne stoppe pas immédiatement les programmes Windows lancés, donc à éviter si ça marche sans
- bug mineur:
cd "$POL_System_TmpDir"
POL_System_TmpCreate n'ayant pas été utilisé auparavant dans le script, aucun répertoire temporaire privé n'a été créé et $POL_System_TmpDir n'est pas defini !
Bon, je viens de vérifier que cd "" ne change pas le répertoire courant, donc le fichier doit atterrir dans ~/.PlayOnLinux/tmp/ (je pense), pas de conséquences funestes ;)
-
POL_SetupWindow_wait "$(eval_gettext 'PlayOnLinux is installing your application...')" "$TITLE"
POL_Wine_WaitBefore "$TITLE"
Le POL_SetupWindow_wait est redondant, POL_Wine_WaitBefore "$TITLE" fait quasiment la même chose
- Plus compliqué à corriger, c'est une bonne pratique de retarder le plus possible la création du disque virtuel ("prefix" dans le parler Wine), de façon à ce qu'en cas d'interruption du script les chances de laisser un disque virtuel à moitié initialisé soient moindres; ça demande de réorganiser un peu l'ordre des opérations :
* présentation
* questions à l'utilisateur (mode d'installation, emplacement du fichier local si c'est le mode choisi)
* téléchargement (si c'est le choix de l'utilisateur)
* création du disque virtuel
* installation de dépendances
* installation du logiciel
* création du/des raccourci(s)
Edité par petch