From dcac4eae6382e2a903b030ef8f3b04293bd3fdec Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Herv=C3=A9=20Boutemy?= Date: Thu, 15 Dec 2022 23:38:12 +0100 Subject: [PATCH] simplify "Compiling" info message: display relative path (#250) simplify "Compiling" info message: display relative path --- .../plexus/compiler/AbstractCompiler.java | 12 ++++++++++++ .../src/main/it/error-prone-compiler/pom.xml | 17 +++++++++++++++++ .../plexus/compiler/ajc/AspectJCompiler.java | 4 +--- .../plexus/compiler/csharp/CSharpCompiler.java | 3 +-- .../compiler/eclipse/EclipseJavaCompiler.java | 2 ++ .../plexus/compiler/j2objc/J2ObjCCompiler.java | 4 +--- .../plexus/compiler/javac/JavacCompiler.java | 7 +------ 7 files changed, 35 insertions(+), 14 deletions(-) diff --git a/plexus-compiler-api/src/main/java/org/codehaus/plexus/compiler/AbstractCompiler.java b/plexus-compiler-api/src/main/java/org/codehaus/plexus/compiler/AbstractCompiler.java index 579e6280..2a64b691 100644 --- a/plexus-compiler-api/src/main/java/org/codehaus/plexus/compiler/AbstractCompiler.java +++ b/plexus-compiler-api/src/main/java/org/codehaus/plexus/compiler/AbstractCompiler.java @@ -299,4 +299,16 @@ protected static String[] toStringArray( List arguments ) return args; } + + protected void logCompiling( String[] sourceFiles, CompilerConfiguration config ) + { + if ( ( getLogger() != null ) && getLogger().isInfoEnabled() ) + { + String to = ( config.getWorkingDirectory() == null ) ? config.getOutputLocation() : + config.getWorkingDirectory().toPath().relativize( new File( config.getOutputLocation() ).toPath() ).toString(); + getLogger().info( "Compiling " + + ( sourceFiles == null ? "" : ( sourceFiles.length + " source file" + ( sourceFiles.length == 1 ? " " : "s " ) ) ) + + "to " + to ); + } + } } diff --git a/plexus-compiler-its/src/main/it/error-prone-compiler/pom.xml b/plexus-compiler-its/src/main/it/error-prone-compiler/pom.xml index 47466656..bbdb70fb 100644 --- a/plexus-compiler-its/src/main/it/error-prone-compiler/pom.xml +++ b/plexus-compiler-its/src/main/it/error-prone-compiler/pom.xml @@ -65,6 +65,23 @@ + + + org.codehaus.plexus + plexus-compiler-api + ${plexus.compiler.version} + + + org.codehaus.plexus + plexus-compiler-javac-errorprone + ${plexus.compiler.version} + + + org.codehaus.plexus + plexus-compiler-javac + ${plexus.compiler.version} + + diff --git a/plexus-compilers/plexus-compiler-aspectj/src/main/java/org/codehaus/plexus/compiler/ajc/AspectJCompiler.java b/plexus-compilers/plexus-compiler-aspectj/src/main/java/org/codehaus/plexus/compiler/ajc/AspectJCompiler.java index 83053d48..ae1a7daf 100644 --- a/plexus-compilers/plexus-compiler-aspectj/src/main/java/org/codehaus/plexus/compiler/ajc/AspectJCompiler.java +++ b/plexus-compilers/plexus-compiler-aspectj/src/main/java/org/codehaus/plexus/compiler/ajc/AspectJCompiler.java @@ -320,9 +320,7 @@ public CompilerResult performCompile( CompilerConfiguration config ) return new CompilerResult(); } - System.out.println( - "Compiling " + sourceFiles.length + " " + "source file" + ( sourceFiles.length == 1 ? "" : "s" ) + " to " - + destinationDir.getAbsolutePath() ); + logCompiling( sourceFiles, config ); // String[] args = buildCompilerArguments( config, sourceFiles ); AjBuildConfig buildConfig = buildCompilerConfig( config ); diff --git a/plexus-compilers/plexus-compiler-csharp/src/main/java/org/codehaus/plexus/compiler/csharp/CSharpCompiler.java b/plexus-compilers/plexus-compiler-csharp/src/main/java/org/codehaus/plexus/compiler/csharp/CSharpCompiler.java index 3266f925..dc6e01ef 100644 --- a/plexus-compilers/plexus-compiler-csharp/src/main/java/org/codehaus/plexus/compiler/csharp/CSharpCompiler.java +++ b/plexus-compilers/plexus-compiler-csharp/src/main/java/org/codehaus/plexus/compiler/csharp/CSharpCompiler.java @@ -115,8 +115,7 @@ public CompilerResult performCompile( CompilerConfiguration config ) return new CompilerResult().success( true ); } - System.out.println( "Compiling " + sourceFiles.length + " " + "source file" + - ( sourceFiles.length == 1 ? "" : "s" ) + " to " + destinationDir.getAbsolutePath() ); + logCompiling( sourceFiles, config ); String[] args = buildCompilerArguments( config, sourceFiles ); diff --git a/plexus-compilers/plexus-compiler-eclipse/src/main/java/org/codehaus/plexus/compiler/eclipse/EclipseJavaCompiler.java b/plexus-compilers/plexus-compiler-eclipse/src/main/java/org/codehaus/plexus/compiler/eclipse/EclipseJavaCompiler.java index 330936d3..f12891f4 100644 --- a/plexus-compilers/plexus-compiler-eclipse/src/main/java/org/codehaus/plexus/compiler/eclipse/EclipseJavaCompiler.java +++ b/plexus-compilers/plexus-compiler-eclipse/src/main/java/org/codehaus/plexus/compiler/eclipse/EclipseJavaCompiler.java @@ -251,6 +251,8 @@ public CompilerResult performCompile( CompilerConfiguration config ) allSources = resortSourcesToPutModuleInfoFirst( allSources ); + logCompiling( null, config ); + // Compile try { diff --git a/plexus-compilers/plexus-compiler-j2objc/src/main/java/org/codehaus/plexus/compiler/j2objc/J2ObjCCompiler.java b/plexus-compilers/plexus-compiler-j2objc/src/main/java/org/codehaus/plexus/compiler/j2objc/J2ObjCCompiler.java index d4098e00..2621968a 100644 --- a/plexus-compilers/plexus-compiler-j2objc/src/main/java/org/codehaus/plexus/compiler/j2objc/J2ObjCCompiler.java +++ b/plexus-compilers/plexus-compiler-j2objc/src/main/java/org/codehaus/plexus/compiler/j2objc/J2ObjCCompiler.java @@ -122,9 +122,7 @@ public CompilerResult performCompile( CompilerConfiguration config ) return new CompilerResult().success( true ); } - System.out.println( - "Compiling " + sourceFiles.length + " " + "source file" + ( sourceFiles.length == 1 ? "" : "s" ) + " to " - + destinationDir.getAbsolutePath() ); + logCompiling( sourceFiles, config ); String[] args = buildCompilerArguments( config, sourceFiles ); diff --git a/plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java b/plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java index 136319e6..b53e52c0 100644 --- a/plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java +++ b/plexus-compilers/plexus-compiler-javac/src/main/java/org/codehaus/plexus/compiler/javac/JavacCompiler.java @@ -141,12 +141,7 @@ public CompilerResult performCompile( CompilerConfiguration config ) return new CompilerResult(); } - if ( ( getLogger() != null ) && getLogger().isInfoEnabled() ) - { - getLogger().info( "Compiling " + sourceFiles.length + " " + - "source file" + ( sourceFiles.length == 1 ? "" : "s" ) + - " to " + destinationDir.getAbsolutePath() ); - } + logCompiling( sourceFiles, config ); String[] args = buildCompilerArguments( config, sourceFiles );