Projet

Général

Profil

Bug #6654

Permettre de définir les chemins --extra dans le fichier wcs.cfg comme --app-dir et --data-dir

Ajouté par Benjamin Dauvergne il y a environ 9 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
09 mars 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Dans wcs.cfg on pourra désormais écrire:

[extra]
auquotidien = /home/bdauvergne/git/auquotidien/extra/

Fichiers

Révisions associées

Révision 82627cdc (diff)
Ajouté par Benjamin Dauvergne il y a environ 9 ans

misc: add support for specifying extra modules in configuration files (#6654)

[extra]
auquotidien = ../auquotidien/extra/

Révision 03d50b9b (diff)
Ajouté par Frédéric Péters il y a environ 9 ans

tests: adapt to changes in declaration of extra modules (follow-up #6654)

Révision 725cbb68 (diff)
Ajouté par Jérôme Schneider il y a environ 9 ans

hobo_deploy: fix support of --extra passed on command line (#6654)

Historique

#1

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

  • Statut changé de Nouveau à En cours

Ack.

#2

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

Unack.

Traceback (most recent call last):
  File "./wcsctl.py", line 8, in <module>
    ctl.run(sys.argv[1:])
  File "/home/fred/src/eo/wcs/wcs/qommon/ctl.py", line 157, in run
    return cmd.run(args, options)
  File "/home/fred/src/eo/wcs/wcs/qommon/ctl.py", line 66, in run
    self.config.set("extra", "cmd_line_extra_%d" % i, extra)
  File "/usr/lib/python2.7/ConfigParser.py", line 396, in set
    raise NoSectionError(section)
ConfigParser.NoSectionError: No section: 'extra'
#3

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

  • Fichier 0001-Allow-an-extra-section-in-w.c.s.-configuration-files.patch supprimé
#4

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

J'ai ajouté les lignes:

+            if not self.config.has_section('extra'):
+                self.config.add_section('extra')

qui manquaient.

#5

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

  • Statut changé de En cours à Résolu (à déployer)

Voilà, poussé en modifiant un peu le message de commit, pour mentionner ce ticket et un exemple d'utilisation.

commit 82627cdc8cf5071052add0edfccd629100c66f7e
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Mar 9 14:58:31 2015 +0100

    misc: add support for specifying extra modules in configuration files (#6654)

    [extra]
    auquotidien = ../auquotidien/extra/
#6

Mis à jour par Jérôme Schneider il y a environ 9 ans

Je viens de trouver un bug lié à ce commit dans hobo_deploy :

Traceback (most recent call last):
  File "/usr/sbin/wcsctl", line 8, in <module>
    ctl.run(sys.argv[1:])
  File "/usr/lib/python2.7/dist-packages/wcs/qommon/ctl.py", line 159, in run
    return cmd.run(args, options)
  File "/usr/lib/python2.7/dist-packages/wcs/qommon/ctl.py", line 69, in run
    return self.execute(base_options, sub_options, args)
  File "/usr/lib/python2.7/dist-packages/wcs/ctl/check_hobos.py", line 40, in execute
    publisher.WcsPublisher.configure(self.config, sub_options.extra)

Ci-joint un patch qui corrige le problème.

#7

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

Ouaip. Poussé avec une correction au message de commit (c'est le support sur la ligne de commande qui a été cassé par l'ajout de la gestion via le fichier de config).

commit 725cbb68451b75cfe40a1386f7921f18c6f3e6d0
Author: Jérôme Schneider <jschneider@entrouvert.com>
Date:   Mon Mar 30 18:07:56 2015 +0200

    hobo_deploy: fix support of --extra passed on command line (#6654)
#8

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

  • Statut changé de En cours à Résolu (à déployer)
#9

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF