[GitHub] [maven-site] kwin commented on a change in pull request #205: [MNG-6994] clarify repository order

2020-10-04 Thread GitBox


kwin commented on a change in pull request #205:
URL: https://github.com/apache/maven-site/pull/205#discussion_r499264802



##
File path: content/apt/guides/introduction/introduction-to-profiles.apt
##
@@ -403,6 +403,56 @@ mvn groupId:artifactId:goal -P !profile-1,!profile-2
   One of the goals in Maven 2 is to consolidate all the information needed to

Review comment:
   This is a section I didn't touch at all. Do you really want me to do 
changes on unrelated and unchanged sections?





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:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] [maven-site] kwin commented on a change in pull request #205: [MNG-6994] clarify repository order

2020-10-04 Thread GitBox


kwin commented on a change in pull request #205:
URL: https://github.com/apache/maven-site/pull/205#discussion_r499259005



##
File path: content/apt/guides/mini/guide-multiple-repositories.apt
##
@@ -102,3 +104,15 @@ mvn -Pmyprofile ...
  activate multiple profiles simultaneously.
  
  <>: The settings descriptor documentation can be found on the 
{{{../../maven-settings/settings.html}Maven Local Settings Model Website}}.
+ 
+ ** Repository Order
+
+Remote repository URLs are queried in the following order for artifacts until 
one returns a valid result:
+
+[[1]] Global <<>>
+[[1]] User <<>>
+[[1]] Local POM
+[[1]] Parent POMs, recursively
+[[1]] Super POM
+
+For each of these locations the repositories within the profiles are queried 
first in the order outlined at 
{{{../introduction/introduction-to-profiles.html}Introduction to build 
profiles}}.

Review comment:
   done

##
File path: content/apt/guides/mini/guide-multiple-repositories.apt
##
@@ -55,8 +55,10 @@ Setting up Multiple Repositories
  <> You will also get the standard set of repositories as defined in 
the 
  {{{../introduction/introduction-to-the-pom.html#Super_POM}Super POM}}.
 
+
+
  The other way you can specify the use of multiple repositories by creating a 
profile in
- your <<<$\{user.home\}/.m2/settings.xml>>> file like the following:
+ your <<<$\{user.home\}/.m2/settings.xml>>> or 
<<<$\{maven.home\}/conf/settings.xml>>> file like the following:

Review comment:
   done





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:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] [maven-site] kwin commented on a change in pull request #205: [MNG-6994] clarify repository order

2020-10-04 Thread GitBox


kwin commented on a change in pull request #205:
URL: https://github.com/apache/maven-site/pull/205#discussion_r499258979



##
File path: content/apt/guides/mini/guide-multiple-repositories.apt
##
@@ -29,7 +29,7 @@
 Setting up Multiple Repositories
 
  There are two different ways that you can specify the use of multiple 
repositories. The first
- way is to specify in a POM which repositories you want to use:
+ way is to specify in a POM which repositories you want to use (both inside 
and outside of build profiles):

Review comment:
   done





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:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] [maven-site] kwin commented on a change in pull request #205: [MNG-6994] clarify repository order

2020-10-04 Thread GitBox


kwin commented on a change in pull request #205:
URL: https://github.com/apache/maven-site/pull/205#discussion_r499258960



##
File path: content/apt/guides/introduction/introduction-to-profiles.apt
##
@@ -403,6 +403,56 @@ mvn groupId:artifactId:goal -P !profile-1,!profile-2
   One of the goals in Maven 2 is to consolidate all the information needed to
   run a build into a single file, or file hierarchy which is the POM.
 
+* Profile Order
+
+All profile elements in a POM from active profiles overwrite/enrich the global 
elements with the same name of the POM (i.e. take precedence). 

Review comment:
   done





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:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



[GitHub] [maven-site] kwin commented on a change in pull request #205: MNG-6994 clarify repository order

2020-10-04 Thread GitBox


kwin commented on a change in pull request #205:
URL: https://github.com/apache/maven-site/pull/205#discussion_r499223023



##
File path: content/apt/guides/mini/guide-multiple-repositories.apt
##
@@ -52,11 +52,15 @@ Setting up Multiple Repositories
 
 ++
 
+You can give repositories in a POM also in profiles.

Review comment:
   Fixed!





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:
us...@infra.apache.org



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org