gnodet commented on PR #793: URL: https://github.com/apache/maven/pull/793#issuecomment-5120550467
> @gnodet do we consider it is fixed in maven 4 and "dont care" in maven 3 land or should i rebase? Actually, it's only partially fixed in Maven 4 I think, see https://github.com/apache/maven/blob/b61b3bd0b0e473ac2a62f41ec635c7df11396b57/impl/maven-core/src/main/java/org/apache/maven/classrealm/DefaultClassRealmManager.java#L114 I'd rather remove it and enforce the use of `org.apache.maven.api.plugin.Log`. So I would lean towards undeprecating, if does not make much sense to deprecate it in Maven 3 and require it in Maven 4, though that's not the exact same class, so it's more about guiding towards the correct direction. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
