Re: [sympa-fr] Dans le code : familles ; alias_manager ; cvs ; soap_server

[Olivier Salaün - CRU <adresse@cachée>]

Bonjour,

Sylvain Amrani wrote:

> Voici quelques remarques liées au code :
>
> * Il n'est pas aisé (possible ?) d'automatiser l'instanciation des familles :
> - Des questions sont posées sur STDIN (fermer les listes qui ne sont plus 
> définies)
> - Les instanciations sont très verbeuses et tout sort sur STDERR, quand bien 
> même il n'y a eu aucune erreur, ce qui handicape pour l'utilisation de cron 
> (je peux fondre STDOUT et STDERR, mais je perds de l'information utile -- 
> distinguo erreur/ok)
>
> Contourner ces difficultés n'est pas immédiat : Je serais tenté :
> - d'ajouter des options à sympa.pl : --close-unknowns pour fermer 
> automatiquement les listes disparues et --quiet pour supprimer les messages 
> produits par une instanciation sur STDOUT ;
>  
Ces 2 options nous paraissent tout à fait justifiées ; nous intégrerons 
votre patch dans la distribution.

> - de faire un distingo STDOUT/STDERR par l'ajout d'une fonction 
> get_instantiation_errors() qui sortirait les erreurs create_hash / 
> create_list / listname_already_used / update_list / previous_list sur STDERR
> - de ne faire sortir que les autres messages sur STDOUT
>  
Il parait difficile de différencier les notifications et les erreurs 
liés à une instanciation car sorties de leur contexte on ne sait plus à 
quelle liste se raporte une erreur.
En revenche il serait plus judicieux d'envoyer le compte-rendu d'une 
instanciation dans STDOUT (au lieu de STDERR)

> Sont-ce des bonnes pistes dans le design général de sympa ? Voulez-vous que 
> je travaille à un patch ?
>  
Tout à fait, sauf peu-être get_instantiation_errors() sur lequel nous 
avons un doute.
Nous ferons de notre mieux pour intégrer les patchs que vous nous 
soumettrez.

> * Il n'est pas possible d'ignorer la gestion des aliases par l'alias_manager, 
> or cela ne concerne pas les sites qui utilisent des regexp pour la gestion 
> des aliases et alourdit les sites avec de nombreuses listes. Ne pourrait-on 
> pas rendre l'utilisation de cet alias_manager paramétrable ? Peut-être que 
> votre philosophie est de ne pas étendre trop la liste des options (déjà 
> importante). Pour l'instant j'ai résolu le problème par l'ajout d'un exit(0) 
> en entête du fichier.
>  
Dans votre cas, le exit(0) me semble tout à fait approprié.
Nous préférons effectivement ne pas alourdir le paramétrage 
dealias_manager.pl, sauf si certains besoins sont d'ordres général.

> * Y-a-t-il une liste de diffusion des cvs-commits ?
>  
Pas pour l'instant, mais si le besoin s'en fait sentir, nous pouvons 
l'envisager...

> * Dans sympa_soap_server.fcgi, le server soap est pointé en dur dans 
> /home/sympa/bin :
>  
Merci, nous avons appliqué votre patch dans les branches stable et dev 
de Sympa.

Cordialement.

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature