Projet

Général

Profil

Development #13259

fournir un paquet permettant à une autre application de connaître la liste des applications gérées

Ajouté par Frédéric Péters il y a plus de 7 ans. Mis à jour il y a environ 5 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
Josué Kouka
Catégorie:
-
Version cible:
-
Début:
21 septembre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Lié à Hobo - Bug #13258: Ajouter Teamnet dans la liste mandayejs_app_settingsRejeté21 septembre 2016

Actions

Révisions associées

Révision c728d430 (diff)
Ajouté par Josué Kouka il y a environ 6 ans

add mandayejs common package (#13259)

Historique

#1

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
#2

Mis à jour par Serghei Mihai il y a plus de 6 ans

Ça devient important d'avoir ça.
Car lors du deploiement d'un mandaye pour Teamnet par exemple, l'application n'apparait pas.

#3

Mis à jour par Josué Kouka il y a plus de 6 ans

  • Lié à Bug #13258: Ajouter Teamnet dans la liste mandayejs_app_settings ajouté
#4

Mis à jour par Josué Kouka il y a plus de 6 ans

Au lieu de fournir un paquet séparé, l'idée ici est que hobo récupère cette liste depuis python-mandayejs.

#5

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é.

#6

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.

#7

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.

#8

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 ?

#9

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).

#10

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.

#11

Mis à jour par Josué Kouka il y a plus de 6 ans

Ok, le patch creant le paquet mandayejs-common.

#12

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 ?

#13

Mis à jour par Josué Kouka il y a plus de 6 ans

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.

#14

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).

#15

Mis à jour par Josué Kouka il y a plus de 6 ans

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.

#16

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.

#17

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.

Formats disponibles : Atom PDF