scalac can't compile Java-classes containing type-annotations

[scabug]

The attached Java-classes contain a valid Annotation in TestAnnotation which is used in AnnotationTest.

Compiling with javac works using the following commandline:
*javac -d classes .java

Compiling with scalac breaks using the following commandline:
*scalac -d classes .java

AnnotationTest.java:4: error: illegal start of type
    public void testMethod(List<@TestAnnotation String> annotatedList);
                                ^
AnnotationTest.java:5: error: `>' expected but `}' found.
}
^
two errors found

This might be related to SI-8056, I am posting it anyway as I think this shouldn't break compiling Java-classes.

[scabug]

Imported From: https://issues.scala-lang.org/browse/SI-9853?orig=1
Reporter: Jochen (codepitbull)
Affected Versions: 2.11.8
See #8056
Attachments:

• AnnotationTest.java (created on Jul 11, 2016 6:54:15 AM UTC, 132 bytes)
• TestAnnotation.java (created on Jul 11, 2016 6:54:15 AM UTC, 265 bytes)

[scabug]

@SethTisue said:
is there a need for scalac to do anything with the annotations besides just ignore them? if not, this ought to be an easy fix. I don't think the Java parser is especially entangled with the rest of scalac

[scabug]

Jochen (codepitbull) said:
I don't think so. In my case the annotation is only used by an annotation parser to generate code.
So I think it would be save to simply ignore them on the Scala side.
But I am not sure that this would cover all usecases.
For now it would definitely be the best to simply ignore them as this will at least allow to build a project containing them.

[scabug]

Matthias Kurz (mkurz) said (edited on Mar 12, 2017 5:03:06 PM UTC):
This is working in Scala 2.12.1 already. (It works with the two files attached to this issue and also with Java classes within our app)

I think issue can be marked as resolved.

[scabug]

Matthias Kurz (mkurz) said (edited on Mar 12, 2017 6:48:16 PM UTC):
Can someone of the core team confirm this is officially fixed and not just a side effect of another fix so it doesn't break again in the future?
Are there any tests for this issue?
We want to rely that this is working not just in Scala 2.12.1 but any future Scala versions.

Thank you!

[scabug]

@som-snytt said (edited on Mar 12, 2017 9:36:38 PM UTC):
scala/scala@7e57bcc

Edit: sorry, you asked for the core team. They might be asleep right now. Also, we had the daylight saving time switch, so even folks in this TZ are asleep. I'm asleep at the keyboard right now.

[scabug]

Matthias Kurz (mkurz) said:
@A. P. Marki - thanks, it's not that urgent ;)