Ciao Mara certo proporre un possibile miglioramento a un programma non vuole certo essere un'accusa ne vuole svilire il lavoro fatto da altri, ho semplicemente fatto osservare che eseguendo lo script add-key viene reimpostata la fiducia delle chiavi importate al valore di default all'interno del keyring principale di sistema; il tutto a causa di un bug (svista, workflow differente da come lo si usa in #debian-it?, chiamalo come vuoi) che la patch proposta tentava di risolvere (non riuscendoci fra l'altro per un'ulteriore svista,vedi dopo [1] :) .
Non capisco il tono polemico a seguito di queste semplici osservazioni... Si poteva fare in altro modo? Sicuro! Supportava già l'inserimento di uid multiple? Si,forse, mbho, invece di provare ho riscritto un ciclo e aggiunto una riga di commento... Il programma è in 'permanent beta' ? Ripeto proporre una patch non è una critica ma uno spunto per un possibile miglioramento (che può essere ignorato) Questa volta specifica veramente il file su cui scrivere diff --git a/add-key b/add-key index e095092..a60d190 100755 --- a/add-key +++ b/add-key @@ -11,7 +11,7 @@ while [ ! -z $1 ]; do --keyserver pgp.mit.edu \ --keyserver wwwkeys.eu.pgp.net \ --recv-keys $1 - gpg --no-default-keyring --export $1 | \ + gpg --no-default-keyring --keyring ./$_KEYRING --export $1 | \ gpg --no-default-keyring --keyring ./$_KEYRING --import #./party-table.pl ./$_KEYRING > ./$_OUTFILE shift diff --git a/create-graph b/create-graph index e763197..c48e075 100755 --- a/create-graph +++ b/create-graph @@ -10,6 +10,6 @@ _TMPPS=$(basename $_TMPGRAPH .dot).ps gpg --no-default-keyring --keyring ./$_KEYRING --list-sigs | \ sig2dot -a > $_TMPGRAPH neato -Tps $_TMPGRAPH > $_TMPPS -convert $_TMPPS debian-it-wot.png +convert $_TMPPS `basename $_KEYRING.png` rm -rf $_TMPGRAPH $_TMPPS Il 13 novembre 2009 16.50, Mara Sorella <marus...@maruscia.it> ha scritto: > Mara Sorella ha scritto: > > [CUT] > > Peraltro la specifica statica del nome del keyring al momento è una > scelta, nel senso che map essenzialmente lo usiamo solo per debian-it, > (se vedi non c'è nè un README nè nulla, è implicito che chi vuole usarlo > dia un'occhiata a cosa fanno quegli script). > A voler fare le cose per bene si dovrebbe usare getopt, non applicare > quella patch. > > In più basta considerare che il keyring (e il suo nome) è una struttura > dati per il programma che non è utilizzata di per sè se non come > passaggio intermedio.. > > L'unico caso in cui lo potresti dover specificare direttamente è se vuoi > lanciare create-graph a mano solo alla fine (per non doverlo fare ad > ogni key addata, anche se è il comportamento di default perchè in > genere uno può avere interesse nel vedere il wot formarsi..) , in quel > caso lo commenti in add-key in fondo e lo lanci a parte.. come ho fatto > io ieri > > Ad ogni modo mi pareva che la natura 'permanent beta' del programma si > intuisse.. > > > > Mara > > > >>> >>> patch proposta in allegato :D >>> >>> Con questa patch potete fare >>> ./add-key ninux.gpg BCEB9E24 00685483 3EDBF9E1 39D5BCEF 019788AC >>> 0DFD7CBB 68729FAF 1458F41C 39D5BCEF C4125E74 >>> (non le ho messe tutte le chiavi, Mara postaci il file di ieri sera :D) >>> >>> e ./create-graph ninux.gpg >>> >>> Il risultato è in allegato >>> >>> >> Claudio si poteva già fare ./add-key "primachiave secondachiave >> terzachiave" .... non capisco qual'è il tuo problema honestly.. >> >> >>> ------------------------------------------------------------------------ >>> >>> ------------------------------------------------------------------------ >>> >>> _______________________________________________ >>> Wireless mailing list >>> Wireless@ml.ninux.org >>> http://ml.ninux.org/mailman/listinfo/wireless >>> >> >> >> > > > -- > > pub 1024D/39D5BCEF BB88 AEBB 4B8C F220 6F3D 04BE 79F5 39D5 BCEF > > _______________________________________________ > Wireless mailing list > Wireless@ml.ninux.org > http://ml.ninux.org/mailman/listinfo/wireless > _______________________________________________ Wireless mailing list Wireless@ml.ninux.org http://ml.ninux.org/mailman/listinfo/wireless