velo / maven-annotation-plugin

Automatically exported from code.google.com/p/maven-annotation-plugin
0 stars 0 forks source link

default value for defaultOutputDirectory #13

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Please change the default expression value in the plugin to
${project.build.directory}/generated-sources/apt-main

or some other direct subdir of generated-sources. Some IDEs (at least
NetBeans but probably more) are relying on this convention when displaying
and referencing generated sources.

Thanks.

Original issue reported on code.google.com by mkle...@gmail.com on 23 Jan 2010 at 9:31

GoogleCodeExporter commented 9 years ago
thx for feedback 

i'm going change it  

Original comment by bartolom...@gmail.com on 25 Jan 2010 at 11:09

GoogleCodeExporter commented 9 years ago
i've deployed a release 1.3.1-SNAPSHOT

could you test it and give me feedback ?

Regards,
Bartolo

Original comment by bartolom...@gmail.com on 25 Jan 2010 at 11:23

GoogleCodeExporter commented 9 years ago
I think I found a problem with this.

What steps will reproduce the problem?
1. Include both process and process-test as goals in a project . Make sure both 
goals
generate sources ( *.java files ).
2. Run mvn test
3. Run mvn test

What is the expected output? What do you see instead?

Both testing runs succeed. Instead, the second testing run fails.

What version of the product are you using? On what operating system?

Plugin version 1.3.1, Maven 2.2.1 .

Please provide any additional information below.

In r189 the default output paths were changed to be the same for both goals. 
This
causes the following problems:

- the first run of mvn test will run process and then process-test. At this 
point
allthe generated sources are found in the generated-sources/apt
- the second run preserves the generated-sources/apt contents, and on invoking
compiler:compile the generated source folder contains source file entries which 
are
only compilable with the test classpath.

Please revert to using separate output folders for process and process-test.

Original comment by robert.munteanu on 8 Feb 2010 at 9:01

GoogleCodeExporter commented 9 years ago
i've deployed a release 1.3.2-SNAPSHOT 

it set as default for *process* goal the defaultOutputDirectory to 
${project.build.directory}/generated-sources/apt 
while for *process-test* the defaultOutputDirectory 
${project.build.directory}/generated-sources/apt-test

could you try and  give me feedbacks?

Original comment by bartolom...@gmail.com on 8 Feb 2010 at 10:03

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Works fine here, thank you.

Original comment by robert.munteanu on 8 Feb 2010 at 10:26

GoogleCodeExporter commented 9 years ago
thank you too 

i'll deploy 1.3.2 soon

Original comment by bartolom...@gmail.com on 8 Feb 2010 at 10:40