You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "gnodet (via GitHub)" <gi...@apache.org> on 2025/12/08 12:11:44 UTC

[PR] Allow ${project.basedir} in profile activation.condition [maven]

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

   Previously, `${project.basedir}` was only allowed in `activation.file.exists` and `activation.file.missing`. This change extends the same allowance to `activation.condition`, which is a new Maven 4.0.0 feature that also needs to reference the project base directory for its expressions.
   
   This fixes a false positive warning when using expressions like `exists("${project.basedir}/src/main/java")` in activation conditions.
   
   ### Changes
   - Updated `DefaultModelValidator` to allow `${project.basedir}` in `activation.condition` path
   - Added unit test for the new behavior
   
   ---
   Pull Request opened by [Augment Code](https://www.augmentcode.com/) with guidance from the PR author


-- 
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: issues-unsubscribe@maven.apache.org

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

Re: [PR] Allow ${project.basedir} in profile activation.condition [maven]

Posted by "gnodet (via GitHub)" <gi...@apache.org>.
gnodet commented on PR #11528:
URL: https://github.com/apache/maven/pull/11528#issuecomment-3634013330

   ## 💚 All backports created successfully
   
   | Status | Branch | Result |
   |:------:|:------:|:------|
   |✅|master|[<img src="https://img.shields.io/github/pulls/detail/state/apache/maven/11531">](https://github.com/apache/maven/pull/11531)|
   
   ### Questions ?
   Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport)
   
   <!--- Backport version: 10.0.1 -->


-- 
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: issues-unsubscribe@maven.apache.org

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

Re: [PR] Allow ${project.basedir} in profile activation.condition [maven]

Posted by "gnodet (via GitHub)" <gi...@apache.org>.
gnodet merged PR #11528:
URL: https://github.com/apache/maven/pull/11528


-- 
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: issues-unsubscribe@maven.apache.org

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