[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:
dependency
is depended upon by BUILD-812 Remove javaVersionWithoutInitialOne p... Accepted
relation
is related to BUILD-303 Build on JDK11+ Selected
is related to BUILD-426 Retire animal-sniffer on Java 9+ Closed
Template:
Acceptance criteria:
Empty
Task DoR:
Empty
Date of First Response:
Epic Link: Build on JDK11+
Team: Foundation

 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

See also https://magnolia-cms.slack.com/archives/CHYN7JZCJ/p1650442476793349?thread_ts=1650442473.788589&cid=CHYN7JZCJ



 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 
Perhaps just a matter of removing the leftovers?

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.

Generated at Sun Feb 11 23:44:50 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.