Controllare le guardie su una patch

# HG changeset patch # User Giulio@puck # Date 1250352230 -7200 # Node ID 99c201b3049161a6b19a95ce6892afea7d19a9fb # Parent 15eeeef467d2c0ed65ad608f6fd9a66ea1abc863 Minor changes and translation of code snippets for Ch.13. diff -r 15eeeef467d2 -r 99c201b30491 it/ch13-mq-collab.xml --- a/it/ch13-mq-collab.xml Sat Aug 15 17:22:37 2009 +0200 +++ b/it/ch13-mq-collab.xml Sat Aug 15 18:03:50 2009 +0200 @@ -4,7 +4,7 @@ Sebbene sia facile imparare gli usi più semplici di Mercurial Queues, sono un pizzico di disciplina e alcune delle funzioni meno usate di MQ che rendono possibile lavorare in ambienti di sviluppo complicati. - In questo capitolo, userò come esempio una tecnica che ho impiegato per gestire lo sviluppo di un driver del kernel di Linux per un dispositivo Infiniband. Il driver in questione è grande (almeno per le dimensioni dei driver), poiché contiene 25.000 righe di codice distribuite su 35 file sorgente. Viene mantenuto da un piccolo gruppo di sviluppatori. + In questo capitolo, userò come esempio una tecnica che ho impiegato per gestire lo sviluppo di un driver del kernel di Linux per un dispositivo Infiniband. Il driver in questione è grande (almeno per le dimensioni dei driver), poiché contiene 25.000 righe di codice distribuite su 35 file sorgente, e viene mantenuto da un piccolo gruppo di sviluppatori. Sebbene la maggior parte del materiale in questo capitolo sia specifica per Linux, gli stessi principi si applicano per qualsiasi base di codice di cui non siate i proprietari principali e su cui dobbiate fare parecchio lavoro. @@ -17,7 +17,7 @@ Un obiettivo è l'albero di sviluppo principale del kernel di Linux. In questo caso, le attività di manutenzione del codice sono parzialmente condivise con altri sviluppatori della comunità del kernel, che effettuano modifiche estemporanee al driver man mano che sviluppano e rifiniscono i sottosistemi del kernel. - Manteniamo anche un certo numero di backport (letteralmente, conversioni all'indietro) verso vecchie versioni del kernel di Linux, per supportare le necessità di clienti che stanno utilizzando distribuzioni di Linux più vecchie che non incorporano i nostri driver. (Effetuare il backport del codice significa modificarlo per farlo funzionare in una versione del suo ambiente di destinazione più vecchia di quella per la quale era stato sviluppato.) + Manteniamo anche un certo numero di backport (letteralmente, conversioni all'indietro) verso vecchie versioni del kernel di Linux, per soddisfare le necessità di clienti che stanno utilizzando distribuzioni di Linux più vecchie che non incorporano i nostri driver. (Effetuare il backport del codice significa modificarlo per farlo funzionare in una versione del suo ambiente di destinazione più vecchia di quella per la quale era stato sviluppato.) Infine, rilasciamo il software seguendo una tabella di marcia che non è necessariamente allineata con quella usata da chi sviluppa il kernel e da chi distribuisce il sistema operativo, in modo da poter consegnare nuove funzionalità ai clienti senza obbligarli ad aggiornare il loro kernel o la loro intera distribuzione. @@ -32,7 +32,7 @@ Il secondo è quello di mantenere un singolo albero di sorgenti pieno di istruzioni condizionali che attivano o disattivano parti di codice a seconda dell'ambiente designato. Dato che queste istruzioni ifdef non sono permesse nell'albero del kernel di Linux, è necessario seguire una procedura manuale o automatica per eliminarle e produrre un albero pulito. Una base di codice mantenuta in questo modo diventa rapidamente un caos di blocchi condizionali che sono difficili da capire e mantenere. - Nessuno di questi approcci è particolarmente adatto a situazioni in cui non possedete la copia ufficiale di un albero di sorgenti. Nel caso di un driver Linux che viene distribuito insieme al kernel standard, l'albero di Linus contiene la copia del codice che verrà considerata ufficiale dal resto del mondo. La versione a monte del mio driver può essere modificata da persone che non conosco, senza che io nemmeno lo scopra fino a quando i cambiamenti non appaiono nell'albero di Linus. + Nessuno di questi approcci è particolarmente adatto a situazioni in cui non possedete la copia ufficiale di un albero di sorgenti. Nel caso di un driver Linux che viene distribuito insieme al kernel standard, l'albero di Linus contiene la copia del codice che verrà universalmente considerata ufficiale. La versione a monte del mio driver può essere modificata da persone che non conosco, senza che io nemmeno lo scopra fino a quando i cambiamenti non appaiono nell'albero di Linus. Questi approcci hanno la debolezza aggiuntiva di rendere difficile generare patch ben formate da presentare a monte. @@ -49,7 +49,7 @@ Questo ci fornisce un piccolo repository contenente due patch che non hanno alcuna dipendenza reciproca, perché coinvolgono file differenti. - L'idea alla base dell'applicazione condizionale è la possibilità di etichettare una patch con una guardia, che è semplicemente una stringa di testo di vostra scelta, per poi dire a MQ di selezionare le guardie specifiche da usare al momento di applicare le patch. MQ applicherà, o salterà, una patch con guardia, a seconda delle guardie che avete selezionato. + L'idea alla base dell'applicazione condizionale è la possibilità di etichettare una patch con una guardia, che è semplicemente una stringa di testo di vostra scelta, per poi dire a MQ di selezionare le guardie specifiche da usare al momento di applicare le patch. MQ applicherà, o salterà, una patch con guardia a seconda delle guardie che avete selezionato. Una patch può avere un numero arbitrario di guardie, ognuna delle quali è positiva (applica questa patch se questa guardia è selezionata) o negativa (salta questa patch se questa guardia è selezionata). Una patch senza alcuna guardia viene sempre applicata. @@ -57,7 +57,7 @@ Controllare le guardie su una patch - Il comando qguard vi permette di determinare quali guardie dovrebbero applicarsi a una patch, o di visualizzare le guardie già in effetto. Senza alcun argomento, il comando mostra le guardie della patch attualmente in cima alla pila. + Il comando qguard vi permette di determinare quali guardie dovrebbero applicarsi a una patch, o di visualizzare le guardie già attive. Senza alcun argomento, il comando mostra le guardie della patch attualmente in cima alla pila. &interaction.mq.guards.qguard; @@ -87,7 +87,7 @@ Il comando qselect determina quali guardie sono attive in un dato momento. Il suo effetto è quello di determinare quali patch verranno applicate da MQ la prossima volta che invocherete qpush. Non ha alcun altro effetto, in particolare non agisce in alcun modo sulle patch che sono già applicate. - Invocato senza argomenti, il comando qselect elenca le guardie attualmente in effetto, una per ogni riga. Gli argomenti vengono trattati come i nomi delle guardie da applicare. + Invocato senza argomenti, il comando qselect elenca le guardie attualmente attive, una per ogni riga. Gli argomenti vengono trattati come i nomi delle guardie da applicare. &interaction.mq.guards.qselect.foo; @@ -119,11 +119,11 @@ Una patch che non ha guardie viene sempre applicata. - Se la patch ha una qualsiasi guardia negativa che corrisponde a una qualsiasi guardia correntemente selezionata, la patch viene saltata. - - Se la patch ha una qualsiasi guardia positiva che corrisponde a una qualsiasi guardia correntemente selezionata, la patch viene applicata. - - Se la patch ha guardie positive o negative, ma nessuna corrisponde a qualsiasi guardia correntemente selezionata, la patch viene saltata. + Se la patch ha una guardia negativa che corrisponde a una guardia attualmente selezionata, la patch viene saltata. + + Se la patch ha una guardia positiva che corrisponde a una guardia attualmente selezionata, la patch viene applicata. + + Se la patch ha guardie positive o negative, ma nessuna corrisponde a una guardia attualmente selezionata, la patch viene saltata. @@ -131,7 +131,7 @@ Ridimensionare l'ambiente di lavoro - Lavorando sul driver di dispositivo menzionato in precedenza, non applico le patch a un normale albero del kernel di Linux, ma uso un repository che contiene solo una fotografia dei file sorgente rilevanti per lo sviluppo del dispositivo Infiniband. È più facile lavorare con questo repository, perché le sue dimensioni sono l'1% delle dimensioni di un repository del kernel. + Lavorando sul driver di dispositivo menzionato in precedenza, non applico le patch a un normale albero del kernel di Linux, ma uso un repository che contiene solo una fotografia dei file sorgente rilevanti per lo sviluppo del dispositivo Infiniband. Lavorare con questo repository è più facile, perché le sue dimensioni sono l'1% delle dimensioni di un repository del kernel. Poi scelgo una versione di base sulla quale le patch vengono applicate. Questa è una fotografia dell'albero del kernel di Linux scattata su una revisione di mia scelta. Quando scatto la fotografia, registro l'identificatore di changeset proveniente dal repository del kernel nel messaggio di commit. Dato che la fotografia mantiene la forma e il contenuto delle parti rilevanti dell'albero del kernel, posso applicare le mie patch sul mio repository ridotto o su un normale albero del kernel. @@ -145,25 +145,25 @@ La sequenza dei gruppi di patch che mantengo è la seguente. L'ordine di questi gruppi è importante per i motivi che verranno descritti dopo aver introdotto i gruppi. - Il gruppo delle patch accettate. Sono le patch che il gruppo di sviluppo ha proposto al mantenitore del sottosistema Infiniband e che sono state accettate, ma che non sono presenti nella fotografia su cui si basa il repository ridotto. Queste sono patch a sola lettura, presenti unicamente allo scopo di trasformare l'albero in uno stato simile a quello in cui si trova nel repository del mantenitore a monte. - - Il gruppo delle patch da rielaborare. Sono le patch che ho proposto ma per le quali il mantenitore a monte ha richiesto alcune modifiche prima di poterle accettare. - - Il gruppo delle patch in sospeso. Sono le patch che non ho ancora proposto al mantenitore a monte, ma su cui dobbiamo finire di lavorare. Queste patch saranno a sola lettura per un po'. Se il mantenitore a monte le accetta al momento di proporle, le sposterò alla fine del gruppo accettate. Se ne richiede la modifica, le sposterò all'inizio del gruppo da rielaborare. + Il gruppo delle patch accettate. Sono le patch che il gruppo di sviluppo ha proposto al manutentore del sottosistema Infiniband e che sono state accettate, ma che non sono presenti nella fotografia su cui si basa il repository ridotto. Queste sono patch a sola lettura, presenti unicamente allo scopo di trasformare l'albero in uno stato simile a quello in cui si trova nel repository del manutentore a monte. + + Il gruppo delle patch da rielaborare. Sono le patch che ho proposto ma per le quali il manutentore a monte ha richiesto alcune modifiche prima di poterle accettare. + + Il gruppo delle patch in sospeso. Sono le patch che non ho ancora proposto al manutentore a monte, ma su cui dobbiamo finire di lavorare. Queste patch saranno a sola lettura per un po'. Se il manutentore a monte le accetta al momento di proporle, le sposterò alla fine del gruppo accettate. Se ne richiede la modifica, le sposterò all'inizio del gruppo da rielaborare. Il gruppo delle patch in corso. Sono le patch che vengono attivamente sviluppate e che non dovrebbero essere ancora presentate a nessuno. Il gruppo delle patch di backport. Sono le patch che riadattano l'albero dei sorgenti a una vecchia versione dell'albero del kernel. - Il gruppo delle patch da non rilasciare. Sono le patch che per qualche ragione non dovrebbero mai essere presentate a monte. Per esempio, una patch di questo tipo potrebbe modificare le stringhe di identificazione incluse nei driver per rendere più facile distinguere, nel testo del campo, tra una versione del driver presa dall'albero e una versione consegnata a un rivenditore di distribuzioni. - - - - Tornando alle ragioni per cui i gruppi di patch sono ordinati in questo modo, ci piacerebbe che le patch più in basso nella pila siano il più possibile stabili, in modo da non aver bisogno di rielaborare le patch più in alto a causa di modifiche a loro contesto. Mettere le patch che non verranno mai cambiate all'inizio del file series serve proprio a questo scopo. + Il gruppo delle patch da non rilasciare. Sono le patch che per qualche ragione non dovrebbero mai essere presentate a monte. Per esempio, una patch di questo tipo potrebbe modificare le stringhe di identificazione incluse nei driver per rendere più facile distinguere, nel testo del campo, una versione del driver presa dall'albero da una versione consegnata a un rivenditore di distribuzioni. + + + + Tornando alle ragioni per cui i gruppi di patch sono ordinati in questo modo, ci piacerebbe che le patch più in basso nella pila siano il più possibile stabili, in modo da non aver bisogno di rielaborare le patch più in alto a causa di modifiche al loro contesto. Mettere le patch che non verranno mai cambiate all'inizio del file series serve proprio a questo scopo. Ci piacerebbe anche applicare le patch che sappiamo di aver bisogno di modificare su un albero di sorgenti che somiglia il più possibile all'albero a monte. Questo è il motivo per cui teniamo le patch accettate in giro per un po'. - Le patch di backport e da non rilasciare si trovano alla fine del file series. Le patch di backport devono essere applicate su tutte le altre patch, e le patch da non rilasciare potrebbero anche rimanere in un posto sicuro. + Le patch di backport e da non rilasciare si trovano alla fine del file series. Le patch di backport devono essere applicate su tutte le altre patch, e anche le patch da non rilasciare dovrebbero essere messe in un posto sicuro. @@ -172,13 +172,13 @@ Nel mio lavoro, uso un certo numero di guardie per controllare quali patch devono essere applicate. - Le patch accettate sono sorvegliate da accepted. Questa guardia è abilitata per la maggior parte del tempo. Quando sto applicando le patch su un albero dove le patch sono già presenti, posso disabilitare questa patch così le patch che la seguono verranno applicate in maniera pulita. + Le patch accettate sono sorvegliate da accettate. Questa guardia è abilitata per la maggior parte del tempo. Quando sto applicando le patch su un albero dove le patch sono già presenti, posso disabilitare questa guardia così le patch che la seguono verranno applicate in maniera pulita. Le patch che sono terminate, ma non sono ancora state proposte, non hanno guardie. Se sto applicando la pila delle patch a una copia dell'albero a monte, non ho bisogno di abilitare alcuna guardia per ottenere un albero di sorgenti ragionevolmente sicuro. - Le patch che hanno bisogno di essere rielaborate prima di venire nuovamente presentate sono sorvegliate da rework. - - Per quelle patch che sono ancora in lavorazione, uso devel. + Le patch che hanno bisogno di essere rielaborate prima di venire nuovamente presentate sono sorvegliate da rielaborare. + + Per quelle patch che sono ancora in lavorazione, uso sviluppo. Una patch di backport potrebbe avere diverse guardie, una per ogni versione del kernel a cui si applica. Per esempio, una patch che effettua il backport di una parte del codice alla versione 2.6.9 del kernel avrà una guardia 2.6.9. @@ -190,9 +190,9 @@ Usando MQ, scrivere una patch di backport è un processo semplice. Tutto quello che una patch di questo tipo deve fare è modificare una parte di codice che usa una funzione del kernel non presente in una vecchia versione del kernel, in modo che il driver continui a funzionare correttamente con la vecchia versione. - Un obiettivo utile da raggiungere nella scrittura di una buona patch di backport è far sembrare che il codice sia stato scritto per la vecchia versione del kernel che state considerando. Meno intrusiva è la patch, più facile sarà da capire e mantenere. Se state scrivendo una collezione di patch di backport per evitare l'effetto caos causato da molte istruzioni #ifdef (contenenti blocchi di codice che vengono usati solo in maniera condizionata) nel vostro codice, evitate di introdurre nelle patch #ifdef dipendenti dalle versioni del kernel. Piuttosto, scrivete diverse patch, ognuna delle quali provoca cambiamenti incondizionati, e controllate la loro applicazione usando le guardie. - - Ci sono due ragioni per separare le patch di backport in un gruppo distinto dalle patch normali i cui effetti vengono modificati da quelle. Il primo è che mescolarle insieme rende più difficile usare uno strumento come l'estensione patchbomb per automatizzare il processo di spedizione delle patch a un mantenitore a monte. Il secondo è che le patch di backport potrebbero perturbare il contesto in cui una successiva patch normale viene applicata, rendendo impossibile applicare la patch normale in maniera pulita senza che la patch di backport precedente sia già stata applicata. + Un obiettivo utile da raggiungere nella scrittura di una buona patch di backport è far sembrare che il codice sia stato scritto per la vecchia versione del kernel che state considerando. Meno intrusiva è la patch, più facile sarà da capire e mantenere. Se state scrivendo una collezione di patch di backport per evitare l'effetto caos causato da molte istruzioni #ifdef (contenenti blocchi di codice che vengono usati solo in maniera condizionata) nel vostro codice, evitate di introdurre #ifdef dipendenti dalle versioni del kernel nelle patch. Piuttosto, scrivete diverse patch, ognuna delle quali provoca cambiamenti incondizionati, e controllate la loro applicazione usando le guardie. + + Ci sono due ragioni per separare le patch di backport in un gruppo distinto dalle patch normali i cui effetti vengono modificati da quelle. Il primo è che mescolarle insieme rende più difficile usare uno strumento come l'estensione patchbomb per automatizzare il processo di spedizione delle patch a un manutentore a monte. Il secondo è che le patch di backport potrebbero perturbare il contesto in cui una successiva patch normale viene applicata, rendendo impossibile applicare la patch normale in maniera pulita senza che la patch di backport precedente sia già stata applicata. @@ -225,7 +225,7 @@ [extdiff] cmd.interdiff = hg-interdiff Questa riga ordina a hgext di rendere disponibile il comando interdiff, in modo che possiate ridurre l'invocazione precedente di extdiff a qualcosa di un po' più maneggevole. - hg interdiff -r A:B my-change.patch + hg interdiff -r A:B mio-cambiamento.patch Il comando interdiff funziona bene solo se i file sottostanti dai quali vengono generate le versioni di una patch rimangono gli stessi. Se create una patch, modificate i file sottostanti e poi rigenerate la patch, interdiff potrebbe non produrre alcun risultato utile. diff -r 15eeeef467d2 -r 99c201b30491 it/examples/auto-snippets.xml --- a/it/examples/auto-snippets.xml Sat Aug 15 17:22:37 2009 +0200 +++ b/it/examples/auto-snippets.xml Sat Aug 15 18:03:50 2009 +0200 @@ -156,17 +156,17 @@ - - - - - - - - - - - + + + + + + + + + + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.init.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.init.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,12 @@ + +$ hg qinit +$ hg qnew ciao.patch +$ echo ciao > ciao +$ hg add ciao +$ hg qrefresh +$ hg qnew arrivederci.patch +$ echo arrivederci > arrivederci +$ hg add arrivederci +$ hg qrefresh + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qguard.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qguard.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,5 @@ + +$ hg qguard +arrivederci.patch: senza guardie + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qguard.neg.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qguard.neg.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,6 @@ + +$ hg qguard -- ciao.patch -quux +$ hg qguard ciao.patch +ciao.patch: -quux + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qguard.pos.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qguard.pos.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,6 @@ + +$ hg qguard +foo +$ hg qguard +arrivederci.patch: +foo + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qselect.cat.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qselect.cat.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,5 @@ + +$ cat .hg/patches/guards +foo + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qselect.error.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qselect.error.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,5 @@ + +$ hg qselect +foo +fallimento: la guardia '+foo' comincia con un carattere non valido: '+' + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qselect.foo.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qselect.foo.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,11 @@ + +$ hg qpop -a +la coda delle patch è vuota +$ hg qselect +nessuna guardia attiva +$ hg qselect foo +il numero di patch non applicate senza guardia è cambiato da 1 a 2 +$ hg qselect +foo + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qselect.foobar.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qselect.foobar.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,11 @@ + +$ hg qselect foo bar +il numero di patch non applicate senza guardia è cambiato da 0 a 2 +$ hg qpop -a +nessuna patch applicata +$ hg qpush -a +applico ciao.patch +applico arrivederci.patch +ora a: arrivederci.patch + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qselect.qpush.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qselect.qpush.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,7 @@ + +$ hg qpush -a +applico ciao.patch +applico arrivederci.patch +ora a: arrivederci.patch + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.qselect.quux.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.qselect.quux.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,9 @@ + +$ hg qselect quux +il numero di patch applicate con guardia è cambiato da 0 a 2 +$ hg qpop -a +la coda delle patch è vuota +$ hg qpush -a +la serie di patch è già stata completamente applicata + + diff -r 15eeeef467d2 -r 99c201b30491 it/examples/mq.guards.series.it --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/it/examples/mq.guards.series.it Sat Aug 15 18:03:50 2009 +0200 @@ -0,0 +1,6 @@ + +$ cat .hg/patches/series +ciao.patch #-quux +arrivederci.patch #+foo + +