<div dir="ltr">Hi.<br><br><div class="gmail_quote"><div dir="ltr">Alistair Leslie-Hughes <<a href="mailto:leslie_alistair@hotmail.com">leslie_alistair@hotmail.com</a>> schrieb am Do., 15. März 2018 um 10:02 Uhr:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div text="#000000" bgcolor="#FFFFFF">
Hi Kai,</div><div text="#000000" bgcolor="#FFFFFF"><br>
<br>
<div class="m_-424948052050203548moz-cite-prefix">On 15/03/18 18:46, Kai Krakow wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">Hello!
<div><br>
</div>
<div>Since a similar discussion has come up, what would be the proposed way to suggest tested patches from staging to mainline?</div>
<div><br>
</div>
<div>I'm currently carrying one or another selected (simple) patch from staging in my local patch-set that I apply to the latest wine version tag (so, I'm not running master). And these patches have proven quality for me since a view versions now.</div>
</div>
</blockquote></div><div text="#000000" bgcolor="#FFFFFF">
That good that other are testing the patches.  The main point here is to make sure they will be acceptable upstream.<br>
For example many of the wined3d/d3dx patches, work really well but need more work before they will be accepted.<br>
<br>
Which patches are looking at?</div></blockquote><div><br></div><div>None special yet, it's more about how I'm going to handle this later. Currently, there's maybe one patch that I would suggest for now after another round of testing.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div text="#000000" bgcolor="#FFFFFF"><blockquote type="cite"><div dir="ltr"><div>Should I just submit the patch to this list and remove Signed-off-by? Should I add Tested-by? Should I contact the original author first?</div>
<div><br>
</div>
<div>Or should I add something to the subject like "PATCH suggest", and let "git send-mail" also ping the original author?</div>
</div>
</blockquote></div><div text="#000000" bgcolor="#FFFFFF">
Leave the "Signed-off-by"<b> </b>there unless you have made major changes to the logic of the original patch.  Your 
<br>
"Signed-off-by" needs to be there as well. <br></div></blockquote><div><br></div>I think depending on the complexity of the patch, I might contact the author first as Andrew suggested. It sounds like a good thing to do.<br><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div text="#000000" bgcolor="#FFFFFF">
Please use git send-mail.</div></blockquote><div> </div><div>I already do.</div><div><br></div><div>Regards,</div><div>Kai</div></div></div>