Re: maven-wagon git commit: [WAGON-471] Clean up dependency mess (reported by dependency:analyze)

2016-12-25 Thread Michael Osipov 

Am 2016-12-25 um 21:34 schrieb Dan Tran:

is it necessary to introduce


 org.slf4j
 slf4j-simple
 

as compile scope?


It is not in compile scope. Please have a look into dependencyManagement 
in parent. The default scope for this dependency is test:

https://git-wip-us.apache.org/repos/asf?p=maven-wagon.git;a=blob;f=pom.xml;h=e127da4bec14c4312511e7d74b935551da5e5fa6;hb=HEAD#l291


On Sun, Dec 25, 2016 at 12:10 PM,  wrote:


Repository: maven-wagon
Updated Branches:
  refs/heads/master 4da2accfd -> b451e418e


[WAGON-471] Clean up dependency mess (reported by dependency:analyze)

Followup fix for SLF4J Simple in test scope for file, scm and
http-lightweight.


Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/b451e418
Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/b451e418
Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/b451e418

Branch: refs/heads/master
Commit: b451e418ed5235f231f5288fd21054f450669aa9
Parents: 4da2acc
Author: Michael Osipov 
Authored: Sun Dec 25 21:09:43 2016 +0100
Committer: Michael Osipov 
Committed: Sun Dec 25 21:09:43 2016 +0100

--
 wagon-providers/wagon-file/pom.xml | 4 
 wagon-providers/wagon-http-lightweight/pom.xml | 4 
 wagon-providers/wagon-scm/pom.xml  | 4 
 3 files changed, 12 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
b451e418/wagon-providers/wagon-file/pom.xml
--
diff --git a/wagon-providers/wagon-file/pom.xml
b/wagon-providers/wagon-file/pom.xml
index 4e3cb52..a96af63 100644
--- a/wagon-providers/wagon-file/pom.xml
+++ b/wagon-providers/wagon-file/pom.xml
@@ -38,5 +38,9 @@ under the License.
   org.codehaus.plexus
   plexus-utils
 
+
+  org.slf4j
+  slf4j-simple
+
   
 

http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
b451e418/wagon-providers/wagon-http-lightweight/pom.xml
--
diff --git a/wagon-providers/wagon-http-lightweight/pom.xml
b/wagon-providers/wagon-http-lightweight/pom.xml
index 9a0e8c8..f723004 100644
--- a/wagon-providers/wagon-http-lightweight/pom.xml
+++ b/wagon-providers/wagon-http-lightweight/pom.xml
@@ -64,5 +64,9 @@ under the License.
   jetty
   test
 
+
+  org.slf4j
+  slf4j-simple
+
   
 

http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
b451e418/wagon-providers/wagon-scm/pom.xml
--
diff --git a/wagon-providers/wagon-scm/pom.xml
b/wagon-providers/wagon-scm/pom.xml
index 55c14e6..881040a 100644
--- a/wagon-providers/wagon-scm/pom.xml
+++ b/wagon-providers/wagon-scm/pom.xml
@@ -65,6 +65,10 @@ under the License.
   ${mavenScmVersion}
   test
 
+
+  org.slf4j
+  slf4j-simple
+
   

   







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

Re: maven-wagon git commit: [WAGON-471] Clean up dependency mess (reported by dependency:analyze)

2016-12-25 Thread Dan Tran 
is it necessary to introduce


 org.slf4j
 slf4j-simple
 

as compile scope?

Thanks

-Dan

On Sun, Dec 25, 2016 at 12:10 PM,  wrote:

> Repository: maven-wagon
> Updated Branches:
>   refs/heads/master 4da2accfd -> b451e418e
>
>
> [WAGON-471] Clean up dependency mess (reported by dependency:analyze)
>
> Followup fix for SLF4J Simple in test scope for file, scm and
> http-lightweight.
>
>
> Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo
> Commit: http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/b451e418
> Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/b451e418
> Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/b451e418
>
> Branch: refs/heads/master
> Commit: b451e418ed5235f231f5288fd21054f450669aa9
> Parents: 4da2acc
> Author: Michael Osipov 
> Authored: Sun Dec 25 21:09:43 2016 +0100
> Committer: Michael Osipov 
> Committed: Sun Dec 25 21:09:43 2016 +0100
>
> --
>  wagon-providers/wagon-file/pom.xml | 4 
>  wagon-providers/wagon-http-lightweight/pom.xml | 4 
>  wagon-providers/wagon-scm/pom.xml  | 4 
>  3 files changed, 12 insertions(+)
> --
>
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
> b451e418/wagon-providers/wagon-file/pom.xml
> --
> diff --git a/wagon-providers/wagon-file/pom.xml
> b/wagon-providers/wagon-file/pom.xml
> index 4e3cb52..a96af63 100644
> --- a/wagon-providers/wagon-file/pom.xml
> +++ b/wagon-providers/wagon-file/pom.xml
> @@ -38,5 +38,9 @@ under the License.
>org.codehaus.plexus
>plexus-utils
>  
> +
> +  org.slf4j
> +  slf4j-simple
> +
>
>  
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
> b451e418/wagon-providers/wagon-http-lightweight/pom.xml
> --
> diff --git a/wagon-providers/wagon-http-lightweight/pom.xml
> b/wagon-providers/wagon-http-lightweight/pom.xml
> index 9a0e8c8..f723004 100644
> --- a/wagon-providers/wagon-http-lightweight/pom.xml
> +++ b/wagon-providers/wagon-http-lightweight/pom.xml
> @@ -64,5 +64,9 @@ under the License.
>jetty
>test
>  
> +
> +  org.slf4j
> +  slf4j-simple
> +
>
>  
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
> b451e418/wagon-providers/wagon-scm/pom.xml
> --
> diff --git a/wagon-providers/wagon-scm/pom.xml
> b/wagon-providers/wagon-scm/pom.xml
> index 55c14e6..881040a 100644
> --- a/wagon-providers/wagon-scm/pom.xml
> +++ b/wagon-providers/wagon-scm/pom.xml
> @@ -65,6 +65,10 @@ under the License.
>${mavenScmVersion}
>test
>  
> +
> +  org.slf4j
> +  slf4j-simple
> +
>
>
>
>
>