[GitHub] [maven] laeubi commented on pull request #1061: [MNG-7038] Introduce public property to point to a root directory of (multi-module) project

["laeubi (via GitHub)" <gi...@apache.org>]

laeubi commented on PR #1061:
URL: https://github.com/apache/maven/pull/1061#issuecomment-1500421719

   > > I do not feel very comfortable to add runtime information into a static model. This does not feel right. Maybe we should just require `.mvn` to be present now, period?
   > 
   > I'll let you discuss with @rmannibucau about that, because that's precisely the very point he disagree with...
   
   My feeling is that hes the only one disagreeing here, so maybe we can not make everyone happy ;-)
   Beside that i find "root" confusing if it is only used to define a path but has no other meaning.
   
   Also as an excerpt from the discussion, I think that almost all use-cases would better to have a property that has the full path of the current **pom.xml file** (!).
   
   So lets say I have a parent `pom.xml` and I use the property `${this.location}` it should always resolve to the path of the parent pom even if used in a child at any level. Thats simple and can be used independent of the project structure.
   
   


-- 
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