<div dir="ltr"><div class="" style="font-family:arial,sans-serif;font-size:medium"><div class=""><div class="" style="font-size:17.27272605895996px"><h2 id=":mj" class="" tabindex="-1"></h2></div></div></div><div class="" style="font-family:arial,sans-serif;font-size:medium">
<div class=""><div class=""><div class="" tabindex="-1"><div class="" style="width:563.6363525390625px"><div class=""><div id=":t4"><div class=""><div class=""><div class="" style="font-size:12.727272033691406px">Hi compiler-dev,<br>
</div><div id=":xl" class="" style="font-size:12.727272033691406px"><div id=":xk" class="" style="overflow:hidden"><div dir="ltr"><br><font face="arial, sans-serif">Please review this fix for JDK-8042699 which fixes a dependency tracking failures in sjavac.</font><br>
<br>Description:<div>The previous change from . to / in the arguments¬†accidentally broke the dependencies inside the javac_state file. A trivial fix is supplied for this, however the tests did not catch this, since the tests did not use packages with more than one level, ie no dots...thus the tests are improved to actually detect this kind of error.¬†<br>
</div><div><br>Link to webrev:</div><div><font face="arial, sans-serif"><a href="http://cr.openjdk.java.net/~ohrstrom/webrev-8042699/">http://cr.openjdk.java.net/~ohrstrom/webrev-8042699/</a></font><br><br>Link to bug report:<br>
<a href="https://bugs.openjdk.java.net/browse/JDK-8042699">https://bugs.openjdk.java.net/browse/JDK-8042699</a></div></div></div></div></div></div></div></div></div></div></div></div></div></div>