[BUILD-751] Consider phasing out animal-sniffer maven plugin Created: 20/Apr/22 Updated: 29/Jun/22 |
|
| Status: | Open |
| Project: | Build |
| Component/s: | Poms |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Neutral |
| Reporter: | Federico Grilli | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | foundation_team | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Template: |
|
||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||
| Task DoR: |
Empty
|
||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||
| Epic Link: | Build on JDK11+ | ||||||||||||||||||||
| Team: | |||||||||||||||||||||
| Description |
|
Please note this plugin is now in maintenance level as the exact same feature can be now easily achieved using the --release flag from Javac see (http://openjdk.java.net/jeps/247) https://github.com/mojohaus/animal-sniffer |
| Comments |
| Comment by Michael Duerig [ 03/Jun/22 ] |
|
I believe this is has already been done with https://git.magnolia-cms.com/projects/BUILD/repos/poms/commits/faf668c590192d70040a827fc63d160fe98800be#poms/super-pom/pom.xml |
| Comment by Federico Grilli [ 13/Jun/22 ] |
|
Possibly this is actually outdated but I see we still have the plugin in poms, not sure why https://git.magnolia-cms.com/projects/BUILD/repos/poms/browse/poms/super-pom/pom.xml#133,386-387,933,941 |
| Comment by Maxime Michel [ 13/Jun/22 ] |
|
The reason for the animal-sniffer leftover properties is that PPOMs are still used for JDK8 builds. As long as we can't completely pull the plug on those, this remains relevant. This should therefore stay open in the meanwhile. |