gnodet commented on PR #1061:
URL: https://github.com/apache/maven/pull/1061#issuecomment-1500435764
> > > 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.
Some of the use cases involve arguments interpolation, so this has to be
done _very early_ in the process.
Also, I disagree that this is _weird_ somehow. I think it's actually
mapping quite well with the property `isRootProject()` which has already been
added to the `Project`, and actually nicely reflects the correct semantic and
what we're looking for.
--
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]
