Skip to content
This repository has been archived by the owner on Feb 23, 2023. It is now read-only.

Conversation

mhalbritter
Copy link
Contributor

@mhalbritter mhalbritter commented Mar 30, 2022

  • Updates the maven-compiler-plugin in the samples to the latest available one, 3.10.1

  • The maven-compiler-plugin starting from 3.9.0 removed the configuration option 'compilePath', which the TestGenerateMojo was using. This option was not used and have been removed by the maven-compiler-plugin team in commit apache/maven-compiler-plugin@419bf04

See gh-1514

@sdeleuze
Copy link
Contributor

Thanks for this PR, does the AOT plugin still work with previous versions (<= 3.8.1) or not?

@bclozel Could you please review the PR?

@mhalbritter
Copy link
Contributor Author

Oh sorry, should have put that in the description. Yes, i tested it with 3.8.1 too, and it works. The compilePath is a noop in 3.8.1.

Copy link
Contributor

@snicoll snicoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. I've added a suggestion, let me know what you think.

samples/maven-parent/pom.xml Outdated Show resolved Hide resolved
samples/maven-parent/pom.xml Outdated Show resolved Hide resolved
@snicoll snicoll added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Mar 30, 2022
@snicoll snicoll added this to the 0.11.4 milestone Mar 30, 2022
@snicoll snicoll changed the title Add support for maven-compiler-plugin > 3.8.1 Spring Aot Maven Plugin does not work with Maven Compiler Plugin > 3.8.1 Mar 30, 2022
- Updates the maven-compiler-plugin in the samples to the latest available one, 3.10.1

- The maven-compiler-plugin starting from 3.9.0 removed the configuration option 'compilePath', which the TestGenerateMojo was using. This option was not used and have been removed by the maven-compiler-plugin team in commit 419bf040893dac8f300deb5b36daf5f487dea8d2

See spring-atticgh-1514
@mhalbritter mhalbritter force-pushed the mh/1514-spring-aot-maven-plugin-does-not-work-with-maven-compiler-plugin branch from 210cc7a to 029793d Compare March 31, 2022 06:21
snicoll pushed a commit that referenced this pull request Mar 31, 2022
The maven-compiler-plugin starting from 3.9.0 removed the configuration
option 'compilePath', which the TestGenerateMojo was using. This option
was not used and have been removed by the maven-compiler-plugin team in
commit 419bf040893dac8f300deb5b36daf5f487dea8d2

See gh-1559
@snicoll snicoll closed this in a094918 Mar 31, 2022
@mhalbritter mhalbritter deleted the mh/1514-spring-aot-maven-plugin-does-not-work-with-maven-compiler-plugin branch March 31, 2022 06:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug A general bug
Development

Successfully merging this pull request may close these issues.

4 participants