Projet

Général

Profil

Development #21980

tox: erreurs d'exectuion des tests pour iparapheur sous Testing/Sid

Ajouté par Serghei Mihai il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
19 février 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/home/serghei/dev/passerelle/passerelle/contrib/iparapheur/models.py", line 18, in <module>
    import magic
ImportError: No module named magic

A partir de la version 2:0.4.15-1 le module n'est plus dans le fichier magic.py.
Il faudrait adapter getmagic.sh pour poser le bon lien.


Fichiers


Demandes liées

Lié à w.c.s. - Development #22414: prendre en charge le module python-magic (qui n'est pas le binding python de la libmagic)Fermé09 mars 2018

Actions

Révisions associées

Révision 80dd8b2d (diff)
Ajouté par Benjamin Dauvergne il y a environ 6 ans

tox: use python-magic from pypi now (#21980)

See https://lists.debian.org/debian-devel/2018/01/msg00449.html for
reference.

Révision 94b4f5a3 (diff)
Ajouté par Benjamin Dauvergne il y a environ 6 ans

Revert "tox: use python-magic from pypi now (#21980)"

This reverts commit 80dd8b2d506f23196520f98dcdd3e49f8020c4cb.

Révision 9bbd1f2c (diff)
Ajouté par Emmanuel Cazenave il y a presque 6 ans

compat with old and new python-magic (#21980)

Historique

#2

Mis à jour par Frédéric Péters il y a environ 6 ans

Contexte : https://lists.debian.org/debian-devel/2018/01/msg00449.html

tl;dr: l'implémentation dans pypi a gagné une couche de compat avec l'autre python-magic, du coup Debian a basculé là-dessus. → on peut arrêter avec getmagic.sh et juste tirer le module de pypi, en demandant une version supérieure ou égale à 0.4.15.

#4

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

  • Patch proposed changé de Non à Oui
#5

Mis à jour par Serghei Mihai il y a environ 6 ans

Ack.

#6

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

Avec ce patch appliqué, un test échoue chez moi, tests/test_iparapheur.py:107 avec :

jsonresponse.py            133 WARNING  Permission denied     
models.py                  510 INFO     endpoint POST /iparapheur/parapheur/create-file?apikey=iparapheur ('{"type": "Courrier", "content-type": "application/pdf", "subtype": "maire", "visibility": "SERVICE", "d
ata": "VGVzdCBEb2N1bWVudA==", "title": "iparapheur_test"}')
jsonresponse.py            129 ERROR    Error occurred while processing request
Traceback (most recent call last):
  File "/home/cazino/src/passerelle/passerelle/utils/jsonresponse.py", line 117, in api                  
    resp = f(*args, **kwargs)                                                                                                      
  File "/home/cazino/src/passerelle/passerelle/views.py", line 276, in perform
    return self.endpoint(request, **self.get_params(request, *args, **kwargs))                                        
  File "/home/cazino/src/passerelle/passerelle/contrib/iparapheur/models.py", line 125, in create_file
    mime = magic.open(magic.MAGIC_MIME_TYPE)                                                                              
AttributeError: 'module' object has no attribute 'open'

#7

Mis à jour par Serghei Mihai il y a environ 6 ans

Un-ack à moi même...

#8

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

J'ai poussé puis reverté, je confirme c'est la loose en fait la couche de compatibilité n'existe que dans une patch du paquet Debian de sid (fourni par upstream mais pas intégrée au paquet pypi :( c'est sur une branche github https://github.com/ahupp/python-magic/tree/libmagic-compat ).

#9

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

Du coup j'implémente la compat avec les deux versions de l'API, je vois pas d'autre solution.

#10

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

  • Lié à Development #22414: prendre en charge le module python-magic (qui n'est pas le binding python de la libmagic) ajouté
#11

Mis à jour par Thomas Noël il y a environ 6 ans

Pour moi c'est un ack, je vois pas non plus d'autre solution dans la situation actuelle. Patch testé et enfin tox remarche : joie.

(la vraie belle et bonne solution est d'interpeller l'upstream ; fait sur https://github.com/ahupp/python-magic/issues/21 sans succès)

#12

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

  • Statut changé de Nouveau à En cours

Je pousserai semaine prochaine.

#13

Mis à jour par Emmanuel Cazenave il y a presque 6 ans

  • Statut changé de En cours à Résolu (à déployer)
  • Assigné à mis à Emmanuel Cazenave
commit 9bbd1f2ccc7f3011d31fe0d20f8abadddcf13f9e
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Thu Mar 8 11:41:33 2018 +0100

    compat with old and new python-magic (#21980)

#14

Mis à jour par Emmanuel Cazenave il y a presque 6 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#15

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF