[rosedu-general] [Techblog] Contributing Upstream

[Sergiu Iordache]

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-