Demande d'intégration about portablemc

[mindstorm38]

Il me semble d'avec tkinter, ou plus généralement les applications "windowed" (fenétrées), il faut explicitement dire à l'interpréteur python de ne pas lancer la console.

Sinon je pense que cette réponse stackoverflow peut vous intéresser (cela dépend de l'OS utilisé).

En tout cas ça faire plaisir de voir que qqun s'intéresse au projet ^^

[SyFizz]

En soi, c'est pas tant mon GUI tkinter le problème, c'est la fênetre de logs du jeu...

Même en lançant lançant le launcher avec pythonw, effectivement les logs spécifiques au laucher ne s'affichent pas, mais à l'ouverture du jeu j'ai une fênetre java.exe qui m'affiche les logs :(

Je pense qu'il faudrait trouver comment le launcher officiel les met dans un fichier plutôt que dans une fênetre qui reste ouverte

P.S : En plus de m'intéresser au projet, j'habite aux alentours de grenoble aussi :p
Et pour info, le projet scolaire sur lequel j'utilise votre launcher se passe au lycée Portes de L'Oisans, à Vizille

[mindstorm38]

Quel OS? C'est pour savoir car la fonction os.system ne régis pas pareil suivant les systèmes

[SyFizz]

Sous Windows 10 :)

[mindstorm38]

Je pense avoir trouvé, un détail m'avait échappé! Vous utilisez la commande "start" pour lancer le script, le problème de cette commande est qu'elle ouvre systématiquement l'application qui peut lancer le programme donné (donc le shell python ici, même si c'est dans un invite de commande). Du coup il faudrait plutôt utiliser l'executable python à la place de "start" (récupérable avec sys.executable).

PS: Sur les dernières versions de python il est conseillé d'utiliser le module subprocess à la place de os.system, ou au moins la fonction subprocess.run. Mais bon, faites ça si mon correctif fonctionne.

[SyFizz]

Donc si je fais os.system(sys.executable() './bin/portablemc.py -u '+getInGameName()+' -v '+getSelectedVersion() + ' --main-dir ./data --jvm ./bin/bin/java.exe'), c'est censé fonctionner ? Si oui, ça ne fonctionne pas, j'ai juste plus rien qui apparaît :(

[mindstorm38]

Dans l'idée c'est ça! Cependant je conseillerais de l'écrire en une seule chaine de formatage, par exemple :

os.system("{} ./bin/portablemc.py -u {} -v {} --main-dir ./data --jvm ./bin/bin/java.exe".format(sys.executable(), getInGameName(), getSelectedVersion()))

Le but est d'éviter les petites erreurs comme l'espace ou le '+' manquant après sys.executable(), ça donne une meilleur idée du résultat final. Ces erreurs pourraient être à l'origine du problème?

[SyFizz]

Désolé, je suis un peu nul en python, mon truc c'est le java 🙁
Je teste ça de suite ^^

EDIT: Marche po

Exception in Tkinter callback
Traceback (most recent call last):
  File "C:\Users\UtilisateurLambda\AppData\Local\Programs\Python\Python39\lib\tkinter\__init__.py", line 1892, in __call__
    return self.func(*args)
  File "E:\MC\main.pyw", line 20, in launchGame
    os.system("{} ./bin/portablemc.py -u {} -v {} --main-dir ./data --jvm ./bin/bin/java.exe".format(sys.executable(), getInGameName(), getSelectedVersion()))
TypeError: 'str' object is not callable

EDIT 2:
J'ai juste remplacé les premiers {} par pythonw (en changeant le format(), évidemment)
Et du coup j'ai toujours le même soucis :

[mindstorm38]

Ahhhhhhh désolé xD sys.executable est une chaine, pas un fonction!
Ah, et aussi il faudrait essayer avec l'executable javaw (au lieu de java).

[SyFizz]

Okay, je tente ça de suite, j'édite ensuite :)

ça fonctionne avec javaw.exe ! Merci beaucoup, le problème était là depuis le début :p

[mindstorm38]

C'est tout de même très bizarre, car en testant sur mon PC j'avais pas le pb (merci windows xD)!

[SyFizz]

Ahah, désolé pour le faux espoir, mais cacher la console n'était pas ma seule requête :p

Du coup, je voudrais savoir si lors du téléchargement des fichiers du jeu, il y a moyen de récupérer le "xx.xx% of total" pour en faire une barre de progression tkinter ?

Si vous préférez, on peut passer sur Discord afin d'éviter de flooder les issues sur le repo x)

[mindstorm38]

=> Mindstorm38#8760

[SyFizz]

Ajouté :)