✨ Print maven output on Maven error (#709)

Fixes #696 --------- Signed-off-by: Juan Manuel Leflet Estrada <[email protected]> Signed-off-by: Cherry Picker <[email protected]>
konveyor · Dec 10, 2024 · 11e5745 · 11e5745
1 parent 7b4c586
commit 11e5745
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/external-providers/java-external-provider/pkg/java_external_provider/dependency.go b/external-providers/java-external-provider/pkg/java_external_provider/dependency.go
@@ -177,7 +177,7 @@ func getMavenLocalRepoPath(mvnSettingsFile string) string {
 	}
 
 	// check errors
-	return string(outb.String())
+	return outb.String()
 }
 
 func (p *javaServiceClient) GetDependenciesFallback(ctx context.Context, location string) (map[uri.URI][]*provider.Dep, error) {
@@ -329,7 +329,7 @@ func (p *javaServiceClient) getDependenciesForMaven(_ context.Context) (map[uri.
 	cmd.Dir = moddir
 	mvnOutput, err := cmd.CombinedOutput()
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("maven dependency:tree command failed with error %w, maven output: %s", err, string(mvnOutput))
 	}
 
 	lines := strings.Split(string(mvnOutput), "\n")

diff --git a/external-providers/java-external-provider/pkg/java_external_provider/provider.go b/external-providers/java-external-provider/pkg/java_external_provider/provider.go
@@ -299,7 +299,7 @@ func (p *javaProvider) Init(ctx context.Context, log logr.Logger, config provide
 		mvnOutput, err := cmd.CombinedOutput()
 		if err != nil {
 			cancelFunc()
-			return nil, additionalBuiltinConfig, fmt.Errorf("error downloading java artifact %s - %w", mvnUri, err)
+			return nil, additionalBuiltinConfig, fmt.Errorf("error downloading java artifact %s - maven output: %s - with error %w", mvnUri, string(mvnOutput), err)
 		}
 		downloadedPath := filepath.Join(outputDir,
 			fmt.Sprintf("%s.jar", strings.Join(mvnCoordinatesParts[1:3], "-")))
@@ -714,7 +714,7 @@ func resolveSourcesJarsForMaven(ctx context.Context, log logr.Logger, fernflower
 	cmd.Dir = location
 	mvnOutput, err := cmd.CombinedOutput()
 	if err != nil {
-		return err
+		return fmt.Errorf("maven downloadSources command failed with error %w, maven output: %s", err, string(mvnOutput))
 	}
 
 	reader := bytes.NewReader(mvnOutput)