Something is still fishy about it.

I have constant IT failure with MavenITmng3748BadSettingsXmlTest.
Log is:
=======================
[INFO] Error stacktraces are turned on.
[WARNING]
[WARNING] Some problems were encountered while building the effective settings [WARNING] Unrecognised tag: 'repositories' (position: START_TAG seen <settings>\r\n <repositories>... @2:19) @ D:\Entwicklung\Projekte\maven-integration-testing\core-it-suite\target\test-classes\mng-3748\settings.xml, line 2, column 19
[WARNING]
[INFO] Scanning for projects...
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building Integration Test Project for MNG-3748 1
[INFO] ------------------------------------------------------------------------ [INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 0.063 s
[INFO] Finished at: 2016-06-12T22:04:22+02:00
[INFO] Final Memory: 7M/241M
[INFO] ------------------------------------------------------------------------
=======================

Validation fails because of this Java snippet:
line.matches( "(?i)\\[WARNING\\].*unrecognised tag.+repositories.+2.*" ) )

If I dig deeper, tens of log files have:
> [INFO] Error stacktraces are turned on.

but the rest is not colorized, e.g., log file of IT 0010 or MNG-2140.
Somewhat inconsistent to me.

Reason: MavenCli disables JAnsi in batch mode (populateRequest()) but this message is issued in commands() which comes earlier. It also applies to all deprecated options.

This should probably go up to #logging().

Another issue: writing to a log file with -l without adding batch mode adds all escapes to the file which makes it unreadable in an editor. Redirecting, however, works because JAnsi detects that.


Am 2016-06-11 um 13:27 schrieb Hervé BOUTEMY:
if someone has an objection, just tell and we can easily revert

Regards,

Hervé

Le samedi 11 juin 2016 11:26:10 [email protected] a écrit :
Repository: maven
Updated Branches:
  refs/heads/master 427f18c3a -> 644ac9c40


[MNG-6038] use Gossip slf4j provider (with level color support)

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

Branch: refs/heads/master
Commit: 644ac9c40ad41bf61e3b099918af33b8eb950621
Parents: 427f18c
Author: Hervé Boutemy <[email protected]>
Authored: Sat Jun 11 13:26:05 2016 +0200
Committer: Hervé Boutemy <[email protected]>
Committed: Sat Jun 11 13:26:05 2016 +0200

----------------------------------------------------------------------
 apache-maven/pom.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/maven/blob/644ac9c4/apache-maven/pom.
xml ----------------------------------------------------------------------
diff --git a/apache-maven/pom.xml b/apache-maven/pom.xml
index 9405df8..37751cc 100644
--- a/apache-maven/pom.xml
+++ b/apache-maven/pom.xml
@@ -92,8 +92,8 @@ under the License.
       <artifactId>aether-transport-wagon</artifactId>
     </dependency>
     <dependency>
-      <groupId>org.slf4j</groupId>
-      <artifactId>slf4j-simple</artifactId>
+      <groupId>com.planet57.gossip</groupId>
+      <artifactId>gossip-slf4j</artifactId>
     </dependency>
   </dependencies>


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]





---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to