You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Arnaud Héritier <ah...@apache.org> on 2015/04/01 02:46:59 UTC
Unreadable commits notifications was Re: [49/50] [abbrv] maven git
commit: Package and configure log4J 2.2 by default. Replace the content of
conf/logging/log4j2.xml by the one from conf/logging/log4j2-color.xml to
enjoy the colorised console
Hi all,
Is it normal to have so much (unreadable) notifications when we rewrite a
branch ?
I updated, cleaned and rewrote the history of the slf4j-log4j2 branch
https://github.com/apache/maven/commits/slf4j-log4j2
I found this better to rewrite such WIP branch than creating a new one
(and deleting this one)
Note that that are several old branches on the same subject that can be
trashed I think :
https://github.com/apache/maven/commits/feature/colorized-console/log4j2
https://github.com/apache/maven/commits/logging/slf4j-log4j2
I think it is important to be notified when a branch is rewritten but
having 50 emails when I replaced an old branch with < 10 commits by a
sanitised one of 2 commits, I don't understand.
Cheers
Arnaud
On Wed, Apr 1, 2015 at 2:24 AM, <ah...@apache.org> wrote:
> Package and configure log4J 2.2 by default.
> Replace the content of conf/logging/log4j2.xml by the one from
> conf/logging/log4j2-color.xml to enjoy the colorised console
>
>
> Project: http://git-wip-us.apache.org/repos/asf/maven/repo
> Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/dbad2e53
> Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/dbad2e53
> Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/dbad2e53
>
> Branch: refs/heads/slf4j-log4j2
> Commit: dbad2e536a7024a277eef1c56eaa2286f9f2a7f9
> Parents: f78742f
> Author: Arnaud Héritier <ah...@apache.org>
> Authored: Wed Apr 1 02:16:56 2015 +0200
> Committer: Arnaud Héritier <ah...@apache.org>
> Committed: Wed Apr 1 02:16:56 2015 +0200
>
> ----------------------------------------------------------------------
> apache-maven/pom.xml | 15 +++++++-
> apache-maven/src/conf/logging/log4j2-color.xml | 36 ++++++++++++++++++++
> apache-maven/src/conf/logging/log4j2.xml | 36 ++++++++++++++++++++
> maven-embedder/pom.xml | 8 +++++
> .../maven/slf4j-configuration.properties | 2 +-
> pom.xml | 31 +++++++++++++++--
> 6 files changed, 123 insertions(+), 5 deletions(-)
> ----------------------------------------------------------------------
>
>
>
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/pom.xml
> ----------------------------------------------------------------------
> diff --git a/apache-maven/pom.xml b/apache-maven/pom.xml
> index e8277b5..c3e9e49 100644
> --- a/apache-maven/pom.xml
> +++ b/apache-maven/pom.xml
> @@ -95,7 +95,20 @@
> </dependency>
> <dependency>
> <groupId>org.slf4j</groupId>
> - <artifactId>slf4j-simple</artifactId>
> + <artifactId>slf4j-ext</artifactId>
> + </dependency>
> + <dependency>
> + <groupId>org.apache.logging.log4j</groupId>
> + <artifactId>log4j-slf4j-impl</artifactId>
> + </dependency>
> + <dependency>
> + <groupId>org.apache.logging.log4j</groupId>
> + <artifactId>log4j-core</artifactId>
> + </dependency>
> + <dependency>
> + <groupId>org.fusesource.jansi</groupId>
> + <artifactId>jansi</artifactId>
> + <scope>runtime</scope>
> </dependency>
> </dependencies>
>
>
>
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/src/conf/logging/log4j2-color.xml
> ----------------------------------------------------------------------
> diff --git a/apache-maven/src/conf/logging/log4j2-color.xml
> b/apache-maven/src/conf/logging/log4j2-color.xml
> new file mode 100644
> index 0000000..bea1e76
> --- /dev/null
> +++ b/apache-maven/src/conf/logging/log4j2-color.xml
> @@ -0,0 +1,36 @@
> +<?xml version="1.0" encoding="UTF-8" ?>
> +<!--
> + ~ Licensed to the Apache Software Foundation (ASF) under one
> + ~ or more contributor license agreements. See the NOTICE file
> + ~ distributed with this work for additional information
> + ~ regarding copyright ownership. The ASF licenses this file
> + ~ to you under the Apache License, Version 2.0 (the
> + ~ "License"); you may not use this file except in compliance
> + ~ with the License. You may obtain a copy of the License at
> + ~
> + ~ http://www.apache.org/licenses/LICENSE-2.0
> + ~
> + ~ Unless required by applicable law or agreed to in writing,
> + ~ software distributed under the License is distributed on an
> + ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> + ~ KIND, either express or implied. See the License for the
> + ~ specific language governing permissions and limitations
> + ~ under the License.
> + -->
> +
> +
> +<configuration>
> + <properties>
> + <property name="maven.logging.root.level">INFO</property>
> + </properties>
> + <appenders>
> + <Console name="console" target="SYSTEM_OUT">
> + <PatternLayout pattern="%highlight{[%p{WARN=WARNING}]}
> %msg%n%throwable" />
> + </Console>
> + </appenders>
> + <loggers>
> + <root level="${sys:maven.logging.root.level}">
> + <appender-ref ref="console"/>
> + </root>
> + </loggers>
> +</configuration>
>
>
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/src/conf/logging/log4j2.xml
> ----------------------------------------------------------------------
> diff --git a/apache-maven/src/conf/logging/log4j2.xml
> b/apache-maven/src/conf/logging/log4j2.xml
> new file mode 100644
> index 0000000..6635597
> --- /dev/null
> +++ b/apache-maven/src/conf/logging/log4j2.xml
> @@ -0,0 +1,36 @@
> +<?xml version="1.0" encoding="UTF-8" ?>
> +<!--
> + ~ Licensed to the Apache Software Foundation (ASF) under one
> + ~ or more contributor license agreements. See the NOTICE file
> + ~ distributed with this work for additional information
> + ~ regarding copyright ownership. The ASF licenses this file
> + ~ to you under the Apache License, Version 2.0 (the
> + ~ "License"); you may not use this file except in compliance
> + ~ with the License. You may obtain a copy of the License at
> + ~
> + ~ http://www.apache.org/licenses/LICENSE-2.0
> + ~
> + ~ Unless required by applicable law or agreed to in writing,
> + ~ software distributed under the License is distributed on an
> + ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> + ~ KIND, either express or implied. See the License for the
> + ~ specific language governing permissions and limitations
> + ~ under the License.
> + -->
> +
> +
> +<configuration> <!--status="debug"-->
> + <properties>
> + <property name="maven.logging.root.level">INFO</property>
> + </properties>
> + <appenders>
> + <Console name="console" target="SYSTEM_OUT">
> + <PatternLayout pattern="[%p{WARN=WARNING}] %msg%n%throwable"/>
> + </Console>
> + </appenders>
> + <loggers>
> + <root level="${sys:maven.logging.root.level}">
> + <appender-ref ref="console"/>
> + </root>
> + </loggers>
> +</configuration>
>
>
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/maven-embedder/pom.xml
> ----------------------------------------------------------------------
> diff --git a/maven-embedder/pom.xml b/maven-embedder/pom.xml
> index 53f0724..ef8b935 100644
> --- a/maven-embedder/pom.xml
> +++ b/maven-embedder/pom.xml
> @@ -82,6 +82,10 @@
> </dependency>
> <dependency>
> <groupId>org.slf4j</groupId>
> + <artifactId>slf4j-ext</artifactId>
> + </dependency>
> + <dependency>
> + <groupId>org.slf4j</groupId>
> <artifactId>slf4j-simple</artifactId>
> <optional>true</optional>
> </dependency>
> @@ -90,6 +94,10 @@
> <artifactId>logback-classic</artifactId>
> <optional>true</optional>
> </dependency>
> + <dependency>
> + <groupId>org.apache.logging.log4j</groupId>
> + <artifactId>log4j-slf4j-impl</artifactId>
> + </dependency>
> <!-- CLI -->
> <dependency>
> <groupId>commons-cli</groupId>
>
>
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> ----------------------------------------------------------------------
> diff --git
> a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> index 8741836..cd01f9e 100644
> ---
> a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> +++
> b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> @@ -18,5 +18,5 @@
> # key = Slf4j effective logger factory implementation
> # value = corresponding o.a.m.cli.logging.Slf4jConfiguration class
> org.slf4j.impl.SimpleLoggerFactory
> org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
> -org.slf4j.helpers.Log4jLoggerFactory
> org.apache.maven.cli.logging.impl.Log4j2Configuration
> +org.apache.logging.slf4j.Log4jLoggerFactory
> org.apache.maven.cli.logging.impl.Log4j2Configuration
> ch.qos.logback.classic.LoggerContext
> org.apache.maven.cli.logging.impl.LogbackConfiguration
>
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/pom.xml
> ----------------------------------------------------------------------
> diff --git a/pom.xml b/pom.xml
> index b0ab4e8..87442f3 100644
> --- a/pom.xml
> +++ b/pom.xml
> @@ -61,6 +61,9 @@
> <jxpathVersion>1.3</jxpathVersion>
> <aetherVersion>1.0.2.v20150114</aetherVersion>
> <slf4jVersion>1.7.5</slf4jVersion>
> + <log4j2Version>2.2</log4j2Version>
> + <logbackVersion>1.0.7</logbackVersion>
> + <jansiVersion>1.11</jansiVersion>
>
> <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>
> <!-- Control the name of the distribution and information output by
> mvn -->
> <distributionId>apache-maven</distributionId>
> @@ -252,6 +255,7 @@
> <artifactId>plexus-interpolation</artifactId>
> <version>${plexusInterpolationVersion}</version>
> </dependency>
> + <!-- Logging -->
> <dependency>
> <groupId>org.slf4j</groupId>
> <artifactId>slf4j-api</artifactId>
> @@ -261,13 +265,34 @@
> <groupId>org.slf4j</groupId>
> <artifactId>slf4j-simple</artifactId>
> <version>${slf4jVersion}</version>
> - <optional>true</optional>
> </dependency>
> <dependency>
> <groupId>ch.qos.logback</groupId>
> <artifactId>logback-classic</artifactId>
> - <version>1.0.7</version>
> - <optional>true</optional>
> + <version>${logbackVersion}</version>
> + </dependency>
> + <dependency>
> + <groupId>org.slf4j</groupId>
> + <artifactId>slf4j-ext</artifactId>
> + <version>${slf4jVersion}</version>
> + <scope>compile</scope>
> + </dependency>
> + <dependency>
> + <groupId>org.apache.logging.log4j</groupId>
> + <artifactId>log4j-slf4j-impl</artifactId>
> + <version>${log4j2Version}</version>
> + <scope>compile</scope>
> + </dependency>
> + <dependency>
> + <groupId>org.apache.logging.log4j</groupId>
> + <artifactId>log4j-core</artifactId>
> + <version>${log4j2Version}</version>
> + </dependency>
> + <dependency>
> + <groupId>org.fusesource.jansi</groupId>
> + <artifactId>jansi</artifactId>
> + <version>${jansiVersion}</version>
> + <scope>runtime</scope>
> </dependency>
> <!-- Wagon -->
> <dependency>
>
>
--
Arnaud
Re: Unreadable commits notifications was Re: [49/50] [abbrv] maven
git commit: Package and configure log4J 2.2 by default. Replace the content
of conf/logging/log4j2.xml by the one from conf/logging/log4j2-color.xml to
enjoy the colorised console
Posted by Jeff MAURY <je...@jeffmaury.com>.
According to the commits archive, there is 1 email per commit and I can see
the 50/50 notification email (
http://mail-archives.apache.org/mod_mbox/maven-commits/201504.mbox/%3Cbfb2482f10c94b7180353e96763ee865%40git.apache.org%3E
)
Jeff
On Thu, Apr 2, 2015 at 10:17 AM, Arnaud Héritier <ah...@gmail.com>
wrote:
> yes but it seems we are receiving one email per commit
> Thus with my change in the branch we are receiving 48 notifications due to
> the rebase + 2 for real commits on this branch
> Note that myself I don't find in my inbox the 50/50 notification email
>
> On Wed, Apr 1, 2015 at 9:38 AM, Jeff MAURY <je...@jeffmaury.com>
> wrote:
>
> > On Wed, Apr 1, 2015 at 2:46 AM, Arnaud Héritier <ah...@apache.org>
> > wrote:
> >
> > > Hi all,
> > >
> > > Is it normal to have so much (unreadable) notifications when we
> > rewrite a
> > > branch ?
> > > I updated, cleaned and rewrote the history of the slf4j-log4j2 branch
> > >
> > > https://github.com/apache/maven/commits/slf4j-log4j2
> > >
> > > I found this better to rewrite such WIP branch than creating a new
> one
> > > (and deleting this one)
> > > Note that that are several old branches on the same subject that can
> be
> > > trashed I think :
> > >
> https://github.com/apache/maven/commits/feature/colorized-console/log4j2
> > > https://github.com/apache/maven/commits/logging/slf4j-log4j2
> > >
> > > I think it is important to be notified when a branch is rewritten but
> > > having 50 emails when I replaced an old branch with < 10 commits by a
> > > sanitised one of 2 commits, I don't understand.
> > >
> > Rewriting history does not reduce the amount of modified/deleted/added
> > code. And as the notification emails also send diffs, ....
> >
> > Jeff
> >
> > >
> > > Cheers
> > >
> > > Arnaud
> > >
> > > On Wed, Apr 1, 2015 at 2:24 AM, <ah...@apache.org> wrote:
> > >
> > > > Package and configure log4J 2.2 by default.
> > > > Replace the content of conf/logging/log4j2.xml by the one from
> > > > conf/logging/log4j2-color.xml to enjoy the colorised console
> > > >
> > > >
> > > > Project: http://git-wip-us.apache.org/repos/asf/maven/repo
> > > > Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/dbad2e53
> > > > Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/dbad2e53
> > > > Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/dbad2e53
> > > >
> > > > Branch: refs/heads/slf4j-log4j2
> > > > Commit: dbad2e536a7024a277eef1c56eaa2286f9f2a7f9
> > > > Parents: f78742f
> > > > Author: Arnaud Héritier <ah...@apache.org>
> > > > Authored: Wed Apr 1 02:16:56 2015 +0200
> > > > Committer: Arnaud Héritier <ah...@apache.org>
> > > > Committed: Wed Apr 1 02:16:56 2015 +0200
> > > >
> > > >
> ----------------------------------------------------------------------
> > > > apache-maven/pom.xml | 15 +++++++-
> > > > apache-maven/src/conf/logging/log4j2-color.xml | 36
> > > ++++++++++++++++++++
> > > > apache-maven/src/conf/logging/log4j2.xml | 36
> > > ++++++++++++++++++++
> > > > maven-embedder/pom.xml | 8 +++++
> > > > .../maven/slf4j-configuration.properties | 2 +-
> > > > pom.xml | 31
> +++++++++++++++--
> > > > 6 files changed, 123 insertions(+), 5 deletions(-)
> > > >
> ----------------------------------------------------------------------
> > > >
> > > >
> > > >
> > > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/pom.xml
> > > >
> ----------------------------------------------------------------------
> > > > diff --git a/apache-maven/pom.xml b/apache-maven/pom.xml
> > > > index e8277b5..c3e9e49 100644
> > > > --- a/apache-maven/pom.xml
> > > > +++ b/apache-maven/pom.xml
> > > > @@ -95,7 +95,20 @@
> > > > </dependency>
> > > > <dependency>
> > > > <groupId>org.slf4j</groupId>
> > > > - <artifactId>slf4j-simple</artifactId>
> > > > + <artifactId>slf4j-ext</artifactId>
> > > > + </dependency>
> > > > + <dependency>
> > > > + <groupId>org.apache.logging.log4j</groupId>
> > > > + <artifactId>log4j-slf4j-impl</artifactId>
> > > > + </dependency>
> > > > + <dependency>
> > > > + <groupId>org.apache.logging.log4j</groupId>
> > > > + <artifactId>log4j-core</artifactId>
> > > > + </dependency>
> > > > + <dependency>
> > > > + <groupId>org.fusesource.jansi</groupId>
> > > > + <artifactId>jansi</artifactId>
> > > > + <scope>runtime</scope>
> > > > </dependency>
> > > > </dependencies>
> > > >
> > > >
> > > >
> > > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/src/conf/logging/log4j2-color.xml
> > > >
> ----------------------------------------------------------------------
> > > > diff --git a/apache-maven/src/conf/logging/log4j2-color.xml
> > > > b/apache-maven/src/conf/logging/log4j2-color.xml
> > > > new file mode 100644
> > > > index 0000000..bea1e76
> > > > --- /dev/null
> > > > +++ b/apache-maven/src/conf/logging/log4j2-color.xml
> > > > @@ -0,0 +1,36 @@
> > > > +<?xml version="1.0" encoding="UTF-8" ?>
> > > > +<!--
> > > > + ~ Licensed to the Apache Software Foundation (ASF) under one
> > > > + ~ or more contributor license agreements. See the NOTICE file
> > > > + ~ distributed with this work for additional information
> > > > + ~ regarding copyright ownership. The ASF licenses this file
> > > > + ~ to you under the Apache License, Version 2.0 (the
> > > > + ~ "License"); you may not use this file except in compliance
> > > > + ~ with the License. You may obtain a copy of the License at
> > > > + ~
> > > > + ~ http://www.apache.org/licenses/LICENSE-2.0
> > > > + ~
> > > > + ~ Unless required by applicable law or agreed to in writing,
> > > > + ~ software distributed under the License is distributed on an
> > > > + ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > > + ~ KIND, either express or implied. See the License for the
> > > > + ~ specific language governing permissions and limitations
> > > > + ~ under the License.
> > > > + -->
> > > > +
> > > > +
> > > > +<configuration>
> > > > + <properties>
> > > > + <property name="maven.logging.root.level">INFO</property>
> > > > + </properties>
> > > > + <appenders>
> > > > + <Console name="console" target="SYSTEM_OUT">
> > > > + <PatternLayout pattern="%highlight{[%p{WARN=WARNING}]}
> > > > %msg%n%throwable" />
> > > > + </Console>
> > > > + </appenders>
> > > > + <loggers>
> > > > + <root level="${sys:maven.logging.root.level}">
> > > > + <appender-ref ref="console"/>
> > > > + </root>
> > > > + </loggers>
> > > > +</configuration>
> > > >
> > > >
> > > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/src/conf/logging/log4j2.xml
> > > >
> ----------------------------------------------------------------------
> > > > diff --git a/apache-maven/src/conf/logging/log4j2.xml
> > > > b/apache-maven/src/conf/logging/log4j2.xml
> > > > new file mode 100644
> > > > index 0000000..6635597
> > > > --- /dev/null
> > > > +++ b/apache-maven/src/conf/logging/log4j2.xml
> > > > @@ -0,0 +1,36 @@
> > > > +<?xml version="1.0" encoding="UTF-8" ?>
> > > > +<!--
> > > > + ~ Licensed to the Apache Software Foundation (ASF) under one
> > > > + ~ or more contributor license agreements. See the NOTICE file
> > > > + ~ distributed with this work for additional information
> > > > + ~ regarding copyright ownership. The ASF licenses this file
> > > > + ~ to you under the Apache License, Version 2.0 (the
> > > > + ~ "License"); you may not use this file except in compliance
> > > > + ~ with the License. You may obtain a copy of the License at
> > > > + ~
> > > > + ~ http://www.apache.org/licenses/LICENSE-2.0
> > > > + ~
> > > > + ~ Unless required by applicable law or agreed to in writing,
> > > > + ~ software distributed under the License is distributed on an
> > > > + ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > > + ~ KIND, either express or implied. See the License for the
> > > > + ~ specific language governing permissions and limitations
> > > > + ~ under the License.
> > > > + -->
> > > > +
> > > > +
> > > > +<configuration> <!--status="debug"-->
> > > > + <properties>
> > > > + <property name="maven.logging.root.level">INFO</property>
> > > > + </properties>
> > > > + <appenders>
> > > > + <Console name="console" target="SYSTEM_OUT">
> > > > + <PatternLayout pattern="[%p{WARN=WARNING}] %msg%n%throwable"/>
> > > > + </Console>
> > > > + </appenders>
> > > > + <loggers>
> > > > + <root level="${sys:maven.logging.root.level}">
> > > > + <appender-ref ref="console"/>
> > > > + </root>
> > > > + </loggers>
> > > > +</configuration>
> > > >
> > > >
> > > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/maven-embedder/pom.xml
> > > >
> ----------------------------------------------------------------------
> > > > diff --git a/maven-embedder/pom.xml b/maven-embedder/pom.xml
> > > > index 53f0724..ef8b935 100644
> > > > --- a/maven-embedder/pom.xml
> > > > +++ b/maven-embedder/pom.xml
> > > > @@ -82,6 +82,10 @@
> > > > </dependency>
> > > > <dependency>
> > > > <groupId>org.slf4j</groupId>
> > > > + <artifactId>slf4j-ext</artifactId>
> > > > + </dependency>
> > > > + <dependency>
> > > > + <groupId>org.slf4j</groupId>
> > > > <artifactId>slf4j-simple</artifactId>
> > > > <optional>true</optional>
> > > > </dependency>
> > > > @@ -90,6 +94,10 @@
> > > > <artifactId>logback-classic</artifactId>
> > > > <optional>true</optional>
> > > > </dependency>
> > > > + <dependency>
> > > > + <groupId>org.apache.logging.log4j</groupId>
> > > > + <artifactId>log4j-slf4j-impl</artifactId>
> > > > + </dependency>
> > > > <!-- CLI -->
> > > > <dependency>
> > > > <groupId>commons-cli</groupId>
> > > >
> > > >
> > > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > >
> ----------------------------------------------------------------------
> > > > diff --git
> > > >
> > >
> >
> a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > >
> > >
> >
> b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > > index 8741836..cd01f9e 100644
> > > > ---
> > > >
> > >
> >
> a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > > +++
> > > >
> > >
> >
> b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > > @@ -18,5 +18,5 @@
> > > > # key = Slf4j effective logger factory implementation
> > > > # value = corresponding o.a.m.cli.logging.Slf4jConfiguration class
> > > > org.slf4j.impl.SimpleLoggerFactory
> > > > org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
> > > > -org.slf4j.helpers.Log4jLoggerFactory
> > > > org.apache.maven.cli.logging.impl.Log4j2Configuration
> > > > +org.apache.logging.slf4j.Log4jLoggerFactory
> > > > org.apache.maven.cli.logging.impl.Log4j2Configuration
> > > > ch.qos.logback.classic.LoggerContext
> > > > org.apache.maven.cli.logging.impl.LogbackConfiguration
> > > >
> > > > http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/pom.xml
> > > >
> ----------------------------------------------------------------------
> > > > diff --git a/pom.xml b/pom.xml
> > > > index b0ab4e8..87442f3 100644
> > > > --- a/pom.xml
> > > > +++ b/pom.xml
> > > > @@ -61,6 +61,9 @@
> > > > <jxpathVersion>1.3</jxpathVersion>
> > > > <aetherVersion>1.0.2.v20150114</aetherVersion>
> > > > <slf4jVersion>1.7.5</slf4jVersion>
> > > > + <log4j2Version>2.2</log4j2Version>
> > > > + <logbackVersion>1.0.7</logbackVersion>
> > > > + <jansiVersion>1.11</jansiVersion>
> > > >
> > > >
> > >
> >
> <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>
> > > > <!-- Control the name of the distribution and information output
> > by
> > > > mvn -->
> > > > <distributionId>apache-maven</distributionId>
> > > > @@ -252,6 +255,7 @@
> > > > <artifactId>plexus-interpolation</artifactId>
> > > > <version>${plexusInterpolationVersion}</version>
> > > > </dependency>
> > > > + <!-- Logging -->
> > > > <dependency>
> > > > <groupId>org.slf4j</groupId>
> > > > <artifactId>slf4j-api</artifactId>
> > > > @@ -261,13 +265,34 @@
> > > > <groupId>org.slf4j</groupId>
> > > > <artifactId>slf4j-simple</artifactId>
> > > > <version>${slf4jVersion}</version>
> > > > - <optional>true</optional>
> > > > </dependency>
> > > > <dependency>
> > > > <groupId>ch.qos.logback</groupId>
> > > > <artifactId>logback-classic</artifactId>
> > > > - <version>1.0.7</version>
> > > > - <optional>true</optional>
> > > > + <version>${logbackVersion}</version>
> > > > + </dependency>
> > > > + <dependency>
> > > > + <groupId>org.slf4j</groupId>
> > > > + <artifactId>slf4j-ext</artifactId>
> > > > + <version>${slf4jVersion}</version>
> > > > + <scope>compile</scope>
> > > > + </dependency>
> > > > + <dependency>
> > > > + <groupId>org.apache.logging.log4j</groupId>
> > > > + <artifactId>log4j-slf4j-impl</artifactId>
> > > > + <version>${log4j2Version}</version>
> > > > + <scope>compile</scope>
> > > > + </dependency>
> > > > + <dependency>
> > > > + <groupId>org.apache.logging.log4j</groupId>
> > > > + <artifactId>log4j-core</artifactId>
> > > > + <version>${log4j2Version}</version>
> > > > + </dependency>
> > > > + <dependency>
> > > > + <groupId>org.fusesource.jansi</groupId>
> > > > + <artifactId>jansi</artifactId>
> > > > + <version>${jansiVersion}</version>
> > > > + <scope>runtime</scope>
> > > > </dependency>
> > > > <!-- Wagon -->
> > > > <dependency>
> > > >
> > > >
> > >
> > >
> > > --
> > >
> > > Arnaud
> > >
> >
> >
> >
> > --
> > Jeff MAURY
> >
> >
> > "Legacy code" often differs from its suggested alternative by actually
> > working and scaling.
> > - Bjarne Stroustrup
> >
> > http://www.jeffmaury.com
> > http://riadiscuss.jeffmaury.com
> > http://www.twitter.com/jeffmaury
> >
>
>
>
> --
> -----
> Arnaud Héritier
> http://aheritier.net
> Mail/GTalk: aheritier AT gmail DOT com
> Twitter/Skype : aheritier
>
--
Jeff MAURY
"Legacy code" often differs from its suggested alternative by actually
working and scaling.
- Bjarne Stroustrup
http://www.jeffmaury.com
http://riadiscuss.jeffmaury.com
http://www.twitter.com/jeffmaury
Re: Unreadable commits notifications was Re: [49/50] [abbrv] maven
git commit: Package and configure log4J 2.2 by default. Replace the content
of conf/logging/log4j2.xml by the one from conf/logging/log4j2-color.xml to
enjoy the colorised console
Posted by Arnaud Héritier <ah...@gmail.com>.
yes but it seems we are receiving one email per commit
Thus with my change in the branch we are receiving 48 notifications due to
the rebase + 2 for real commits on this branch
Note that myself I don't find in my inbox the 50/50 notification email
On Wed, Apr 1, 2015 at 9:38 AM, Jeff MAURY <je...@jeffmaury.com> wrote:
> On Wed, Apr 1, 2015 at 2:46 AM, Arnaud Héritier <ah...@apache.org>
> wrote:
>
> > Hi all,
> >
> > Is it normal to have so much (unreadable) notifications when we
> rewrite a
> > branch ?
> > I updated, cleaned and rewrote the history of the slf4j-log4j2 branch
> >
> > https://github.com/apache/maven/commits/slf4j-log4j2
> >
> > I found this better to rewrite such WIP branch than creating a new one
> > (and deleting this one)
> > Note that that are several old branches on the same subject that can be
> > trashed I think :
> > https://github.com/apache/maven/commits/feature/colorized-console/log4j2
> > https://github.com/apache/maven/commits/logging/slf4j-log4j2
> >
> > I think it is important to be notified when a branch is rewritten but
> > having 50 emails when I replaced an old branch with < 10 commits by a
> > sanitised one of 2 commits, I don't understand.
> >
> Rewriting history does not reduce the amount of modified/deleted/added
> code. And as the notification emails also send diffs, ....
>
> Jeff
>
> >
> > Cheers
> >
> > Arnaud
> >
> > On Wed, Apr 1, 2015 at 2:24 AM, <ah...@apache.org> wrote:
> >
> > > Package and configure log4J 2.2 by default.
> > > Replace the content of conf/logging/log4j2.xml by the one from
> > > conf/logging/log4j2-color.xml to enjoy the colorised console
> > >
> > >
> > > Project: http://git-wip-us.apache.org/repos/asf/maven/repo
> > > Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/dbad2e53
> > > Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/dbad2e53
> > > Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/dbad2e53
> > >
> > > Branch: refs/heads/slf4j-log4j2
> > > Commit: dbad2e536a7024a277eef1c56eaa2286f9f2a7f9
> > > Parents: f78742f
> > > Author: Arnaud Héritier <ah...@apache.org>
> > > Authored: Wed Apr 1 02:16:56 2015 +0200
> > > Committer: Arnaud Héritier <ah...@apache.org>
> > > Committed: Wed Apr 1 02:16:56 2015 +0200
> > >
> > > ----------------------------------------------------------------------
> > > apache-maven/pom.xml | 15 +++++++-
> > > apache-maven/src/conf/logging/log4j2-color.xml | 36
> > ++++++++++++++++++++
> > > apache-maven/src/conf/logging/log4j2.xml | 36
> > ++++++++++++++++++++
> > > maven-embedder/pom.xml | 8 +++++
> > > .../maven/slf4j-configuration.properties | 2 +-
> > > pom.xml | 31 +++++++++++++++--
> > > 6 files changed, 123 insertions(+), 5 deletions(-)
> > > ----------------------------------------------------------------------
> > >
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/apache-maven/pom.xml b/apache-maven/pom.xml
> > > index e8277b5..c3e9e49 100644
> > > --- a/apache-maven/pom.xml
> > > +++ b/apache-maven/pom.xml
> > > @@ -95,7 +95,20 @@
> > > </dependency>
> > > <dependency>
> > > <groupId>org.slf4j</groupId>
> > > - <artifactId>slf4j-simple</artifactId>
> > > + <artifactId>slf4j-ext</artifactId>
> > > + </dependency>
> > > + <dependency>
> > > + <groupId>org.apache.logging.log4j</groupId>
> > > + <artifactId>log4j-slf4j-impl</artifactId>
> > > + </dependency>
> > > + <dependency>
> > > + <groupId>org.apache.logging.log4j</groupId>
> > > + <artifactId>log4j-core</artifactId>
> > > + </dependency>
> > > + <dependency>
> > > + <groupId>org.fusesource.jansi</groupId>
> > > + <artifactId>jansi</artifactId>
> > > + <scope>runtime</scope>
> > > </dependency>
> > > </dependencies>
> > >
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/src/conf/logging/log4j2-color.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/apache-maven/src/conf/logging/log4j2-color.xml
> > > b/apache-maven/src/conf/logging/log4j2-color.xml
> > > new file mode 100644
> > > index 0000000..bea1e76
> > > --- /dev/null
> > > +++ b/apache-maven/src/conf/logging/log4j2-color.xml
> > > @@ -0,0 +1,36 @@
> > > +<?xml version="1.0" encoding="UTF-8" ?>
> > > +<!--
> > > + ~ Licensed to the Apache Software Foundation (ASF) under one
> > > + ~ or more contributor license agreements. See the NOTICE file
> > > + ~ distributed with this work for additional information
> > > + ~ regarding copyright ownership. The ASF licenses this file
> > > + ~ to you under the Apache License, Version 2.0 (the
> > > + ~ "License"); you may not use this file except in compliance
> > > + ~ with the License. You may obtain a copy of the License at
> > > + ~
> > > + ~ http://www.apache.org/licenses/LICENSE-2.0
> > > + ~
> > > + ~ Unless required by applicable law or agreed to in writing,
> > > + ~ software distributed under the License is distributed on an
> > > + ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > + ~ KIND, either express or implied. See the License for the
> > > + ~ specific language governing permissions and limitations
> > > + ~ under the License.
> > > + -->
> > > +
> > > +
> > > +<configuration>
> > > + <properties>
> > > + <property name="maven.logging.root.level">INFO</property>
> > > + </properties>
> > > + <appenders>
> > > + <Console name="console" target="SYSTEM_OUT">
> > > + <PatternLayout pattern="%highlight{[%p{WARN=WARNING}]}
> > > %msg%n%throwable" />
> > > + </Console>
> > > + </appenders>
> > > + <loggers>
> > > + <root level="${sys:maven.logging.root.level}">
> > > + <appender-ref ref="console"/>
> > > + </root>
> > > + </loggers>
> > > +</configuration>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/src/conf/logging/log4j2.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/apache-maven/src/conf/logging/log4j2.xml
> > > b/apache-maven/src/conf/logging/log4j2.xml
> > > new file mode 100644
> > > index 0000000..6635597
> > > --- /dev/null
> > > +++ b/apache-maven/src/conf/logging/log4j2.xml
> > > @@ -0,0 +1,36 @@
> > > +<?xml version="1.0" encoding="UTF-8" ?>
> > > +<!--
> > > + ~ Licensed to the Apache Software Foundation (ASF) under one
> > > + ~ or more contributor license agreements. See the NOTICE file
> > > + ~ distributed with this work for additional information
> > > + ~ regarding copyright ownership. The ASF licenses this file
> > > + ~ to you under the Apache License, Version 2.0 (the
> > > + ~ "License"); you may not use this file except in compliance
> > > + ~ with the License. You may obtain a copy of the License at
> > > + ~
> > > + ~ http://www.apache.org/licenses/LICENSE-2.0
> > > + ~
> > > + ~ Unless required by applicable law or agreed to in writing,
> > > + ~ software distributed under the License is distributed on an
> > > + ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > + ~ KIND, either express or implied. See the License for the
> > > + ~ specific language governing permissions and limitations
> > > + ~ under the License.
> > > + -->
> > > +
> > > +
> > > +<configuration> <!--status="debug"-->
> > > + <properties>
> > > + <property name="maven.logging.root.level">INFO</property>
> > > + </properties>
> > > + <appenders>
> > > + <Console name="console" target="SYSTEM_OUT">
> > > + <PatternLayout pattern="[%p{WARN=WARNING}] %msg%n%throwable"/>
> > > + </Console>
> > > + </appenders>
> > > + <loggers>
> > > + <root level="${sys:maven.logging.root.level}">
> > > + <appender-ref ref="console"/>
> > > + </root>
> > > + </loggers>
> > > +</configuration>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/maven-embedder/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/maven-embedder/pom.xml b/maven-embedder/pom.xml
> > > index 53f0724..ef8b935 100644
> > > --- a/maven-embedder/pom.xml
> > > +++ b/maven-embedder/pom.xml
> > > @@ -82,6 +82,10 @@
> > > </dependency>
> > > <dependency>
> > > <groupId>org.slf4j</groupId>
> > > + <artifactId>slf4j-ext</artifactId>
> > > + </dependency>
> > > + <dependency>
> > > + <groupId>org.slf4j</groupId>
> > > <artifactId>slf4j-simple</artifactId>
> > > <optional>true</optional>
> > > </dependency>
> > > @@ -90,6 +94,10 @@
> > > <artifactId>logback-classic</artifactId>
> > > <optional>true</optional>
> > > </dependency>
> > > + <dependency>
> > > + <groupId>org.apache.logging.log4j</groupId>
> > > + <artifactId>log4j-slf4j-impl</artifactId>
> > > + </dependency>
> > > <!-- CLI -->
> > > <dependency>
> > > <groupId>commons-cli</groupId>
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > ----------------------------------------------------------------------
> > > diff --git
> > >
> >
> a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > >
> >
> b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > index 8741836..cd01f9e 100644
> > > ---
> > >
> >
> a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > +++
> > >
> >
> b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > > @@ -18,5 +18,5 @@
> > > # key = Slf4j effective logger factory implementation
> > > # value = corresponding o.a.m.cli.logging.Slf4jConfiguration class
> > > org.slf4j.impl.SimpleLoggerFactory
> > > org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
> > > -org.slf4j.helpers.Log4jLoggerFactory
> > > org.apache.maven.cli.logging.impl.Log4j2Configuration
> > > +org.apache.logging.slf4j.Log4jLoggerFactory
> > > org.apache.maven.cli.logging.impl.Log4j2Configuration
> > > ch.qos.logback.classic.LoggerContext
> > > org.apache.maven.cli.logging.impl.LogbackConfiguration
> > >
> > > http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/pom.xml
> > > ----------------------------------------------------------------------
> > > diff --git a/pom.xml b/pom.xml
> > > index b0ab4e8..87442f3 100644
> > > --- a/pom.xml
> > > +++ b/pom.xml
> > > @@ -61,6 +61,9 @@
> > > <jxpathVersion>1.3</jxpathVersion>
> > > <aetherVersion>1.0.2.v20150114</aetherVersion>
> > > <slf4jVersion>1.7.5</slf4jVersion>
> > > + <log4j2Version>2.2</log4j2Version>
> > > + <logbackVersion>1.0.7</logbackVersion>
> > > + <jansiVersion>1.11</jansiVersion>
> > >
> > >
> >
> <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>
> > > <!-- Control the name of the distribution and information output
> by
> > > mvn -->
> > > <distributionId>apache-maven</distributionId>
> > > @@ -252,6 +255,7 @@
> > > <artifactId>plexus-interpolation</artifactId>
> > > <version>${plexusInterpolationVersion}</version>
> > > </dependency>
> > > + <!-- Logging -->
> > > <dependency>
> > > <groupId>org.slf4j</groupId>
> > > <artifactId>slf4j-api</artifactId>
> > > @@ -261,13 +265,34 @@
> > > <groupId>org.slf4j</groupId>
> > > <artifactId>slf4j-simple</artifactId>
> > > <version>${slf4jVersion}</version>
> > > - <optional>true</optional>
> > > </dependency>
> > > <dependency>
> > > <groupId>ch.qos.logback</groupId>
> > > <artifactId>logback-classic</artifactId>
> > > - <version>1.0.7</version>
> > > - <optional>true</optional>
> > > + <version>${logbackVersion}</version>
> > > + </dependency>
> > > + <dependency>
> > > + <groupId>org.slf4j</groupId>
> > > + <artifactId>slf4j-ext</artifactId>
> > > + <version>${slf4jVersion}</version>
> > > + <scope>compile</scope>
> > > + </dependency>
> > > + <dependency>
> > > + <groupId>org.apache.logging.log4j</groupId>
> > > + <artifactId>log4j-slf4j-impl</artifactId>
> > > + <version>${log4j2Version}</version>
> > > + <scope>compile</scope>
> > > + </dependency>
> > > + <dependency>
> > > + <groupId>org.apache.logging.log4j</groupId>
> > > + <artifactId>log4j-core</artifactId>
> > > + <version>${log4j2Version}</version>
> > > + </dependency>
> > > + <dependency>
> > > + <groupId>org.fusesource.jansi</groupId>
> > > + <artifactId>jansi</artifactId>
> > > + <version>${jansiVersion}</version>
> > > + <scope>runtime</scope>
> > > </dependency>
> > > <!-- Wagon -->
> > > <dependency>
> > >
> > >
> >
> >
> > --
> >
> > Arnaud
> >
>
>
>
> --
> Jeff MAURY
>
>
> "Legacy code" often differs from its suggested alternative by actually
> working and scaling.
> - Bjarne Stroustrup
>
> http://www.jeffmaury.com
> http://riadiscuss.jeffmaury.com
> http://www.twitter.com/jeffmaury
>
--
-----
Arnaud Héritier
http://aheritier.net
Mail/GTalk: aheritier AT gmail DOT com
Twitter/Skype : aheritier
Re: Unreadable commits notifications was Re: [49/50] [abbrv] maven
git commit: Package and configure log4J 2.2 by default. Replace the content
of conf/logging/log4j2.xml by the one from conf/logging/log4j2-color.xml to
enjoy the colorised console
Posted by Jeff MAURY <je...@jeffmaury.com>.
On Wed, Apr 1, 2015 at 2:46 AM, Arnaud Héritier <ah...@apache.org>
wrote:
> Hi all,
>
> Is it normal to have so much (unreadable) notifications when we rewrite a
> branch ?
> I updated, cleaned and rewrote the history of the slf4j-log4j2 branch
>
> https://github.com/apache/maven/commits/slf4j-log4j2
>
> I found this better to rewrite such WIP branch than creating a new one
> (and deleting this one)
> Note that that are several old branches on the same subject that can be
> trashed I think :
> https://github.com/apache/maven/commits/feature/colorized-console/log4j2
> https://github.com/apache/maven/commits/logging/slf4j-log4j2
>
> I think it is important to be notified when a branch is rewritten but
> having 50 emails when I replaced an old branch with < 10 commits by a
> sanitised one of 2 commits, I don't understand.
>
Rewriting history does not reduce the amount of modified/deleted/added
code. And as the notification emails also send diffs, ....
Jeff
>
> Cheers
>
> Arnaud
>
> On Wed, Apr 1, 2015 at 2:24 AM, <ah...@apache.org> wrote:
>
> > Package and configure log4J 2.2 by default.
> > Replace the content of conf/logging/log4j2.xml by the one from
> > conf/logging/log4j2-color.xml to enjoy the colorised console
> >
> >
> > Project: http://git-wip-us.apache.org/repos/asf/maven/repo
> > Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/dbad2e53
> > Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/dbad2e53
> > Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/dbad2e53
> >
> > Branch: refs/heads/slf4j-log4j2
> > Commit: dbad2e536a7024a277eef1c56eaa2286f9f2a7f9
> > Parents: f78742f
> > Author: Arnaud Héritier <ah...@apache.org>
> > Authored: Wed Apr 1 02:16:56 2015 +0200
> > Committer: Arnaud Héritier <ah...@apache.org>
> > Committed: Wed Apr 1 02:16:56 2015 +0200
> >
> > ----------------------------------------------------------------------
> > apache-maven/pom.xml | 15 +++++++-
> > apache-maven/src/conf/logging/log4j2-color.xml | 36
> ++++++++++++++++++++
> > apache-maven/src/conf/logging/log4j2.xml | 36
> ++++++++++++++++++++
> > maven-embedder/pom.xml | 8 +++++
> > .../maven/slf4j-configuration.properties | 2 +-
> > pom.xml | 31 +++++++++++++++--
> > 6 files changed, 123 insertions(+), 5 deletions(-)
> > ----------------------------------------------------------------------
> >
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/apache-maven/pom.xml b/apache-maven/pom.xml
> > index e8277b5..c3e9e49 100644
> > --- a/apache-maven/pom.xml
> > +++ b/apache-maven/pom.xml
> > @@ -95,7 +95,20 @@
> > </dependency>
> > <dependency>
> > <groupId>org.slf4j</groupId>
> > - <artifactId>slf4j-simple</artifactId>
> > + <artifactId>slf4j-ext</artifactId>
> > + </dependency>
> > + <dependency>
> > + <groupId>org.apache.logging.log4j</groupId>
> > + <artifactId>log4j-slf4j-impl</artifactId>
> > + </dependency>
> > + <dependency>
> > + <groupId>org.apache.logging.log4j</groupId>
> > + <artifactId>log4j-core</artifactId>
> > + </dependency>
> > + <dependency>
> > + <groupId>org.fusesource.jansi</groupId>
> > + <artifactId>jansi</artifactId>
> > + <scope>runtime</scope>
> > </dependency>
> > </dependencies>
> >
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/src/conf/logging/log4j2-color.xml
> > ----------------------------------------------------------------------
> > diff --git a/apache-maven/src/conf/logging/log4j2-color.xml
> > b/apache-maven/src/conf/logging/log4j2-color.xml
> > new file mode 100644
> > index 0000000..bea1e76
> > --- /dev/null
> > +++ b/apache-maven/src/conf/logging/log4j2-color.xml
> > @@ -0,0 +1,36 @@
> > +<?xml version="1.0" encoding="UTF-8" ?>
> > +<!--
> > + ~ Licensed to the Apache Software Foundation (ASF) under one
> > + ~ or more contributor license agreements. See the NOTICE file
> > + ~ distributed with this work for additional information
> > + ~ regarding copyright ownership. The ASF licenses this file
> > + ~ to you under the Apache License, Version 2.0 (the
> > + ~ "License"); you may not use this file except in compliance
> > + ~ with the License. You may obtain a copy of the License at
> > + ~
> > + ~ http://www.apache.org/licenses/LICENSE-2.0
> > + ~
> > + ~ Unless required by applicable law or agreed to in writing,
> > + ~ software distributed under the License is distributed on an
> > + ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > + ~ KIND, either express or implied. See the License for the
> > + ~ specific language governing permissions and limitations
> > + ~ under the License.
> > + -->
> > +
> > +
> > +<configuration>
> > + <properties>
> > + <property name="maven.logging.root.level">INFO</property>
> > + </properties>
> > + <appenders>
> > + <Console name="console" target="SYSTEM_OUT">
> > + <PatternLayout pattern="%highlight{[%p{WARN=WARNING}]}
> > %msg%n%throwable" />
> > + </Console>
> > + </appenders>
> > + <loggers>
> > + <root level="${sys:maven.logging.root.level}">
> > + <appender-ref ref="console"/>
> > + </root>
> > + </loggers>
> > +</configuration>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/apache-maven/src/conf/logging/log4j2.xml
> > ----------------------------------------------------------------------
> > diff --git a/apache-maven/src/conf/logging/log4j2.xml
> > b/apache-maven/src/conf/logging/log4j2.xml
> > new file mode 100644
> > index 0000000..6635597
> > --- /dev/null
> > +++ b/apache-maven/src/conf/logging/log4j2.xml
> > @@ -0,0 +1,36 @@
> > +<?xml version="1.0" encoding="UTF-8" ?>
> > +<!--
> > + ~ Licensed to the Apache Software Foundation (ASF) under one
> > + ~ or more contributor license agreements. See the NOTICE file
> > + ~ distributed with this work for additional information
> > + ~ regarding copyright ownership. The ASF licenses this file
> > + ~ to you under the Apache License, Version 2.0 (the
> > + ~ "License"); you may not use this file except in compliance
> > + ~ with the License. You may obtain a copy of the License at
> > + ~
> > + ~ http://www.apache.org/licenses/LICENSE-2.0
> > + ~
> > + ~ Unless required by applicable law or agreed to in writing,
> > + ~ software distributed under the License is distributed on an
> > + ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > + ~ KIND, either express or implied. See the License for the
> > + ~ specific language governing permissions and limitations
> > + ~ under the License.
> > + -->
> > +
> > +
> > +<configuration> <!--status="debug"-->
> > + <properties>
> > + <property name="maven.logging.root.level">INFO</property>
> > + </properties>
> > + <appenders>
> > + <Console name="console" target="SYSTEM_OUT">
> > + <PatternLayout pattern="[%p{WARN=WARNING}] %msg%n%throwable"/>
> > + </Console>
> > + </appenders>
> > + <loggers>
> > + <root level="${sys:maven.logging.root.level}">
> > + <appender-ref ref="console"/>
> > + </root>
> > + </loggers>
> > +</configuration>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/maven-embedder/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/maven-embedder/pom.xml b/maven-embedder/pom.xml
> > index 53f0724..ef8b935 100644
> > --- a/maven-embedder/pom.xml
> > +++ b/maven-embedder/pom.xml
> > @@ -82,6 +82,10 @@
> > </dependency>
> > <dependency>
> > <groupId>org.slf4j</groupId>
> > + <artifactId>slf4j-ext</artifactId>
> > + </dependency>
> > + <dependency>
> > + <groupId>org.slf4j</groupId>
> > <artifactId>slf4j-simple</artifactId>
> > <optional>true</optional>
> > </dependency>
> > @@ -90,6 +94,10 @@
> > <artifactId>logback-classic</artifactId>
> > <optional>true</optional>
> > </dependency>
> > + <dependency>
> > + <groupId>org.apache.logging.log4j</groupId>
> > + <artifactId>log4j-slf4j-impl</artifactId>
> > + </dependency>
> > <!-- CLI -->
> > <dependency>
> > <groupId>commons-cli</groupId>
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > ----------------------------------------------------------------------
> > diff --git
> >
> a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> >
> b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > index 8741836..cd01f9e 100644
> > ---
> >
> a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > +++
> >
> b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
> > @@ -18,5 +18,5 @@
> > # key = Slf4j effective logger factory implementation
> > # value = corresponding o.a.m.cli.logging.Slf4jConfiguration class
> > org.slf4j.impl.SimpleLoggerFactory
> > org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
> > -org.slf4j.helpers.Log4jLoggerFactory
> > org.apache.maven.cli.logging.impl.Log4j2Configuration
> > +org.apache.logging.slf4j.Log4jLoggerFactory
> > org.apache.maven.cli.logging.impl.Log4j2Configuration
> > ch.qos.logback.classic.LoggerContext
> > org.apache.maven.cli.logging.impl.LogbackConfiguration
> >
> > http://git-wip-us.apache.org/repos/asf/maven/blob/dbad2e53/pom.xml
> > ----------------------------------------------------------------------
> > diff --git a/pom.xml b/pom.xml
> > index b0ab4e8..87442f3 100644
> > --- a/pom.xml
> > +++ b/pom.xml
> > @@ -61,6 +61,9 @@
> > <jxpathVersion>1.3</jxpathVersion>
> > <aetherVersion>1.0.2.v20150114</aetherVersion>
> > <slf4jVersion>1.7.5</slf4jVersion>
> > + <log4j2Version>2.2</log4j2Version>
> > + <logbackVersion>1.0.7</logbackVersion>
> > + <jansiVersion>1.11</jansiVersion>
> >
> >
> <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>
> > <!-- Control the name of the distribution and information output by
> > mvn -->
> > <distributionId>apache-maven</distributionId>
> > @@ -252,6 +255,7 @@
> > <artifactId>plexus-interpolation</artifactId>
> > <version>${plexusInterpolationVersion}</version>
> > </dependency>
> > + <!-- Logging -->
> > <dependency>
> > <groupId>org.slf4j</groupId>
> > <artifactId>slf4j-api</artifactId>
> > @@ -261,13 +265,34 @@
> > <groupId>org.slf4j</groupId>
> > <artifactId>slf4j-simple</artifactId>
> > <version>${slf4jVersion}</version>
> > - <optional>true</optional>
> > </dependency>
> > <dependency>
> > <groupId>ch.qos.logback</groupId>
> > <artifactId>logback-classic</artifactId>
> > - <version>1.0.7</version>
> > - <optional>true</optional>
> > + <version>${logbackVersion}</version>
> > + </dependency>
> > + <dependency>
> > + <groupId>org.slf4j</groupId>
> > + <artifactId>slf4j-ext</artifactId>
> > + <version>${slf4jVersion}</version>
> > + <scope>compile</scope>
> > + </dependency>
> > + <dependency>
> > + <groupId>org.apache.logging.log4j</groupId>
> > + <artifactId>log4j-slf4j-impl</artifactId>
> > + <version>${log4j2Version}</version>
> > + <scope>compile</scope>
> > + </dependency>
> > + <dependency>
> > + <groupId>org.apache.logging.log4j</groupId>
> > + <artifactId>log4j-core</artifactId>
> > + <version>${log4j2Version}</version>
> > + </dependency>
> > + <dependency>
> > + <groupId>org.fusesource.jansi</groupId>
> > + <artifactId>jansi</artifactId>
> > + <version>${jansiVersion}</version>
> > + <scope>runtime</scope>
> > </dependency>
> > <!-- Wagon -->
> > <dependency>
> >
> >
>
>
> --
>
> Arnaud
>
--
Jeff MAURY
"Legacy code" often differs from its suggested alternative by actually
working and scaling.
- Bjarne Stroustrup
http://www.jeffmaury.com
http://riadiscuss.jeffmaury.com
http://www.twitter.com/jeffmaury