grossws commented on code in PR #1141:
URL: https://github.com/apache/logging-log4j2/pull/1141#discussion_r1022793318


##########
src/site/markdown/maven-artifacts.md.vm:
##########
@@ -149,14 +149,29 @@ file. When you specify the version identifier in this 
section, you don't have to
 </dependencyManagement>
 ```
 
-To use this with Gradle, an additional [Gradle 
plugin](https://github.com/spring-gradle-plugins/dependency-management-plugin)
+Gradle 5.0+ supports importing Maven BOM as a 
[platform](https://docs.gradle.org/current/userguide/platforms.html#sub:bom_import)
+to align dependency versions.
+
+`build.gradle`
+
+```
+dependencies {
+  implementation 
platform('org.apache.logging.log4j:log4j-bom:${Log4jReleaseVersion}')
+
+  implementation 'org.apache.logging.log4j:log4j-api'
+  implementation 'org.apache.logging.log4j:log4j-core'

Review Comment:
   I usually use `runtimeOnly` for it but saw the following:
   
   > However, below we list Log4j core as a compile time dependency to improve 
the startup time for [custom 
plugins](https://logging.apache.org/log4j/2.x/manual/plugins.html) as it 
provides an annotation processor that generates a metadata file to cache plugin 
information as well as the necessary code to compile against to create custom 
plugins



-- 
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]

Reply via email to