Skip to content

[WFCORE-7380] Remove JDK 23 specific profile and enable the annotation processor in the main compiler configuration#6842

Open
javideros wants to merge 1 commit into
wildfly:mainfrom
Deloitte:WFCORE-7380
Open

[WFCORE-7380] Remove JDK 23 specific profile and enable the annotation processor in the main compiler configuration#6842
javideros wants to merge 1 commit into
wildfly:mainfrom
Deloitte:WFCORE-7380

Conversation

@javideros

@javideros javideros commented Jul 8, 2026

Copy link
Copy Markdown

Fixes: https://issues.redhat.com/browse/WFCORE-7380

Removes the jdk23 profile and enables the jboss-logging annotation processor in the main compiler configuration via annotationProcessorPaths in the root pom's pluginManagement, as discussed on the JIRA:

  • The per-module provided jboss-logging-processor dependency is removed from the 31 modules that declared it; the processor path in pluginManagement now applies to all modules.
  • The jboss-logging-processor entry in dependencyManagement is removed (no BOM safety net, as agreed).
  • patching keeps compiling via an explicit jboss-logging-annotations dependency (the annotations previously arrived transitively through the processor).

Verified with full clean builds on JDK 17, 21 and 25; annotation-processor output confirmed identical across all three.

Counterpart of WFLY-21910 on the WildFly side.

Relates to wildfly/wildfly#20205

…n processor in the main compiler configuration

Signed-off-by: javideros <javideros@gmail.com>
@wildfly-ci

Copy link
Copy Markdown

Hello, javideros. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

@yersan

yersan commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

/ok-to-test

@wildfly-ci

Copy link
Copy Markdown

Core -> Full Integration Build 15246 outcome was FAILURE using a merge of d445501
Summary: Exit code 1 (Step: Build and test full with JDK 21 (Maven)) (new) Build time: 00:04:33

@wildfly-ci

Copy link
Copy Markdown

Core -> Full Integration (with SecurityManager) Build 246 outcome was FAILURE using a merge of d445501
Summary: Exit code 1 (Step: Build and test full with JDK 17 (Maven)) (new) Build time: 00:05:07

@wildfly-ci

Copy link
Copy Markdown

Core -> WildFly Preview Integration Build 252 outcome was FAILURE using a merge of d445501
Summary: Exit code 1 (Step: Build & test full (Maven)) (new) Build time: 00:07:13

@yersan

yersan commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Just for the record, we can forget the failures on the "Integration" Jobs, we need the WildFly counterpart PR merged first to proceed with this one.

@wildfly-ci

This comment was marked as outdated.

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.

3 participants