You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by GitBox <gi...@apache.org> on 2021/02/12 07:15:29 UTC
[GitHub] [maven-site] hboutemy opened a new pull request #228: [MNGSITE-430] add dependency POMs
hboutemy opened a new pull request #228:
URL: https://github.com/apache/maven-site/pull/228
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] [maven-site] hboutemy commented on pull request #228: [MNGSITE-430] add dependency POMs
Posted by GitBox <gi...@apache.org>.
hboutemy commented on pull request #228:
URL: https://github.com/apache/maven-site/pull/228#issuecomment-778500519
yes, we can add such an image in a future step
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] [maven-site] hboutemy commented on pull request #228: [MNGSITE-430] add dependency POMs
Posted by GitBox <gi...@apache.org>.
hboutemy commented on pull request #228:
URL: https://github.com/apache/maven-site/pull/228#issuecomment-778125511
@rfscholte local POM does not have any resolution: it is locally on disk, from Git
resolution is always for dependencies
> transitive dependencies are never effected by repos of their calling pom.
they are affected by repos of their calling pom, that is exactly what resolution order is about
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] [maven-site] rfscholte commented on pull request #228: [MNGSITE-430] add dependency POMs
Posted by GitBox <gi...@apache.org>.
rfscholte commented on pull request #228:
URL: https://github.com/apache/maven-site/pull/228#issuecomment-778141870
The last description is still not clear for me. I think what we're trying to say: repositories within context, i.e the reposities of the pom that defined this pom as a dependency.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] [maven-site] rfscholte commented on pull request #228: [MNGSITE-430] add dependency POMs
Posted by GitBox <gi...@apache.org>.
rfscholte commented on pull request #228:
URL: https://github.com/apache/maven-site/pull/228#issuecomment-778408664
Sometimes images say more than words. Looks like this one should have a picture of a graph to support the logic.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] [maven-site] asfgit merged pull request #228: [MNGSITE-430] add dependency POMs
Posted by GitBox <gi...@apache.org>.
asfgit merged pull request #228:
URL: https://github.com/apache/maven-site/pull/228
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] [maven-site] hboutemy edited a comment on pull request #228: [MNGSITE-430] add dependency POMs
Posted by GitBox <gi...@apache.org>.
hboutemy edited a comment on pull request #228:
URL: https://github.com/apache/maven-site/pull/228#issuecomment-778125511
@rfscholte local POM does not have any resolution: it is locally on disk, from Git
resolution is always for dependencies
> transitive dependencies are never effected by repos of their calling pom.
they are affected by repos of their calling pom, that is exactly what resolution order is about
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] [maven-site] rfscholte commented on pull request #228: [MNGSITE-430] add dependency POMs
Posted by GitBox <gi...@apache.org>.
rfscholte commented on pull request #228:
URL: https://github.com/apache/maven-site/pull/228#issuecomment-778089075
I suggest to split this up into two:
- The repository resolution order for a local/reactor pom
- The repository resolution order for a dependency pom
It must be very clear that transitive dependencies are never effected by repos of their calling pom.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[GitHub] [maven-site] hboutemy commented on pull request #228: [MNGSITE-430] add dependency POMs
Posted by GitBox <gi...@apache.org>.
hboutemy commented on pull request #228:
URL: https://github.com/apache/maven-site/pull/228#issuecomment-778379785
yes, the last description is the hard one where we need creativity...
"POMs from dependency path to the artifact"?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org