You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by "ramanathan1504 (via GitHub)" <gi...@apache.org> on 2026/09/09 14:33:03 UTC

[PR] Fix `ThreadContextTestAccess` out of the `org.apache.logging.log4j.core` package [logging-log4j2]

ramanathan1504 opened a new pull request, #4315:
URL: https://github.com/apache/logging-log4j2/pull/4315

   `log4j-core-test` had one class directly in `org.apache.logging.log4j.core`, the package `log4j-core` also exports. JPMS forbids reading one package from two modules, so a   
     consumer requiring both fails:                                                                                                                                                
                                                                                                                                                                                   
         module org.apache.logging.log4j.samples.nashorn reads package                                                                                                             
         org.apache.logging.log4j.core from both org.apache.logging.log4j.core                                                                                                     
         and org.apache.logging.log4j.core.test                                                                                                                                    
                                                                                                                                                                                   
     That is what fails `integration-test (3.0.0-SNAPSHOT)` on `log4j-nashorn-test`. `2.x` has no class in that package.                                                           
                                                                                                                                                                                   
     The class only touches public API and reaches `resetThreadContextMap` by reflection, so the placement is vestigial. One call site, updated here.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix `ThreadContextTestAccess` out of the `org.apache.logging.log4j.core` package [logging-log4j2]

Posted by "ramanathan1504 (via GitHub)" <gi...@apache.org>.
ramanathan1504 merged PR #4315:
URL: https://github.com/apache/logging-log4j2/pull/4315


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org