Le lundi 27 septembre 2004 à 23:55 +0200, [EMAIL PROTECTED] a écrit : > Selon [EMAIL PROTECTED]:
> Faut que je regarde le détail avant d'émettre un avis ;-). idem... m'enfin à première vue, je serais plutôt pour faire une version spécifique pour les points qu'on n'arrive pas à rendre génériques... yaurait pas perl sur ta micro-distrib ??? Notre souci (voir l'historique de la ML) c'est que certaines distrib' n'installent pas gawk par défaut... donc la question se pose réellement des outils à retenir. > > 4/ patch.eaglediag > > - simplfication de VERSION_DIAG > > > > J'ai aussi essaye de reduire le nombre de processus lances a l'execution > > en reduisant les pipe genre 'grep | sed | cut' par du awk > > C'est discutable. > La ligne de awk que tu proposes parse tout le fichier > donc l'économie n'est pas évidente. > D'autre part on essaie de limiter les dépendances sur les programmes > qui ne sont pas toujours présents. > > Donc je propose à Baud un sed pour changer ;-) > VERSION_DIAG=`sed -n -r -e \ > 's/^# \$Id: eaglediag,v ([^ ]*) ([^ ]*) .*/\1 \2/p; 2Q' \ > </usr/sbin/eaglediag` > > Une seule commande et on ne parse que deux lignes. > > Ou même (économie de bout de chandelle oblige) : > VERSION_DIAG=`sed -n -r -e \ > '1t; s/^# \$Id: eaglediag,v ([^ ]*) ([^ ]*) .*/\1 \2/p; 2Q' \ > </usr/sbin/eaglediag` > Et on ne parse qu'une seule ligne. eh oh einh faut peut-être pas exagérer non plus avec tes sed, c'est lancé qu'une fois et c'est justement sed qui n'est pas dispo si j'ai bien suivi, pour une fois que ça va dans mon sens avec des awk élégants :-)) il doit y avoir plus simple : c'est quoi le code pour faire que CVS me crache tout crû le n° de version (sccs me le faisait... ce pourquoi j'ai pris l'habitude d'écrire des date +"%Y""%M" pour éviter de me faire bouffer la commande). Si ça peut m'éviter un des horribles sed de mcoolive, je prends ;-) @++ Ben'. aka baud123
