RFR: 8042699: sjavac does not properly track dependencies

Jonathan Gibbons jonathan.gibbons at oracle.com
Wed May 7 21:40:58 UTC 2014


Add the bug number 8042699 into the @bug line in the test and you're 
good to go.

You need the bug number to be present so that down stream processes can 
verify that there was a specified test added that is directly relevant 
to the fix.

-- Jon


On 05/07/2014 02:36 PM, Fredrik Öhrström wrote:
> Hi compiler-dev,
>
> Please review this fix for JDK-8042699 which fixes a dependency 
> tracking failures in sjavac.
>
> Description:
> 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.
>
> Link to webrev:
> http://cr.openjdk.java.net/~ohrstrom/webrev-8042699/ 
> <http://cr.openjdk.java.net/%7Eohrstrom/webrev-8042699/>
>
> Link to bug report:
> https://bugs.openjdk.java.net/browse/JDK-8042699

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20140507/b5d33f9f/attachment.html>


More information about the compiler-dev mailing list