[rosedu-general] [Techblog] Contributing Upstream
Sergiu Iordache
sergiu.iordache at gmail.com
Sun Nov 27 17:33:39 EET 2011
2011/11/27 Mihai Maruseac <mihai.maruseac at gmail.com>:
> 2011/11/26 Sergiu Iordache <sergiu.iordache at gmail.com>:
>> 2011/11/26 Mihai Maruseac <mihai.maruseac at gmail.com>:
>>> 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 :)
Sergiu
More information about the rosedu-general
mailing list