Project

General

Profile

Anomalie #4121

Erreur de patch si présence de commentaires

Added by Ludovic Landucci over 7 years ago. Updated about 7 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Category:
-
Start date:
09/21/2012
Due date:
% Done:

100%

Estimated time:
0.50 h
Spent time:
Distribution:
EOLE 2.3

Description

Bonjour,

Sur Amon 2.3.5 en maj stable, un patch du fichier common-squid1.conf renvoi une erreur lorsque ce patch contient des commentaires.

En supprimant tous les commentaires du patch, je n'ai plus d'erreurs.

Merci,

Ludo.


Related issues

Related to creole - Anomalie #4414: testpatches.py : lignes avec commentaires Fermé 11/13/2012
Related to Amon - Anomalie #4478: oubli de templates dans la liste no_comment_tmpl Fermé 11/23/2012
Follows zephir-client - Anomalie #2072: patch avec commentaires en erreur sur Amon 2.3 Fermé 09/19/2011
Follows eole-proxy - Evolution #2952: regrouper squid.conf et squid2.conf et faire des includes sur les parties différentes Fermé 02/23/2012

Associated revisions

Revision b48e539d (diff)
Added by Joël Cuissinat over 7 years ago

L'agent patch ignore les lignes débutant par un commentaire

  • agents/patches.py : modification et nettoyage du code

Fixes #4121 @30m

Revision d2c4b300 (diff)
Added by Fabrice Barconnière about 7 years ago

zephir/monitor/agents/patches.py : teste si le patch ajoute une ligne
vide
fixes #4121 @5m
fixes #4414 @5m

Revision d075164e (diff)
Added by moyooo about 7 years ago

Targets deletion trouble see #4121

History

#1 Updated by Joël Cuissinat over 7 years ago

  • Project changed from Amon to creole
  • Status changed from Nouveau to Accepté
  • Assigned To set to Joël Cuissinat
  • Target version set to Mises à jour 2.3.7 RC
  • Estimated time set to 0.50 h

Le test des patch devrait ignorer les lignes débutant par '#' et ';'.

Cette astuce, sans être la solution absolue, me semble largement suffisante.

#2 Updated by Joël Cuissinat over 7 years ago

  • Project changed from creole to zephir-client

#3 Updated by Joël Cuissinat over 7 years ago

  • Status changed from Accepté to Résolu
  • % Done changed from 0 to 100

#5 Updated by Fabrice Barconnière about 7 years ago

  • Status changed from Résolu to Fermé

#6 Updated by Fabrice Barconnière about 7 years ago

  • Status changed from Fermé to Réouvert

Traceback : string index out of range

#7 Updated by Fabrice Barconnière about 7 years ago

  • Status changed from Réouvert to Résolu

#8 Updated by Fabrice Barconnière about 7 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF