<br><br><div class="gmail_quote">On Tue, Feb 7, 2012 at 10:46 AM, Patrick Mézard <span dir="ltr"><<a href="mailto:patrick@mezard.eu">patrick@mezard.eu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
</div>Can you be more specific? Are you talking about this patch:<br>
<br>
  [PATCH hglib] merge and resolve return boolean instead of raising an exception<br>
  <a href="http://selenic.com/pipermail/mercurial-devel/2012-January/037664.html" target="_blank">http://selenic.com/pipermail/mercurial-devel/2012-January/037664.html</a><br>
<br>
which Idan commented afterwards?<br></blockquote><div><br></div><div> It is not the only one.  I totally agree with the review, but the question is general: why does anyone spend time on a review if it would be quicker and more efficient just to change it and apply to the code base ?</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
So yes, when correct looking patches (commented code, good indentation, not kludgy) lands on mercurial-devel, they are considered as regular submission and their author are assumed to be OK to be motioned through the review process. If it is not your intent, which is perfectly fine, please make it clear and possibly open an issue in the bug tracker, and paste your diff there as a fix suggestion. But in this case, you explicitely give up on attribution, at least that is the way I see it.<br>

<span class="HOEnZb"><font color="#888888"><br>
--<br>
Patrick Mézard<br></font></span></blockquote><div><br></div><div>This is my question !!! HOW should I make it clear ??? Does the patchbomb extension provide such an option ?</div><div>(I did not even know that some kind of  'attribution' is involved...)</div>
<div><br></div><div>><span style>The BTS is not a patch submission channel. But my understanding is your patch is more about describing the >problem, analyzing the issue. The BTS sounds like a good place for this.</span></div>
<div><span style><br></span></div><div><span style>I did not quite catch you. Do you mean that the wiki shall say instead: "We do appreciate patches on the BTS" ?</span></div><div><span style><br></span></div><div>
<span style><font color="#222222" face="arial, sans-serif">(By the way what BTS stands for ? I could find any explanation...)</font></span></div><div><span style><font color="#222222" face="arial, sans-serif"><br></font></span></div>
<div><span style><font color="#222222" face="arial, sans-serif">If the way to report a problem is unclear and/or complex then I would rather stop reporting. I am afraid I will not be the only one...</font></span></div><div>
<span style><font color="#222222" face="arial, sans-serif"><br></font></span></div><div><span style><font color="#222222" face="arial, sans-serif">-</font></span></div><div><span style><font color="#222222" face="arial, sans-serif">Andrey</font></span></div>
</div>