Skip to content

Add transitive compileOnlyApi dependencies - #4468

Open
Vivacity5X wants to merge 1 commit into
apache:mainfrom
Vivacity5X:issue-3600-compile-only-api
Open

Vivacity5X wants to merge 1 commit into
apache:mainfrom
Vivacity5X:issue-3600-compile-only-api

Conversation

@Vivacity5X

Copy link
Copy Markdown

Summary

This PR addresses [#3600](#3600) by porting the transitive compile-only dependency support to the main branch.

  • Configure Gradle Module Metadata generation using gradle-module-metadata-maven-plugin.
  • Expose annotation dependencies through the apiElements variant for Gradle consumers.
  • Add JPMS requires static directives for annotation modules.
  • Add a changelog entry for issue [main] Add transitive compileOnlyApi dependencies #3600.

Validation

  • Maven reactor package and install completed successfully using JDK 17.
  • Verified generated Gradle Module Metadata for log4j-core, log4j-kit, log4j-plugins, and log4j-plugin-processor.
  • Verified the log4j-core module descriptor contains the expected requires static directives.
  • git diff --check passed.

Notes

No new automated tests were added in this contribution.

@ramanathan1504 ramanathan1504 added build Affects the build system or other automation dependencies Related to third party dependency updates or migrations port-done No further port needed: all required ports are merged, or none was needed labels Nov 10, 2026
@ramanathan1504

Copy link
Copy Markdown
Contributor

The generated module.json and module descriptors for log4j-core, log4j-kit and log4j-plugins match #3450 on 2.x.
#4034 asks to add these libraries only to the modules that use them. @ppkarwasz do we port the 2.x list to main as is, or settle #4034 first?

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

Labels

build Affects the build system or other automation dependencies Related to third party dependency updates or migrations port-done No further port needed: all required ports are merged, or none was needed

Projects

Development

Successfully merging this pull request may close these issues.

2 participants