Double deprecation message when using publishing plugin

[jnizet]

Expected Behavior

When using the publishing plugin without enabling the feature "STABLE_PUBLISHING", I should get a correct deprecation message.

Current Behavior

I get a message with two deprecation warnings:

[...] is now deprecated. has been deprecated.

This is caused by org.gradle.util.SingleMessageLogger#nagUserWithDeprecatedBuildInvocationFeature() appending " has been deprecated." to the summary, which itself already contains the deprecation warning.

Context

Not a big issue of course. I noticed it while translating a sample to Kotlin.

Steps to Reproduce (for bugs)

Use the publishing pugin without enabling the STABLE_PUBLISHING feature

I would be happy to file a PR, but I'm not sure if the correct fix would be to print the prefix as is, without appending "has been deprecated.", or if the specific summary used in PublishingPlugin.java should be reworded (i.e. if its " is now deprecated." ending should be removed)

[marcphilipp]

Good catch! Looks like this was broken in a1006da.

I think removing the " is now deprecated." ending should be fine.

[jnizet]

I'll file a PR tonight (French time)

[marcphilipp]

Please file it against the release branch so we can include the fix in 4.10.1.