<div dir="ltr"><div dir="ltr"><div>Jiri & Mario,</div><div><br></div><div>Here is my final patch (after merging with backports):</div><div><a href="http://cr.openjdk.java.net/~lbourges/itw/patch_EDT.log.3">http://cr.openjdk.java.net/~lbourges/itw/patch_EDT.log.3</a></div><div><br></div><div>Laurent<br></div></div></div><br><div class="gmail_quote"><div dir="ltr">Le mar. 11 sept. 2018 Ã Â 08:48, Laurent Bourgès <<a href="mailto:bourges.laurent@gmail.com">bourges.laurent@gmail.com</a>> a Ã©crit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>Jiri,</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
>  Â  Â Please note <a href="http://icedtea.classpath.org/hg/icedtea-web/rev/5290684409aa" rel="noreferrer noreferrer" target="_blank">http://icedtea.classpath.org/hg/icedtea-web/rev/5290684409aa</a> it will interact with the<br>
>  Â  Â patch for bleeding edge.  Maybe it is worthy to backport it to 1.7 and to adapt your patch on<br>
>  Â  Â top of it?<br>
> <br>
> <br>
> Please backport to 1.7.<br>
done. Plus few more.<br>
Head and 1.7 are now very very similar. All except rust launchers.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Thanks a lot, I merged and it works well.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> I merged so the patch should not have these changes after backport:<br>
<br>
So your patch now do not apply correctly, but the two rejected hunks can be happily ignored/removed. <br>
Feel free to provide updated version, but not necessary.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I agree you should just ignore changes to JNLPRuntime.</div><div dir="auto">I can post an updated patch for tracability ?</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> I tested again and both 'javaws -viewer' and itw-settings dialogs are OK.<br>
> <br>
<br>
Nothing more on my nitpicking list. Unless you or Mario will bring some news tomorrow, I will apply <br>
it to 1.7 and will look forward for patch for head, and other patches.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Excellent: my first patch is accepted.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Note, that I'm on vacation 13-23.9, and I will be a bit delayed with replies. Lets guess a week <br>
delayed :)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Enjoy holidays, I will have more time to improve shell scripts.</div><div dir="auto">Probably I will setup an ITW mirror on github following your instructions (license...).</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I will write an changelog on your behalf. If you wish to write it on your own, feel free.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I can review the update to the change log... </div><div dir="auto"><br></div><div dir="auto">Laurent</div></div>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">-- <br>Laurent Bourgès</div>