From 3285d2f238993f53d00f5e9f1069a33f3d5ba1a2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=BCdiger=20Herrmann?= <ruediger.herrmann@gmx.de>
Date: Sun, 11 Jan 2015 16:20:51 +0100
Subject: [PATCH] [pgm] Prevent commands from writing progress to System.err
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Commands which report progress used to write to System.err. This is not
desirable in cases where jgit.pgm is embedded. This change redirects
progress output to the error stream that is configured by the command.

Change-Id: I01fa5e167437e619448ac201fcb1cbf63bad96d7
Signed-off-by: Rüdiger Herrmann <ruediger.herrmann@gmx.de>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
---
 org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Diff.java         | 2 +-
 org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Fetch.java        | 2 +-
 org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Gc.java           | 2 +-
 org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/IndexPack.java    | 2 +-
 org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java         | 2 +-
 .../src/org/eclipse/jgit/pgm/debug/RebuildCommitGraph.java      | 2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Diff.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Diff.java
index 8f051688b..fe2df6554 100644
--- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Diff.java
+++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Diff.java
@@ -199,7 +199,7 @@ class Diff extends TextBuiltin {
 			} else if (newTree == null)
 				newTree = new FileTreeIterator(db);
 
-			TextProgressMonitor pm = new TextProgressMonitor();
+			TextProgressMonitor pm = new TextProgressMonitor(errw);
 			pm.setDelayStart(2, TimeUnit.SECONDS);
 			diffFmt.setProgressMonitor(pm);
 			diffFmt.setPathFilter(pathFilter);
diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Fetch.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Fetch.java
index 8e3ec03c7..186fdd8a2 100644
--- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Fetch.java
+++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Fetch.java
@@ -123,7 +123,7 @@ class Fetch extends AbstractFetchCommand {
 		if (thin != null)
 			fetch.setThin(thin.booleanValue());
 		if (quiet == null || !quiet.booleanValue())
-			fetch.setProgressMonitor(new TextProgressMonitor());
+			fetch.setProgressMonitor(new TextProgressMonitor(errw));
 
 		FetchResult result = fetch.call();
 		if (result.getTrackingRefUpdates().isEmpty())
diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Gc.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Gc.java
index aa5c90590..bf454760a 100644
--- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Gc.java
+++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Gc.java
@@ -56,6 +56,6 @@ class Gc extends TextBuiltin {
 	protected void run() throws Exception {
 		Git git = Git.wrap(db);
 		git.gc().setAggressive(aggressive)
-				.setProgressMonitor(new TextProgressMonitor()).call();
+				.setProgressMonitor(new TextProgressMonitor(errw)).call();
 	}
 }
diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/IndexPack.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/IndexPack.java
index 42114062b..4afb6d5f8 100644
--- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/IndexPack.java
+++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/IndexPack.java
@@ -71,7 +71,7 @@ class IndexPack extends TextBuiltin {
 				ObjectDirectoryPackParser imp = (ObjectDirectoryPackParser) p;
 				imp.setIndexVersion(indexVersion);
 			}
-			p.parse(new TextProgressMonitor());
+			p.parse(new TextProgressMonitor(errw));
 			inserter.flush();
 		} finally {
 			inserter.release();
diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java
index c7c27b4c4..111933790 100644
--- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java
+++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/Push.java
@@ -113,7 +113,7 @@ class Push extends TextBuiltin {
 		PushCommand push = git.push();
 		push.setDryRun(dryRun);
 		push.setForce(force);
-		push.setProgressMonitor(new TextProgressMonitor());
+		push.setProgressMonitor(new TextProgressMonitor(errw));
 		push.setReceivePack(receivePack);
 		push.setRefSpecs(refSpecs);
 		if (all)
diff --git a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/debug/RebuildCommitGraph.java b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/debug/RebuildCommitGraph.java
index 214093983..afa469636 100644
--- a/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/debug/RebuildCommitGraph.java
+++ b/org.eclipse.jgit.pgm/src/org/eclipse/jgit/pgm/debug/RebuildCommitGraph.java
@@ -110,7 +110,7 @@ class RebuildCommitGraph extends TextBuiltin {
 	@Argument(index = 1, required = true, metaVar = "metaVar_refs", usage = "usage_logAllPretty")
 	File graph;
 
-	private final ProgressMonitor pm = new TextProgressMonitor();
+	private final ProgressMonitor pm = new TextProgressMonitor(errw);
 
 	private Map<ObjectId, ObjectId> rewrites = new HashMap<ObjectId, ObjectId>();