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

#!/bin/bash
# Date: (2009-06-14 14-32)
# Last revision: (2009-06-17 16-45)
# Distribution used to test: Ubuntu Jaunty
# Wine version used: 1.1.22 
# Licence: Free
# Author: Berillions
 
#fetching PROGRAMFILES environmental variable
PROGRAMFILES=`wine cmd /c echo "%ProgramFiles%"`
PROGRAMFILES=${PROGRAMFILES:3} 
 
#Vérifier que PlayOnLinux est bien exécuté avant
[ "$PLAYONLINUX" = "" ] && exit 0 
 
#Charger les librairies
source "$PLAYONLINUX/lib/sources"
 
Title="Max Payne"
 
if [ "$POL_LANG" == "fr" ]; then
LNG_ERR="Si vous rencontrer deux messages d'erreur concernant un manque de mémoire au début de l'installation,
ne vous en fiez pas. le jeu fonctionnera tout de même."
LNG_PATCH="Voulez-vous installer le patch 1.05 ?"
LNG_DLPATCH="Téléchargement du patch. 
Cliquez sur valider pour continuer."
LNG_WAIT_END="Appuyez sur \\"Suivant\\" UNIQUEMENT quand l'installation du jeu sera
terminée sous peine de devoir recommencer l'installation."
else
LNG_ERR="If you have two error messages before the installation, don't worry. You will be able to launch the game."
LNG_PATCH="Do you want to install the patch 1.05 ?"
LNG_DLPATCH="Downloading patch.
Click to go on."
LNG_WAIT_END="Click on \\"Next\\" ONLY when the game installation
is finished or you will have to redo the installation.."
fi
 
POL_SetupWindow_Init "" ""
 
#Presentation
POL_SetupWindow_presentation "MaxPayne" "RockstarGame" "http://maxpayne.godgames.com/main.htm/" "Berillions" "MaxPayne"

#Installation de Wine
POL_SetupWindow_install_wine "1.1.22"

#Préparation de Wine
select_prefixe "$REPERTOIRE/wineprefix/MaxPayne"
POL_SetupWindow_prefixcreate
 
#Configuration de Wine
Set_OS winXP
 
#Détection du cd-rom
POL_SetupWindow_cdrom
POL_SetupWindow_check_cdrom "Install.exe" 
 
POL_SetupWindow_message "$LNG_ERR" "$Title"
 
wine $CDROM/Install.exe
 
POL_SetupWindow_message "$LNG_WAIT_END" "$Title"
 
#Création Launcher 
POL_SetupWindow_make_shortcut "MaxPayne" "$PROGRAMFILES/Max Payne" "MaxPayne.exe" "" "Max Payne"

Set_WineVersion_Assign "1.1.22" "Max Payne"
 
POL_SetupWindow_question "$LNG_PATCH" "Max Payne Patch 1.05"
if [ "$APP_ANSWER" == "TRUE" ] ;then
 
cd $HOME/.PlayOnLinux/tmp
POL_SetupWindow_download "$LNG_DLPATCH" "Max Payne Patch 1.05" "http://ftpclubic92.clubic.com/temp-clubic-rx492/longtail/patch/Max_Payne_Patch_1.05.exe"
 
wine "./Max_Payne_Patch_1.05.exe"
 
rm -f "./Max_Payne_Patch_1.05.exe"
 
fi
POL_SetupWindow_Close
exit

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 :)