Hi Tatu, Thank you for your reply. Comments inline.
On Friday, September 16, 2022 at 10:18:08 PM UTC+1 Tatu Saloranta wrote: > ... > Ok, so this annotation was added for better OSGi support as per: > > https://github.com/FasterXML/jackson-core/issues/768 > > but would not want to add a requirement for non-OSGi users to have a > dependency to OSGi annotations. > Ok, that explains why this new lint warning is occurring. > Is there some actual problem here, or just general concern it might be > problematic? My understanding of Java annotation > handling is that JVM, JDK and class loaders must assume that > annotation definitions may well not be available in many cases. > And so missing should not be considered an error condition. There is no immediate issue - everything operates as expected without the presence of `aQute.bnd.annotation`. I raised this as a "mild" concern, to ensure that the compile-time (with Xl int:classfile) dependency on `aQute.bnd.annotation` was explicitly known (rather than any oversight). It's fine to leave things exactly as they are - folk compiling against Jackson core will just have to disable the `classfile` lint option. (this is most noticeable when compiling with warnings as errors, -Werror) -Chris. -- You received this message because you are subscribed to the Google Groups "jackson-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jackson-dev/c6457021-2165-43ec-9e6a-f3a435b94c3fn%40googlegroups.com.
