Produits Entr'ouvert » w.c.s.

Comme je disais déjà à l'époque "ça me dépasse un peu" et donc si d'autres peuvent jeter un oeil, c'est cool.

Ce que je vois moi c'est qu'il y a des différences par rapport à ce qui a été fait dans hobo (#22981), où on a aussi une version Python 3 de MainThread::_init__ mais pas de _stop ou de is_alive sur _DummyThread ... Ça serait sans doute mieux de tenter de converger (quitte à revenir un peu sur #22981 aussi) ?

Je regarde.

Ça serait sans doute mieux de tenter de converger.

Oui, même si c'est aussi moi côté adaptation hobo pour Python 3, j'ai hier préféré dupliquer pleinement la classe, plutôt que mêler py2/3 dans les méthodes. (sans doute pour être sûr qu'il faudrait ajouter les nouvelles méthodes Python 3 côté Hobo).

Je suis comme Thomas je regarde /usr/lib/python3.5/threading.py et je ne vois pas ce code, on pourrait aussi revoir la façon de faire (en monkeypatchant juste Thread.start() / Thread.run() plutôt que de remplacer la classe).

Juste une proposition.

À noter que je ne trouve qu'un usage de threading.Thread dans wcs.middleware on pourrait aussi le remplacer directement par TenantAwareThread.... remarque valant aussi pour les autres briques mais il faudra faire d'hobo une dépendance explicite de toutes les briques alors que pour l'instant c'est transparent.

/usr/lib/python3.5/threading.py

Il est dans 3.7.

Et je préférerais limiter les changements au code Python 2; il sera temps, lorsqu'on nettoiera pour en supprimer la prise en charge, de passer sur tout ça.