Skip to content

Add missing ASF license header to subscription-emulation logback-test.xml#2646

Closed
ascheman wants to merge 1 commit into
apache:developfrom
ascheman:bugfix/rat-subscription-emulation-logback-header
Closed

Add missing ASF license header to subscription-emulation logback-test.xml#2646
ascheman wants to merge 1 commit into
apache:developfrom
ascheman:bugfix/rat-subscription-emulation-logback-header

Conversation

@ascheman

Copy link
Copy Markdown

Building PLC4X from a fresh checkout of develop currently fails after ~30 s in the root module's apache-rat:check: plc4j/utils/subscription-emulation/src/test/resources/logback-test.xml is the single file without an ASF license header (RAT: Unapproved: 1).

This adds the same header the sibling logback-test.xml files (modbus, eip, profinet-ng, …) carry. Verified locally: mvn -N apache-rat:checkUnapproved: 0, BUILD SUCCESS.

Found while testing PLC4X against a current Maven 4.0.0-SNAPSHOT (pre-RC-6) — thanks for adopting Maven 4 so early! 🙏

🤖 Generated with Claude Code

plc4j/utils/subscription-emulation/src/test/resources/logback-test.xml
was the only file in the tree without a license header, failing the
apache-rat check for every clean checkout of the main branch.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ascheman

Copy link
Copy Markdown
Author

Superseded by 6a72ab5 — thanks for the quick direct fix, Chris! 🙂 (Answering the commit message's question: RAT only failed from a clean checkout / mvn -N apache-rat:check; incremental local builds may have skipped the root rat execution.) A follow-up PR for Maven 4 RC-6 readiness is coming.

@ascheman ascheman closed this Jul 16, 2026
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