Status Update
Comments
ap...@google.com <ap...@google.com> #2
ju...@google.com <ju...@google.com> #3
To clarify as in, has not reached Google maven, but should be part of the beta02 release.
du...@google.com <du...@google.com> #5
The bug here is that BCV does not respect @RestrictTo and still includes it into the API files as a regular public API, when in reality we should not throw on changes to these or only track them in -restricted api files like Metalava does.
We also need to check that the restricted API lint / error works for non-JVM source sets.
ju...@google.com <ju...@google.com> #6
Metalava treats @RestrictTo
APIs differently depending on the scope used. LIBRARY
scope isn't tracked in signature files. LIBRARY_GROUP_PREFIX
scope is tracked in restricted_
signature files and checked for compatibility. LIBRARY_GROUP
scope is treated like LIBRARY
for atomic library groups and treated like LIBRARY_GROUP_PREFIX
for non-atomic groups.
However, it looks like the nonPublicMarkers
from the KlibDumpFilters
currently can only be qualified annotation names, so we can't differentiate between the RestrictTo
annotations based on the scope value.
ju...@google.com <ju...@google.com> #7
Where ProtoConstructorCall
is really IrConstructorCall
Description