maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hervé Boutemy (JIRA) <>
Subject [jira] [Commented] (MNG-6059) Important use cases not covered, as child.inherit.append.path affects all children
Date Sat, 09 Jul 2016 15:25:11 GMT


Hervé Boutemy commented on MNG-6059:

trying to work back on this, a good number of monthes after the initial implementation:
new eyes on my own work, re-discovering it, and seeing if it is really usable/clear (and where
are misunderstandings)

First, there not only 1 {{child.inherit.append.path}} attribute, but 3:
1. one on project for {{project.url}}:
2. one on project.scm for {{project.scm.connection}}, {{project.scm.developerConnection}}
and {{project.scm.url}}:
3. one on for {{}}:

IIUC your issue, it's one the second one that affects the 3 project.scm urls consistently
where you'd need something else

My own issue is that the documentation talks 3 times about {{child.urls.inherit.append.path="false"}}:
this is not the attribute name (notice the ".urls"), and this does not help understanding
that there are 3 different attributes.
Clearly, the attribute names will have to be reworked and model documentation will have to
be modified accordingly

then regarding project.scm: I'm not convinced at all with Robert's analysis :)
to me, Maven's git SCM url should be improved to support naming a path into a git repo (or
any SCM that separates the repo location and the path inside the repo)
for example: {{<connection>scm:git:git://</connection>}}
this would improve SCM support, instead of removing useful feature

but this improvement is not in Maven core: it's in maven-scm
And it will perhaps require updates in MPIR plugin, or even maven-scm api, to clearly separate
repo location from path inside the repo (something that is completely hidden in CVS or svn)

> Important use cases not covered, as child.inherit.append.path affects all children
> ----------------------------------------------------------------------------------
>                 Key: MNG-6059
>                 URL:
>             Project: Maven
>          Issue Type: Bug
>          Components: Inheritance and Interpolation
>         Environment: Apache Maven 3.4.0-SNAPSHOT (227085283b6379038ec16f4cf9ad2e8869cef694;
>            Reporter: Andreas Sewe
> The {{child.inherit.append.path}} attribute introduced with MNG-5951 unfortunately does
not support the use case where the children of the element with the attribute should follow
different inheritance rules. Take a typical configuration for Github, for example (taken from
> {noformat}
> <scm>
>   <connection>scm:git:git://</connection>
>   <developerConnection>scm:git:ssh://</developerConnection>
>   <url></url>
> </scm>
> {noformat}
> If the {{ossrh-demo.git}} repository contains a child module called {{some-module}},
then that child’s {{scm/url}} should become {{}}
as per the normal inheritance rules, but both the {{scm/connection}} and {{scm/developerConnection}}
URLs should remain unchanged.
> Unfortunately, this is not possible with {{*child*.inherit.append.path}}, which acts
on all children simultaneously.
> IMHO, this is a conceptual problem. In particular, setting {{child.inherit.append.path}}
on the *root* element to just control a single child ({{project/url}}) feels wrong, as the
attribute is in all likelihood not even located close to the {{<url>}} element it controls.

This message was sent by Atlassian JIRA

View raw message