Merci infiniment pour tes commentaires. Effectivement la nuit aidant (et après avoir testé cette nouvelle version sur le PC de ma copine) je me suis rendu compte des erreurs que j'avais introduit après mon dernier test mais avant de publier le script.
> - Pas vraiment nécessaire de préfixer les messages de log avec le titre, vu que chaque log d'installation va dans son propre fichier
Done.
> - l'exportation de POL_WINEVERSION est inutile, vu que la variable n'est utilisée qu'à l'intérieur du script
Effectivement, j'ai trouvé ce bout de code sur un autre script mis à jour récemment, j'ai pensé bêtement que la variable POL_WINEVERSION était interne à PlayOnLinux, d'où le besoin de l'exporter.
Ok, je regarde ça dès que j'ai résolu tout le reste.
> - Juste pour info il me semble bien que POL_System_SetArch "x86" est la valeur par défaut (mais ça ne fait pas de mal)
La majorité des problèmes que j'ai rencontré avec l'ancien script provenaient de conflits avec les dépendances installés dans le wineprefix, qui ne respectent pas toujours la structure Program Files / Program Files (x86), ou du raccourci qui ne retrouve pas sa cible (il ca cherche dans System32...). De plus dotnet ne peut pas s'installer sur une archi x64, donc je préfère laisser cette ligne.
> - export POL_Wine_PrefixCreate "$POL_WINEVERSION" euh? POL_Wine_PrefixCreate est une fonction pas une variable, je pense que l'"export" est un reste d'une ancienne ligne...
Alors là j'ai carrément craqué. Je devais pas avoir les yeux en face des trous. Mon script ne fonctionnait plus du tout à cause de cette ligne... Corrigé
J'en prend bonne note et j'update
> - Set_SoundDriver "alsa"
> Set_Managed "Off"
> Pas sûr du réel bénéfice de ces deux lignes; La première est censé contourner des problèmes avec PulseAudio, la deuxième peut aider (ou pas) sous Mac. Perso on m'a conseillé de ne les mettre que si elles ont un bénéfice avéré
Pour alsa ok, je retire. Pour le Set_Managed j'ai des problèmes si je le laisse à On (par défaut) sur ma 12.10. En passant à Off, le plein écran fonctionne parfaitement. Le seul petit hic pour lequel je n'ai pas de fix pour le moment, c'est que mon second écran est désactivé quand je lance le jeu, et n'est pas réactivé à la fermeture. Je dois donc aller dans les options d'affichage pour réactiver l'écran à chaque fois. C'est pas hyper contraignant, mais je suis preneur si une solution existe.
> - Pourquoi préfixer les raccourcis avec "PlayOnLinux - " ? Leurs noms vont apparaître dans l'interface de PlayOnLinux, ça prend de la place sans aucun bénéfice
A la fin de l'install des sims, un raccourci est crée sur le bureau (Linux) et n'est pas (totalement) fonctionnel. Il s'agit d'un lanceur qui contrôle entre autre le checksum de l’exécutable du jeu. J'ai mis le préfix pour que l'utilisateur puisse voir le bon raccourci, celui qui correspond à l'install POL.
> - POL_System_TmpDelete va de pair avec POL_System_TmpCreate, en l'état il peut disparaître
Done.
> Mais c'est clairement sur la bonne voie, je suis allé voir le script existant pour me faire peur :p
> Cordialement,
> Pierre.
Encore merci pour tes retours, je me remet au boulot et je reposte une nouvelle version soon ;)
Edited by Tawane