El Foro

Un script pour la version originale sur CD de Sacrifice.
Ce coup-ci je me suis relu avant de poster. ;)

Voir le script

Editado por: Xodetaetl

Bonjour :)

POL_SetupWIndow_message "$(eval_gettext 'This will let you install Sacrifice, then will download and install its patch #3. Do not launch the game until the patch is installed.')" "$(eval_gettext 'Note')"

Je parie que tu ne vois pas ce message pendant l'installation, est-ce que tu vois pourquoi ? ;)

Editado por: petch

Huh, j'ai ajouté ce morceau au dernier moment et je n'ai pas testé. Ça me paraissait inutile de re-tester... pratiquement. :P

Donc effectivement le message n'apparaît pas, et après 2 minutes de réflexion et tests, je ne vois pas pourquoi.

Ah c'est bon, trouvé. ^^
La coloration syntaxique ici-bas est utile. :)

Editado por: Xodetaetl

Bonjour,

Il faudrait changer 2-3 petits points, en particuliers pour les nouveaux scripts (les scripts déjà validés seront progressivement changés) :

[code language=playonlinux]POL_SetupWindow_wait "$(eval_gettext 'Please wait while $TITLE is installed.')" "$TITLE"[/code]
À remplacer par :
[code language=playonlinux]POL_Wine_WaitBefore "$TITLE"[/code]

Concernant le message suivant :
[code language=playonlinux]POL_SetupWindow_browse "$(eval_gettext 'Please select the setup file to run.')" "$TITLE"[/code]
il faudrait juste supprimer le point à la fin.

Ces modifications des messages ne sont pas là pour compliquer la vie, mais afin d'éviter la multiplication des messages à traduire pour les traducteurs du lanchpad.

Et enfin, dernier point important :
[code language=playonlinux]POL_Shortcut_Document "$SHORTCUT_NAME" "$WINEPREFIX/drive_c/Program Files/Shiny/Sacrifice/Readme.txt"[/code]
à remplacer par :
[code language=playonlinux]POL_Shortcut_Document "$SHORTCUT_NAME" "$WINEPREFIX/drive_c/$PROGRAMFILES/Shiny/Sacrifice/Readme.txt"[/code]
L'utilisation de la variable $PROGRAMFILES est obligatoire pour "Program Files", car le nom du dossier change avec certaines langues.

Autrement, c'est parfait.

Editado por: SuperPlumus

Pas de problème SuperPlumus, je corrige ça pour les futurs scripts. Celui-ci je ne peux plus l'éditer dans la BDD, j'ai demandé sa signature.

Cela dit, ça vous gagnerait peut-être du temps de mettre à jour en amont la doc plutôt que corriger les scripts des petits nouveaux après coup. ;)
Ayant vu que la doc était pas mal obsolète et incomplète, je me suis inspiré de scripts ici et là, fait des collages douteux... :D

Pour la doc, on sait quelle est pas vraiment à jour.

Ce n'était pas une priorité ces derniers temps, car on avait pas de personnes qui faisait des scripts en dehors de l'équipe, donc, on s'occupait de plus important.

Mais ces derniers temps, on voit que ça pose parfois problème.

Je compte la remettre à jour (généralement, c'est moi qui m'en occupe), quand je trouverai un peu plus de temps.

C'est pour ça que j'ai signalé ces 2-3 choses, c'est que ce n'est pas possible de les trouver dans la doc.

Mais en attendant, on corrige ce que l'on peut.

Donc, pas de souci, tu ne pouvais pas savoir ces règles. Puisque-elles sont indiquée nulle part.

Editado por: SuperPlumus

Pour la doc, on sait quelle est pas vraiment à jour.

Ce n'était pas une priorité ces derniers temps, car on avait pas de personnes qui faisait des scripts en dehors de l'équipe, donc, on s'occupait de plus important.

Je comprends bien, et ce n'est pas la chose la plus drôle à faire mettre à jour une doc.

Pour ce qui est de la complétion, ce qui me manque pas mal c'est une liste des variables globales.