Skip to content

IT: verify annotation processor in the same reactor can be used by a sibling module - #459

Draft
gnodet wants to merge 1 commit into
apache:maven-3.10.xfrom
gnodet:feat/reactor-ap-it
Draft

gnodet wants to merge 1 commit into
apache:maven-3.10.xfrom
gnodet:feat/reactor-ap-it

Conversation

@gnodet

@gnodet gnodet commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Add MavenITgh13911ReactorAnnotationProcessorTest to verify that an annotation processor built from source in the same Maven reactor can be used by a sibling module's compile phase on Maven 3.10.x.

Background

See apache/maven#13914 for the full context. This is the 3.10.x companion to that PR.

Motivated by a report that this scenario didn't work with Maven 3.8.x. This test confirms it works from Maven 3.9.x onwards (3.10.x included), using a provided POM dependency for reactor ordering and <annotationProcessorPaths> in maven-compiler-plugin.

Test design

Two-module reactor:

  • processor/ — jar with a Gh13911Processor that writes META-INF/generated-by-gh13911.txt for any @Gh13911Generated-annotated type
  • client/ — jar with processor as a provided dep and <annotationProcessorPaths> config; verifies the marker file exists after compile

Fixes apache/maven#13911

Hermès Agent (Claude Sonnet 4.6) on behalf of Guillaume Nodet

…sibling module

Adds MavenITgh13911ReactorAnnotationProcessorTest (sequential builder only,
as Maven 3.x has no concurrent builder) with a two-module reactor:
- processor/ — jar with @Gh13911Generated annotation and processor that
  writes META-INF/generated-by-gh13911.txt to the class output directory
- client/ — jar that declares processor as provided dep and configures
  it via <annotationProcessorPaths>

Companion to apache/maven#13914.
Fixes apache/maven#13911

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
@gnodet
gnodet force-pushed the feat/reactor-ap-it branch from 96bd7c0 to de9115f Compare November 9, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant