Skip to content
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

-sourcepath is not configured #63

Closed
htfv opened this issue Sep 8, 2016 · 8 comments
Closed

-sourcepath is not configured #63

htfv opened this issue Sep 8, 2016 · 8 comments
Assignees
Milestone

Comments

@htfv
Copy link

htfv commented Sep 8, 2016

I'm writing an annotation processor which reads the source files using processingEnv.getFiler().getResource(StandardLocation.SOURCE_PATH, ...). It works if annotations are processed with m-compiler-p, but it does not work with m-annotation-p because it does not add the -sourcepath option to the compiler.

Possible workaround is to add the -sourcepath option manually through the compilerArguments property. It would be nice if the plugin could add all source roots automatically.

@bsorrentino bsorrentino self-assigned this Sep 8, 2016
@bsorrentino
Copy link
Owner

Hi

You right, i'll add sourcepath support asap

@bsorrentino bsorrentino added this to the 3.2 milestone Sep 8, 2016
@bsorrentino
Copy link
Owner

I've released 3.2.0-SNAPSHOT with fix

Please test it and let me know

Thanks in advance

@htfv
Copy link
Author

htfv commented Sep 27, 2016

Thank you for the quick fix and sorry for the late response.

Unfortunately, it does not work correctly. -sourcepath should be the list of source directories, not the compile classpath. m-compiler-p sets this parameter to compileSourceRoots + generatedSourcesDirectory.

@bsorrentino
Copy link
Owner

Yes ... I miss this detail :-(

I'll fix it very soon

@bsorrentino
Copy link
Owner

bsorrentino commented Sep 28, 2016

I've released a new 3.2.0-SNAPSHOT with fix

Please, test it and let me know

Thanks in advance

@htfv
Copy link
Author

htfv commented Oct 4, 2016

It works now! Many thanks!

@bsorrentino
Copy link
Owner

Thank you too

I'll arrange a new release soon

@bsorrentino
Copy link
Owner

version 3.2.0 released

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants