-
Notifications
You must be signed in to change notification settings - Fork 13
Add example to test transitive compile-time dependencies in Gradle #279
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
4799c71 to
2fbeaf0
Compare
log4j-samples-gradle-metadata/gradle/wrapper/gradle-wrapper.jar
Outdated
Show resolved
Hide resolved
|
@ppkarwasz, I'd appreciate if we can co-review this one. Would you mind sharing your thoughts, please? |
2fbeaf0 to
b562081
Compare
b562081 to
853fd87
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
853fd87 to
bdce04c
Compare
bdce04c to
4fd2a25
Compare
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>
4fd2a25 to
bf205b7
Compare
There was a problem hiding this 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.
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.