Exercice ShipDict : l'option -z (--gzip) de merger.py ne fonctionne pas

[aollier]

$ python3 merger.py --gzip json/*
Opening ALL_SHIPS.txt for listing ships
Opening ALL_SHIPS.kmz for ship ALL_SHIPS
Something went wrong, <class 'ValueError'>, Argument 'newline' not supported in binary mode
Traceback (most recent call last):
  File "merger.py", line 188, in main
    kml_filename = self.write_kml_output(ships, output_name)
  File "merger.py", line 126, in write_kml_output
    with gzip.open(kml_filename, 'w', newline="\n") if self.args.gzip \
  File "/usr/lib/python3.8/gzip.py", line 54, in open
    raise ValueError("Argument 'newline' not supported in binary mode")
ValueError: Argument 'newline' not supported in binary mode

$ python3 merger.py -z json/*
Opening ALL_SHIPS.txt for listing ships
Opening ALL_SHIPS.kmz for ship ALL_SHIPS
Something went wrong, <class 'ValueError'>, Argument 'newline' not supported in binary mode
Traceback (most recent call last):
  File "merger.py", line 188, in main
    kml_filename = self.write_kml_output(ships, output_name)
  File "merger.py", line 126, in write_kml_output
    with gzip.open(kml_filename, 'w', newline="\n") if self.args.gzip \
  File "/usr/lib/python3.8/gzip.py", line 54, in open
    raise ValueError("Argument 'newline' not supported in binary mode")
ValueError: Argument 'newline' not supported in binary mode

[parmentelat]

sans regarder le code, ça ressemble à un paramètre qu'on passe au moment de forker gzip, et ça pourrait dater du passage de Python2 à Python3; rien de bien méchant à corriger, si un volontaire voulait s'en occuper :)

[aollier]

Je vais m'y coller. Je ne sais pas combien de temps ça va me prendre parce que je dois regarder dans le détail, trouver, corriger et tester.

[parmentelat]

ce bug est là depuis plusieurs années sans que personne ne s'en soit plaint avant toi, il n'y aucune espèce d'urgence :)