From e8d37e1fb52322945ef14f519870999f95686825 Mon Sep 17 00:00:00 2001 From: "Shawn O. Pearce" Date: Thu, 4 Sep 2008 16:42:17 -0700 Subject: [PATCH] Include URIish in bundle transport within any TransportExceptions This way the bundle path (as entered by the user) is reported as part of the error. Signed-off-by: Shawn O. Pearce Signed-off-by: Robin Rosenberg --- .../org/spearce/jgit/transport/TransportBundle.java | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/org.spearce.jgit/src/org/spearce/jgit/transport/TransportBundle.java b/org.spearce.jgit/src/org/spearce/jgit/transport/TransportBundle.java index 87834b31..6e1d51de 100644 --- a/org.spearce.jgit/src/org/spearce/jgit/transport/TransportBundle.java +++ b/org.spearce.jgit/src/org/spearce/jgit/transport/TransportBundle.java @@ -128,7 +128,7 @@ class TransportBundle extends PackTransport { in = new FileInputStream(bundle); bin = new RewindBufferedInputStream(in); } catch (FileNotFoundException err) { - throw new TransportException(bundle.getPath() + ": not found"); + throw new TransportException(uri, "not found"); } try { @@ -137,8 +137,7 @@ class TransportBundle extends PackTransport { readBundleV2(); break; default: - throw new TransportException(bundle.getPath() - + ": not a bundle"); + throw new TransportException(uri, "not a bundle"); } in.getChannel().position( @@ -149,12 +148,10 @@ class TransportBundle extends PackTransport { throw err; } catch (IOException err) { close(); - throw new TransportException(bundle.getPath() + ": " - + err.getMessage(), err); + throw new TransportException(uri, err.getMessage(), err); } catch (RuntimeException err) { close(); - throw new TransportException(bundle.getPath() + ": " - + err.getMessage(), err); + throw new TransportException(uri, err.getMessage(), err); } } @@ -162,7 +159,7 @@ class TransportBundle extends PackTransport { final String rev = readLine(new byte[1024]); if (V2_BUNDLE_SIGNATURE.equals(rev)) return 2; - throw new TransportException(bundle.getPath() + ": not a bundle"); + throw new TransportException(uri, "not a bundle"); } private void readBundleV2() throws IOException { @@ -189,8 +186,8 @@ class TransportBundle extends PackTransport { } private PackProtocolException duplicateAdvertisement(final String name) { - return new PackProtocolException("duplicate advertisements of " - + name); + return new PackProtocolException(uri, + "duplicate advertisements of " + name); } private String readLine(final byte[] hdrbuf) throws IOException { @@ -217,10 +214,10 @@ class TransportBundle extends PackTransport { ip.renameAndOpenPack(); } catch (IOException err) { close(); - throw new TransportException(err.getMessage(), err); + throw new TransportException(uri, err.getMessage(), err); } catch (RuntimeException err) { close(); - throw new TransportException(err.getMessage(), err); + throw new TransportException(uri, err.getMessage(), err); } } -- 2.11.4.GIT