[rosedu-general] [Techblog] Contributing Upstream
Mihai Maruseac
mihai.maruseac at gmail.com
Mon Nov 28 10:26:00 EET 2011
2011/11/27 Sergiu Iordache <sergiu.iordache at gmail.com>:
> 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 :)
>
Articolul e up[1].
[1]: http://techblog.rosedu.org/contributing-upstream.html
--
MM
More information about the rosedu-general
mailing list