gnodet opened a new pull request, #13452:
URL: https://github.com/apache/maven/pull/13452

   ## Summary
   
   Backport of #13450 to `maven-4.0.x`.
   
   - A dependency POM's own `<properties>` must not activate its property-based 
profiles
   - The sandbox was merging model properties into the system-property lookup, 
causing POMs like jruby-base to self-activate profiles unintentionally (#13390)
   - Remove the merge — only system properties and negated-property checks 
drive profile activation in external builds, restoring 3.9.x semantics
   - This does **not** affect #13084 (RESTEasy), which uses a negated-property 
profile (`!resteasy.dependencies.eap`) that works correctly without the merge
   
   ### Changes
   
   **Both stacks** (compat `maven-model-builder` and new `impl/maven-impl`):
   - `DefaultModelBuilder` / `DefaultProfileActivationContext`: remove 
POM-property merge from external-model sandbox
   - `PropertyProfileActivator`: update comments
   - `ProfileActivationContext` interfaces: update Javadoc on model/project 
properties
   
   **Tests**: added jruby-style (#13390 regression) and resteasy-style (#13084 
non-regression) test cases
   
   ## Test plan
   
   - [x] Compat `ExternalModelProfileActivationTest` — 12 tests pass
   - [ ] Impl `DefaultModelBuilderTest` — has pre-existing compile failure on 
4.0.x (unrelated)
   - [ ] Integration test with jruby-base-9.4.2.0 dependency (manual 
verification)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to