<div class="gmail_quote">On Fri, Feb 24, 2012 at 7:21 AM, Paolo Cavallini <span dir="ltr"><<a href="mailto:cavallini@faunalia.it">cavallini@faunalia.it</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Per chiarire: l'inclusione nel codice sorgente non e' mai automatica, e non e' certamente necessario essere sviluppatori di quel particolare pezzo di software per esserne certi.<br>
Le probabilita' comunque, IMHO vanno in modo decrescente con questo ordine approssimativo:<br>
sviluppatore di quel determinato pezzo>sviluppatore di quel progetto>sviluppatore di un altro progetto libero>non contributore di alcun progetto.<br>
E ovviamente la qualita' del codice e la governance del progetto hanno un'importanza molto rilevante.<br></blockquote><div><br></div><div>Concordo sulla gerarchia, in termini di probabilità generale, completamente slegata dal singolo contesto.</div>
<div>Per fare un esempio che mi è familiare, GeoServer, una patch ha elevata probabilità di essere integrata se:</div><div>- è stata discussa con la comunità prima dello svilupppo (per assicurarsi che non confligga con altri</div>
<div>  sforzi e sia in linea con l'architettura del prodotto)</div><div>- è stata sviluppata seguendo le stesse convezioni di codifica del progetto, senza riformattare codice</div><div>  esistente (in modo che le modifiche siano tutte e sole le parti evidenti dal file di patch)</div>
<div>- è dotata di test automatici (junit nel nostro caso) che ne dimostrano il corretto funzionamento</div><div>  (oggi e anche in futuro)</div><div>- se è una nuova funzionalità, è stata anche aggiunta una patch alla documentazione (questo non è </div>
<div>  di fatto richiesto, ma è così bello quando succede!)</div></div><div><br></div><div>Se una patch rispetta le regole di massima esposte sopra entra. Detto questo, non ci sono garanzie sui tempi,</div><div>per fare un esempio un paio di settimane fa è stata proposta una patch relativamente piccola, ma molto</div>
<div>ben fatta, su geoserver-devel: </div><div><a href="http://osgeo-org.1560.n6.nabble.com/Proposal-to-enhance-control-flow-module-td4474108.html">http://osgeo-org.1560.n6.nabble.com/Proposal-to-enhance-control-flow-module-td4474108.html</a></div>
<div><br></div><div>Una prima review a partire dalla presentazione del lavoro (senza guardare la patch)</div><div>ha individuato problemi nel lavoro, che sono stati corretti.</div><div>La patch è poi stata aggiunta qui, ma inizialmente non si applicava a un checkout:</div>
<div><a href="https://jira.codehaus.org/browse/GEOS-4961">https://jira.codehaus.org/browse/GEOS-4961</a></div><div>Visto che la review e il commit di roba non lavorativa ho tempo di farlo solo il fine settimana,</div><div>
la cosa è andata avanti un po' nel tempo, forse domani riuscirò a guardarla e a committarla.</div><div>Da notare che questo è il primo contributo per lo sviluppatore in questione, ma bisogna dire</div><div>che si è presentato nel migliore dei modi.</div>
<div><br></div><div>Non sempre le cose vanno altrettanto bene. Qui c'e' un caso in cui si è andati</div><div>avanti un paio di mesi partendo da una prima patch un po' pasticciata, con</div><div>alcuni bug, e senza test </div>
<div>(e in cui ho dovuto aggiungere del mio per dare una sistemata alla patch):</div><div><a href="https://jira.codehaus.org/browse/GEOS-4927">https://jira.codehaus.org/browse/GEOS-4927</a></div><div><br></div>Ciao<div>Andrea<br clear="all">
<div><br></div>-- <br>-------------------------------------------------------<br>Ing. Andrea Aime<br>GeoSolutions S.A.S.<br>Tech lead<br><br>Via Poggio alle Viti 1187<br>55054  Massarosa (LU)<br>Italy<br><br>phone: +39 0584 962313<br>
fax:      +39 0584 962313<br>mob:    +39 339 8844549<br><br><a href="http://www.geo-solutions.it" target="_blank">http://www.geo-solutions.it</a><br><a href="http://geo-solutions.blogspot.com/" target="_blank">http://geo-solutions.blogspot.com/</a><br>
<a href="http://www.youtube.com/user/GeoSolutionsIT" target="_blank">http://www.youtube.com/user/GeoSolutionsIT</a><br><a href="http://www.linkedin.com/in/andreaaime" target="_blank">http://www.linkedin.com/in/andreaaime</a><br>
<a href="http://twitter.com/geowolf" target="_blank">http://twitter.com/geowolf</a><br><br>-------------------------------------------------------<br>
</div>