<div dir="ltr">Possibly an appendix defining coreboot leadership would help.<div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 30, 2015 at 9:51 AM Martin Roth <<a href="mailto:gaumless@gmail.com">gaumless@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Alex,<br>
  Thanks for voicing your concerns.  I do think that most of the<br>
issues you bring up are handled, or aren't as big issues you make them<br>
out to be.  As always, I could be wrong, so I'd again invite others to<br>
give their opinions.  I've tried to address your issues below.<br>
<br>
> ... the "maintainer" is given almost absolute veto power<br>
> in all matters relating to that code.<br>
...<br>
> There are also no responsibilities defined for a "maintainer", despite<br>
> such person being given near-inter-stellar of power.<br>
I did try to define maintainers in the document:<br>
* Try to coordinate with platform maintainers when making changes to<br>
  platforms. The platform maintainers are the users who initially pushed the<br>
  code for that platform, as well as users who have made significant changes<br>
  to a platform. To find out who maintains a piece of code, please use<br>
  util/scripts/maintainers.go or refer to the original author of the code in<br>
  git log.<br>
I'm not certain how "Try to coordinate" is being read as "almost<br>
absolute veto power" or "near-inter-stellar of power".<br>
<br>
>  Concerns from external reviewers often get<br>
> ignored or brushed off as "don't care".<br>
This is absolutely addressed by these three guidelines:<br>
* Respond to anyone who has taken the time to review your patches.<br>
* If there have been comments or discussion on a patch, verify that the<br>
  comments have been addressed before giving a +2.<br>
* If there is still ongoing discussion to a patch, try to wait for a<br>
conclusion to the discussion before submitting it to the tree.<br>
<br>
>  I've seen plenty of cases where patches have been -2'd<br>
> by <a href="mailto:W@g.com" target="_blank">W@g.com</a> until other @<a href="http://g.com" rel="noreferrer" target="_blank">g.com</a> patches were merged, only for the -2 to be<br>
> removed once it was on the contributor to rebase his work.<br>
I haven't seen this, but if you see a situation where you feel like<br>
someone is being treated unfairly, please bring it to the attention of<br>
the coreboot leadership or post something to the mailing list.<br>
<br>
> This then creates another significant issue: the bar for contributions<br>
> from such teams is very low, while anyone else experiences a much higher<br>
> bar. This has many times resulted in sub-optimal code being merged, and<br>
> great contributions getting delayed and abandoned.<br>
Again, I feel like if you comment on these patches, the rules above<br>
should handle these issues.  And again, if you feel like something is<br>
unfair, bring attention to it.<br>
<br>
> ...  Raptor Eng. was asking around $15000 to upstream ...<br>
> All of a sudden, we've made coreboot a very expensive<br>
> project to support.<br>
Additionally, I specifically reached out to tpearson with these<br>
guidelines to get his feedback before they were posted.  I have tried<br>
to address the concerns he brought up.  If you (or anyone else) feel<br>
like there's something specific we can add here that will help him or<br>
future developers get code submitted, please recommend the addition.<br>
<br>
> Then there's talk about a vague "coreboot leadership"...<br>
Patrick addressed this in the review - here's Stefan talking about<br>
coreboot leadership:<br>
<a href="http://blogs.coreboot.org/blog/2015/05/11/on-coreboot-leadership/" rel="noreferrer" target="_blank">http://blogs.coreboot.org/blog/2015/05/11/on-coreboot-leadership/</a><br>
<br>
--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br>
</blockquote></div>