Right now, the plugin supports only one source directory. It would be interesting to add an arbitrary number of directories to the processor, to include, for example, other generated-sources that need to be processed.
<additionalSourceDirectories> <additionalSourceDirectory>${basedir}/generated-sources/jelly</additionalSourceDirectory> </additionalSourceDirectories>
Comment #1
Posted on Aug 25, 2012 by Happy OxThis is a patch to implement this feature :)
- fix.patch 4.21KB
Comment #2
Posted on Aug 26, 2012 by Quick RabbitHi, thx for support
I'll apply an deploy new release soon
Comment #3
Posted on Aug 26, 2012 by Quick Rabbit(No comment was entered for this change.)
Comment #4
Posted on Aug 26, 2012 by Quick Rabbit(No comment was entered for this change.)
Comment #5
Posted on Aug 26, 2012 by Quick RabbitHi,
Before adding new feature coming with your patch, i've preferred to get paths information directly from maven project using methods : getCompileSourceRoots & getTestCompileSourceRoots.
This should solve also your issue using the build-helper-maven-plugin ( see http://mojo.codehaus.org/build-helper-maven-plugin/usage.html ).
I've deployed this stuff in release 2.0.7-SNAPSHOT
Comment #6
Posted on Aug 27, 2012 by Happy OxHi,
yes, that's a nice addition. However, sometimes it is still very useful to make APT process sources from a completely different source tree without having to involve that source in the rest of the maven compile/package lifecycle. For example, I use APT-Jelly to generate source code using web-like templates (for example, to generate GWT RequestFactory proxies from JPA entities into a separate Maven project). For this I'm using the apt-maven-plugin, which provides this functionality (and support for the APT --factory option), but it'd be nice if I could depend on just on one plugin.
Comment #7
Posted on Aug 27, 2012 by Quick RabbitCompletely agree
I'd already planned to apply your patch .... But while was working on it, i've identified and fixed a lack of my implementation
Stay tuned ... I'll deploy it very soon
Comment #8
Posted on Aug 27, 2012 by Quick RabbitDeployed a new 2.0.7-SNAPSHOT with your patch.
Please test it and let me know
Comment #9
Posted on Sep 1, 2012 by Quick Rabbitto be verified
Status: Fixed
Labels:
Type-Enhancement
Priority-Medium
Milestone-Release2.0.7