The forum

Bonjour à tous,
Ceci est mon tout 1er script et je dois dire que pour ma part, il fonctionne à merveille.
Mais cette fois-ci, c'est à vous à en juger. :-°
Script et jeu fonctionnels sous Wine 1.1.22

Edited by Berillions

Enlève :

POL_SetupWindow_message "Je vais télécharger le patch sur le ftp de Clubic.com.
Cette opération peut nécessiter du temps.
Cliquez sur Suivant pour commencer."

POL_SetupWindow_message "Le patch a été correctement installé"

message "Bienvenue dans le script d'installation du patch 1.05 pour Max Payne"

Modification du script suite aux remarques de Tinou et NSLW

Ces lignes la ne servent à rien

select_prefixe "$HOME/.PlayOnLinux/wineprefix/MaxPayne"
POL_SetupWindow_message "$LNG_INSTPATCH" "Max Payne Patch 1.05"

Remplace
POL_SetupWindow_Close

else
POL_SetupWindow_Close
fi

Par

fi
POL_SetupWindow_Close

et

if [ "$PLAYONLINUX" = "" ]
then
exit 0
fi

par [ "$PLAYONLINUX" = "" ] && exit 0

Modification faites.

Edited by Berillions

Salut Berillions,

J'ai juste quelques petites remarques à faire :

- À la fin de ta commande POL_SW_presentation, le "1 4" est inutile. Je ne sais pas ce quel script tu t'es basé pour faire celui-ci mais ce genre de chose en fin de commande sont des vestiges de la version 2 de POL et donc ne servent plus aujourd'hui.

- Je ne sais pas si tu attend d'avoir un left.jpg ou bien des retours d'expérience d'autres utilisateurs mais quand tu sera décidé à proposer ton script pour qu'il devienne officiel, ça passe par "Add a new script" en haut de la page "Les scripts". (Je précise au cas où, certaines personne ne remarque pas ça)

- Quand tu sera sûr qu'une application marche parfaitement avec une version de Wine, tu pourra ajouter des commandes POL_SW_install_wine et Set_WineVersion_Assign pour que cette version soit utilisée automatiquement et ainsi éviter de possible régressions dans les futures versions de Wine.

- Enfin, tes commentaires sont principalement en français. Ici ça ne pose pas de problème car tu ne fais rien de très compliqué dans le script mais si tu es amené a faire des choses un peu plus complexes, c'est mieux de commenter en anglais car ton script peut très bien être vérifié par quelqu'un qui ne parle pas français.

Voilà, à part c'est très bien pour un premier script, continue comme ça ;-)

Bye

Petites corrections effectués. En attente du left.jpg pour en faire un script officiel.

Concernant la partie du code pour l'assignation de Wine 1.1.22 au jeu, ça n'est pas bon.

D'abord, quand j'écris POL_SW_truc, c'est un abréviation de POL_SetupWindow_truc et ça ne marchera pas si tu écris la commande en abrévié.

Ensuite, il faut faire attention à respecter la casse dans le nom des fonctions, set_WineVersion_Assign n'est pas correct, seul Set_WineVersion_Assign fonctionne.
Toujours à propos de cette commande, tu ne peux pas l'utiliser avant d'avoir utilisé au moins une commande POL_SetupWindow_make_shortcut, comme expliqué dans la référence des fonctions.

Bye

Voilà, j'ai fait les petites modifications. En espérant que ca soit bon cette fois ci :p

Et non, toujours pas ! :D
Tu as laissé set_WineVersion_Assign au lieu de Set_WineVersion_Assign.

Arghh... Merci pour la correction de ces petites fautes :)