2011/11/27 Sergiu Iordache <[email protected]>: > 2011/11/27 Mihai Maruseac <[email protected]>: >> 2011/11/26 Sergiu Iordache <[email protected]>: >>> 2011/11/26 Mihai Maruseac <[email protected]>: >>>> Salut, >>>> >>>> Inspirat de ce-am făcut pentru primul patch pe kernel am scris un >>>> scurt articol pe techblog despre pașii care trebuiesc urmăriți pentru >>>> a trimit un patch upstream. Please review, îl publicăm luni >>>> dimineață/duminică seară. >>>> >>>> Also, cine/ce scrie săptămâna viitoare? >>> >>> Aș pune accentul pe câteva elemente care zic eu sunt cruciale pentru >>> acceptarea unui patch(mai mult decât link către articolul lui RD): >>> - existent mesaj de commit standard, adică: >>> <componenta>: <rezumat commit scurt> >>> >>> <mesaj mai lung> >>> - câte o schimbare logică per commit. Dacă incerci să adaugi/schimbi >>> 2 sau mai multe chestii într-un commit nu e bine. >>> - commit-uri cât mai mici posibile. Cu cât schimbările sunt mai mici, >>> cu atât șansele sunt mai mari să fie acceptat. >>> Extra: >>> - sunt șanse mari ca tu înainte să îți dai seama că vrei să faci >>> commit upstream lucrai în tree-ul propriu. În cazul ăsta git >>> cherry-pick e super util ca sa muți commit-ul dintr-un tree în altul >>> (una din comenzile ignorate / neștiute dar pe care le-am folosit de o >>> suta de ori vara asta, super importantă atunci când ești sub review >>> control și commit-urile trebuie să arate într-un fel anume). >>> - comanda următoare, luata din ghidul[1] de la Chromium OS, te >>> scutește de copy paste când tre să selectezi cui îi dai mail: >>> ./scripts/get_maintainer.pl 0001-mypatch.patch | \ >>> sed 's/ *([^)]*) *//g' | \ >>> sed 's/"//g' | \ >>> sed 's/^\(.*\)$/--cc="\1" /' | \ >>> tr -d '\n' >>> >>> Nu știu unde ar intra mai bine în articol sau dacă toată lumea le >>> consideră relevante, de aia le-am pus aici întai. >>> Sergiu >>> >>> [1] >>> http://dev.chromium.org/chromium-os/how-tos-and-troubleshooting/kernel-faq#TOC-How-do-I-send-a-patch-upstream- >> >> Mersi mult, am integrat și sugestiile tale și link la wiki-ul de la >> Chromium. Also, am pus link la prezentarea lui Greg KH de pe youtube, >> cea dată de Daniel pe listă când a propus bere pentru primul commit >> upstream. >> >> Please review :) > > LGTM :) >
Articolul e up[1]. [1]: http://techblog.rosedu.org/contributing-upstream.html -- MM _______________________________________________ http://lists.rosedu.org/cgi-bin/mailman/listinfo/rosedu-general
