OK
AJAX error!

Les forumsGrammalecte[PATCH] grammalecte-server: fix wrong version compare

[PATCH] grammalecte-server: fix wrong version compare

Bonjour,

Le code utilisé dans grammalecte-server pour comparer la version de Python utilisée n'est pas bon (comparaison d'un entier vs. un tuple). Voici un patch appliqué à grammalecte dans Debian et qui aurait intérêt à être intégré dans le code source original pour que tout le monde en profite.

Cordialement,

Romain.

---
grammalecte-server.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/grammalecte-server.py b/grammalecte-server.py
index 3cd9a82..38c89e0 100644
--- a/grammalecte-server.py
+++ b/grammalecte-server.py
@@ -320,7 +320,7 @@ def main (sHost="localhost", nPort=8080, dOptions=None, bTestPage=False, nMultiC

# Python version
print("Python: " + sys.version)
- if sys.version_info.major < (3, 7):
+ if sys.version_info.major < 3 or sys.version_info.minor < 7:
print("Python 3.7+ required")
return
# Grammalecte
le 21 mai 2021 à 22:21
Attention, cette correction n’est pas bonne. C’est l’opérateur « ET » qui devrait être utilisé dans la condition… Sans quoi les versions futures seront, pour partie, détectées incorrectement.
le 22 mai 2021 à 14:14
Ça ne marcherait pas non plus.
La version 4.0 par exemple échouerait au test.
il faut écrire quelque chose comme :
if sys.version_info.major < 3 or (sys.version_info.major == 3 and sys.version_info.minor < 7):
le 23 mai 2021 à 09:27
Merci pour vos retours, voici la nouvelle version du patch appliquée dans le paquet Debian de grammalecte :

diff --git a/grammalecte-server.py b/grammalecte-server.py
index 3cd9a82..a91ed13 100644
--- a/grammalecte-server.py
+++ b/grammalecte-server.py
@@ -320,7 +320,7 @@ def main (sHost="localhost", nPort=8080, dOptions=None, bTestPage=False, nMultiC

# Python version
print("Python: " + sys.version)
- if sys.version_info.major < (3, 7):
+ if sys.version_info.major < 3 or (sys.version_info.major == 3 and sys.version_info.minor < 7):
print("Python 3.7+ required")
return
# Grammalecte
le 25 mai 2021 à 16:53

Notification par e-mail    0