Development #13259
fournir un paquet permettant à une autre application de connaître la liste des applications gérées
0%
Description
Pour le moment on se trouve à devoir modifier hobo (ex: #13258) quand la prise en charge d'une application est ajoutée à Mandaye.
Ça serait bien si cette info se trouvait dans, mettons, un paquet mandayejs-common, qui pourrait se trouver installé sur la machine hobo.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Josué Kouka il y a plus de 7 ans
- Tracker changé de Bug à Development
- Statut changé de Nouveau à En cours
- Assigné à mis à Josué Kouka
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Fichier mandaye_apps.png mandaye_apps.png ajouté
Ça devient important d'avoir ça.
Car lors du deploiement d'un mandaye pour Teamnet par exemple, l'application n'apparait pas.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Lié à Bug #13258: Ajouter Teamnet dans la liste mandayejs_app_settings ajouté
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-add-function-listing-configured-apps-13259.patch 0001-add-function-listing-configured-apps-13259.patch ajouté
- Patch proposed changé de Non à Oui
Au lieu de fournir un paquet séparé, l'idée ici est que hobo récupère cette liste depuis python-mandayejs
.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Il faudrait nettoyer les dépendances inutiles, pas envie de les trainer vers hobo à cause de négligences dans mandaye.
Concernant le patch, même les vieux n'utilisent plus map. -10 pour la lisibilité. Et name c'est le nom de la classe c'est pas terrible, ça va foirer dès qu'on voudra présenter un nom d'application avec un espace ou un accent, il faut taper un attribut dédié.
Mis à jour par Josué Kouka il y a plus de 6 ans
Frédéric Péters a écrit :
Il faudrait nettoyer les dépendances inutiles, pas envie de les trainer vers hobo à cause de négligences dans mandaye.
Concernant le patch, même les vieux n'utilisent plus map. -10 pour la lisibilité. Et name c'est le nom de la classe c'est pas terrible, ça va foirer dès qu'on voudra présenter un nom d'application avec un espace ou un accent, il faut taper un attribut dédié.
Et name c'est le nom de la classe c'est pas terrible, ça va foirer dès qu'on voudra présenter un nom d'application avec un espace ou un accent, il faut taper un attribut dédié.
Ok. Il y'a déja un ticket sur cette partie (#19649).
Il faudrait nettoyer les dépendances inutiles, pas envie de les trainer vers hobo à cause de négligences dans mandaye.
Ok, ça me fait penser à rejeter ce ticket, de toute façon on ajoute pas de nouvelles app sur mandaye tout le temps.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Ok, ça me fait penser à rejeter ce ticket, de toute façon on ajoute pas de nouvelles app sur mandaye tout le temps.
Il n'y aura pas de commit dans Hobo pour ajouter une à une les applications Mandaye; ça relève de Mandaye. C'est ce ticket.
Mis à jour par Josué Kouka il y a plus de 6 ans
Frédéric Péters a écrit :
Ok, ça me fait penser à rejeter ce ticket, de toute façon on ajoute pas de nouvelles app sur mandaye tout le temps.
Il n'y aura pas de commit dans Hobo pour ajouter une à une les applications Mandaye; ça relève de Mandaye. C'est ce ticket.
Ok. Mais concernant
Il faudrait nettoyer les dépendances inutiles, pas envie de les trainer vers hobo à cause de négligences dans mandaye.
On reste bien sur un paquet fourni contenant la liste des applications, paquet qui sera listé dans les recommends
de hobo, non ?
Mis à jour par Frédéric Péters il y a plus de 6 ans
Il faudrait nettoyer les dépendances inutiles, pas envie de les trainer vers hobo à cause de négligences dans mandaye.
Par là je veux dire qu'utiliser python-mandayejs d'accord mais que ses dépendances déclarées mais irréelles devaient d'abord être nettoyées (python-ldap). (et cela en considérant que d'autres points ne seraient pas corrigés, genre phantomjs dont la dépendance devrait théoriquement être sur python-mandayejs, mais que je ne voudrais pas emmener dans hobo, et c'est pour ça qu'il y avait idée d'un paquet -common).
Mis à jour par Josué Kouka il y a plus de 6 ans
Frédéric Péters a écrit :
Il faudrait nettoyer les dépendances inutiles, pas envie de les trainer vers hobo à cause de négligences dans mandaye.
Par là je veux dire qu'utiliser python-mandayejs d'accord mais que ses dépendances déclarées mais irréelles devaient d'abord être nettoyées (python-ldap). (et cela en considérant que d'autres points ne seraient pas corrigés, genre phantomjs dont la dépendance devrait théoriquement être sur python-mandayejs, mais que je ne voudrais pas emmener dans hobo, et c'est pour ça qu'il y avait idée d'un paquet -common).
Ok, merci je comprends mieux. Bon je pars sur le paquet mandayejs-common
.
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-add-mandayejs-common-package-13259.patch 0001-add-mandayejs-common-package-13259.patch ajouté
Ok, le patch creant le paquet mandayejs-common
.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Il y a un quart d'explication concernant l'étrange particularité de DEFAULT_APP_SETTINGS ?
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-add-mandayejs-common-package-13259.patch 0001-add-mandayejs-common-package-13259.patch ajouté
Frédéric Péters a écrit :
Il y a un quart d'explication concernant l'étrange particularité de DEFAULT_APP_SETTINGS ?
C'était juste pour définir un default
à l'attribut site_app
de hobo.environment.models.MandayeJS
. Je l'ai enlevé, je vais le définir en dur.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Ça ne va vraiment pas aider le développement local d'avoir ce bout présent et prétendument spécifique à Debian dans debian/; ça devrait simplement aller dans mandayejs/common.py (genre).
Et donc pour faire le boulot, debian/python-mandayejs-common.install deviendrait :
usr/lib/python2*/*-packages/mandayejs/__init__.py usr/lib/python2*/*-packages/mandayejs/common.py
Et soit on veut permettre l'installation des deux paquets, ce qui va bien avec le nom, et alors il faut ajouter la dépendance et retirer les fichiers en double du paquet python-mandayejs, par exemple dans le debian/rules :
+ +override_dh_install: + dh_install + # don't ship thos files in python-mandayejs, they are also in + # python-mandayejs-common + rm debian/python-mandayejs/usr/lib/python2.*/dist-packages/mandayejs/__init__.py + rm debian/python-mandayejs/usr/lib/python2.*/dist-packages/mandayejs/common.py
Soit on ne veut pas permettre l'installation parallèle, et alors il faut déclarer le conflit, et sans doute trouver un nom plus approprié, genre python-mandayejs-known-applications (Mandayejs - list of known apps for external modules).
Mis à jour par Josué Kouka il y a plus de 6 ans
- Fichier 0001-add-mandayejs-common-package-13259.patch 0001-add-mandayejs-common-package-13259.patch ajouté
Frédéric Péters a écrit :
Ça ne va vraiment pas aider le développement local d'avoir ce bout présent et prétendument spécifique à Debian dans debian/; ça devrait simplement aller dans mandayejs/common.py (genre).
Et donc pour faire le boulot, debian/python-mandayejs-common.install deviendrait :
[...]
Et soit on veut permettre l'installation des deux paquets, ce qui va bien avec le nom, et alors il faut ajouter la dépendance et retirer les fichiers en double du paquet python-mandayejs, par exemple dans le debian/rules :
[...]
J'ai opté pour la première solution.
Mis à jour par Frédéric Péters il y a plus de 6 ans
J'ai opté pour la première solution.
Le copié/collé a fonctionné mais il reste des parties de mon commentaire à lire.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de En cours à Rejeté
On oublie l'intégration fine dans hobo.
add mandayejs common package (#13259)