Skip to content

Conversation

@jjohannes
Copy link
Contributor

This adds an example that show that the Gradle Metadata published for Log4j has the desired effect on the compile classpath when Log4j is used in Gradle builds.

See: apache/logging-log4j2#3437

This changes the "Android test pipeline" to a "Gradle test pipeline" to run all Gradle-based tests. Which are at the moment the "Android test" and this newly added test.

@vy
Copy link
Member

vy commented Mar 2, 2025

@ppkarwasz, I'd appreciate if we can co-review this one. Would you mind sharing your thoughts, please?

@jjohannes jjohannes force-pushed the add-gradle-metadata-example branch from 2fbeaf0 to b562081 Compare March 3, 2025 21:24
@jjohannes jjohannes force-pushed the add-gradle-metadata-example branch from b562081 to 853fd87 Compare March 20, 2025 13:21
@jjohannes jjohannes requested review from ppkarwasz and vy March 20, 2025 13:25
@ppkarwasz ppkarwasz moved this from To triage to In review in Log4j bug tracker Apr 9, 2025
@ppkarwasz ppkarwasz added the enhancement New feature or request label Apr 9, 2025
Copy link
Contributor

@ppkarwasz ppkarwasz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jjohannes,

Sorry for the detail. This looks good to me.
Storing gradle-wrapper.jar in the repository looks like an acceptable solution to me. We'll probably just need to check its checksum each time it is upgraded.

@jjohannes jjohannes force-pushed the add-gradle-metadata-example branch from 853fd87 to bdce04c Compare April 10, 2025 06:04
@jjohannes jjohannes force-pushed the add-gradle-metadata-example branch from bdce04c to 4fd2a25 Compare April 10, 2025 06:12
@jjohannes jjohannes requested a review from ppkarwasz April 10, 2025 06:13
This adds an example that show that the Gradle Metadata published
for Log4j has the desired effect on the compile classpath when
Log4j is used in Gradle builds.

This changes the "Android test pipeline" to a "Gradle test pipeline"
to run all Gradle-based tests. Which are at the moment the
"Android test" and this newly added test.

Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
@jjohannes jjohannes force-pushed the add-gradle-metadata-example branch from 4fd2a25 to bf205b7 Compare April 10, 2025 06:22
Copy link
Contributor

@ppkarwasz ppkarwasz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can probably remove the -Plog4j.repository.url=$LOG4J_REPOSITORY_URL parameters, otherwise it looks good to go.

@ppkarwasz ppkarwasz merged commit b12af68 into apache:main Apr 10, 2025
11 of 13 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Log4j bug tracker Apr 10, 2025
@ppkarwasz
Copy link
Contributor

Some additional changes were required post merge:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants