
<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>Hi Max,</div><div><br></div><div>I started to split the issue because it became too complex for me all at once, I must admit, hoping being able to cope with it better in smaller tasks. I fully agree that it is neither serious nor urgent enough to solve it independently but it would help at least me to keep the overview better organized.</div><div><br></div><div>If you don't object I would suggest another separate bug and patch for ManifestDigester confusing "Manifest-Main-Attributes individual section with main attributes, also neither serious nor urgent. This one and manifest ending in \r would then fall away from JDK-8217375.</div><div><br></div><div><div>I would personally welcome to see the jar signing update incompatibility JDK-8217375 fixed in JDK 11 because I suggested the change that introduced the bug there originally but that does not necessarily require also to backport the \r and the "Manifest-Main-Attributes"-confusion bugs. By splitting up JDK-8217375 as suggested we would also get the suitable set of changes to backport.</div><div><br></div><div>I'm flattered that you ask for my opinion how to proceed but I cannot decide this alone. I hope I described one option a bit clearer. It might be worth two more bugs in jira or would this unnecessarily spoil it?</div><div><br></div><div>Philipp</div></div><div><br></div><div><br></div><div>On Sun, 2019-01-20 at 17:27 +0800, Weijun Wang wrote:</div><blockquote type="cite"><div dir="ltr">Hi Philipp </div><div dir="ltr"><br></div><div dir="ltr">This fix is also part of another fix you proposed for jarsigner re-signing. Shall we fix it in one big changeset? IMO this is not serious enough to be included in JDK 12 and let’s fix all in JDK 13. </div><div dir="ltr"><br></div><div dir="ltr">Thanks</div><div dir="ltr">Max</div><div dir="ltr"><br></div></blockquote></body></html>