RFR (and sponsor): java.lang.annotation.ContainerAnnotation is not built with the JDK (7158930)

[Erik Joelsson]

Looks ok to me.

(Not an official reviewer)

/Erik

On 2012-04-04 19:27, Joel Borggrén-Franck wrote:
> Hi,
>
> java.lang.annotation.ContainerAnnotation.java that was added with CR 7151008 is not built and added to rt.jar.
>
> I suspect this is because the source directory java/lang/annotation is not in the AUTO_FILES_JAVA_DIRS list in the Makefile for reflection and annotations, make/java/java/reflect/Makefile . Also the ContainerAnnotaion class is not referenced from the JDK.
>
> I have posted a webrev for a fix that adds this directory:
>
> http://cr.openjdk.java.net/~jfranck/7158930/webrev.00/
>
> This seems to work, with this addition ContainerAnnotation gets compiled and included in rt.jar and rt. jar contains exactly 1 more file than before the patch.
>
> cheers
> /Joel