Fixed in #5521
Locally tested with SCM Activity plugin.
Now we can override each SanityChecks individually :
By default both variables are initialized with skipSanityChecks value (false)
IMO goal of this ticket is to allow usage of a standard plugin properties like "license.skip" instead of introduction of a new ones. Rationale - is to make possible to use maven-bash-completion instead of remembering new properties.
If you want and if no hurry, then I can take a look on this a bit later.
Evgeny, I did not find any simple solution to keep license.skip or animal.sniffer.skip properties and use a skipSanityChecks to control them also.
So I came back to my first solution that's simple and works.
Maybe profiles may be a solution, but I make the pom.xml quite more complex for me.
So I let you either look at it or resolved it.
Done in r5584: introduced new profile skipSanityChecks, which can be activated via property 'skipSanityChecks' (so no changes in behaviour) and which allows to use standard properties "animal.sniffer.skip" and "license.skip" alone.
Eric, could you please verify this on your side?
Evgeny, the drawback with this solution is that as soon as the property skipSanityChecks is defined to any value. The sanity checks are bypassed.
For instance using -DskipSanityChecks=false will skip the checks.
After r5604 behaviour is :
Ok tested every parameter given at command line overriddes the value on the profile.
So everything works perfectly for me.