Y
Hacker News
new
|
ask
|
show
|
jobs
by
jakeinspace
1614 days ago
It’s a 1-line patch, I didn’t send it in my initial email but maybe that was a faux pas.
1 comments
tych0
1614 days ago
I'd say sending the patch with git send-email --in-reply-to=<the header of your last email> is good. A patch is much easier to apply than write :)
link
jakeinspace
1614 days ago
Thanks! Would you recommend I send to the maintainer(s), and Cc the mailing list?
link
tych0
1614 days ago
Yeah, I'd just use whatever the output of ./scripts/get_maintainer.pl says. It will also suggest any recent committers to that area of the code, which I've found useful in the past. Usually I put the maintainers as To:, and everyone else as Cc:.
link