-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use Vert.x RPC service code generation #53
Comments
@pmlopes @vietj would you happen to have any idea why this seemingly simple change to use the io.vertx:vertx-service-proxy annotationProcessor for
where this
then the
|
An upgrade from Gradle 4.6 to 4.7 (incl. "Incremental annotation processing") does not help. @edewit FYI |
whoa there's a bigger problem, it's not just that class ( NB: We use the (new, since Gradle 4.6) annotationProcessor scope:
because wtih
it fails like this for us in this project:
|
@pmlopes @vietj FYI @edewit and I just figured out that this is due to our use of Error Prone, see tbroyer/gradle-errorprone-plugin#70. |
@pmlopes @vietj FYI @edewit I now believe this more a Vert.x than an Error Prone problem, see eclipse-vertx/vertx-codegen#181. |
#61 has the start of this ready for review now. Keeping this issue open until loose ends are resolved. |
@edewit I don't suppose you would have any idea why what's now on branch issue53_codegen_opposite-direction (as of 60d7cc7) fails like this:
it's somehow related to this which is is (now) in
that is not found - but I'm really struggling to understand which path is relative to what here why it's not found. I initially thought that shouldn't contain
which is even weirder, because that is definitely there. |
@edewit helped me figure that if we (manually) replace:
with:
and then |
I see the issue, module resolution states that modules which path start with dot are relative to the current file. Modules not starting with dot then follow the node_modules lookup. So indeed the generator is broken. |
Closing this issue, as we are currently not using Vert.x client code generation anymore. |
to replace current event bus messaging related code
The text was updated successfully, but these errors were encountered: