Browse Source

Merge changes Ib7d18bb2,I3dc8e0b4,I923b65ef,I9e021ba2,I4e79cefe

* changes:
  Improve exception thrown when pull can't find advertised ref
  Silence unchecked conversion warning in TransportSftp
  Silence deprecation warning in WindowCacheConfig
  Silence deprecation warning in DirCacheCheckout
  Fix hidden field warnings in bundle org.eclipse.jgit
stable-4.1
Shawn Pearce 10 years ago committed by Gerrit Code Review @ Eclipse.org
parent
commit
b10259f44b
  1. 2
      org.eclipse.jgit/resources/org/eclipse/jgit/internal/JGitText.properties
  2. 14
      org.eclipse.jgit/src/org/eclipse/jgit/api/PullCommand.java
  3. 54
      org.eclipse.jgit/src/org/eclipse/jgit/api/errors/RefNotAdvertisedException.java
  4. 1
      org.eclipse.jgit/src/org/eclipse/jgit/dircache/DirCacheCheckout.java
  5. 11
      org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/file/ObjectDirectoryPackParser.java
  6. 1
      org.eclipse.jgit/src/org/eclipse/jgit/storage/file/WindowCacheConfig.java
  7. 1
      org.eclipse.jgit/src/org/eclipse/jgit/transport/TransportSftp.java
  8. 8
      org.eclipse.jgit/src/org/eclipse/jgit/transport/UploadPack.java

2
org.eclipse.jgit/resources/org/eclipse/jgit/internal/JGitText.properties

@ -181,7 +181,7 @@ corruptObjectTruncatedInObjectId=truncated in object id
couldNotCheckOutBecauseOfConflicts=Could not check out because of conflicts couldNotCheckOutBecauseOfConflicts=Could not check out because of conflicts
couldNotDeleteLockFileShouldNotHappen=Could not delete lock file. Should not happen couldNotDeleteLockFileShouldNotHappen=Could not delete lock file. Should not happen
couldNotDeleteTemporaryIndexFileShouldNotHappen=Could not delete temporary index file. Should not happen couldNotDeleteTemporaryIndexFileShouldNotHappen=Could not delete temporary index file. Should not happen
couldNotGetAdvertisedRef=Could not get advertised Ref for branch {0} couldNotGetAdvertisedRef=Remote {0} did not advertise Ref for branch {1}. This Ref may not exist in the remote or may be hidden by permission settings.
couldNotGetRepoStatistics=Could not get repository statistics couldNotGetRepoStatistics=Could not get repository statistics
couldNotLockHEAD=Could not lock HEAD couldNotLockHEAD=Could not lock HEAD
couldNotReadIndexInOneGo=Could not read index in one go, only {0} out of {1} read couldNotReadIndexInOneGo=Could not read index in one go, only {0} out of {1} read

14
org.eclipse.jgit/src/org/eclipse/jgit/api/PullCommand.java

@ -54,6 +54,7 @@ import org.eclipse.jgit.api.errors.InvalidConfigurationException;
import org.eclipse.jgit.api.errors.InvalidRemoteException; import org.eclipse.jgit.api.errors.InvalidRemoteException;
import org.eclipse.jgit.api.errors.JGitInternalException; import org.eclipse.jgit.api.errors.JGitInternalException;
import org.eclipse.jgit.api.errors.NoHeadException; import org.eclipse.jgit.api.errors.NoHeadException;
import org.eclipse.jgit.api.errors.RefNotAdvertisedException;
import org.eclipse.jgit.api.errors.RefNotFoundException; import org.eclipse.jgit.api.errors.RefNotFoundException;
import org.eclipse.jgit.api.errors.WrongRepositoryStateException; import org.eclipse.jgit.api.errors.WrongRepositoryStateException;
import org.eclipse.jgit.internal.JGitText; import org.eclipse.jgit.internal.JGitText;
@ -171,6 +172,7 @@ public class PullCommand extends TransportCommand<PullCommand, PullResult> {
* @throws InvalidRemoteException * @throws InvalidRemoteException
* @throws CanceledException * @throws CanceledException
* @throws RefNotFoundException * @throws RefNotFoundException
* @throws RefNotAdvertisedException
* @throws NoHeadException * @throws NoHeadException
* @throws org.eclipse.jgit.api.errors.TransportException * @throws org.eclipse.jgit.api.errors.TransportException
* @throws GitAPIException * @throws GitAPIException
@ -178,7 +180,7 @@ public class PullCommand extends TransportCommand<PullCommand, PullResult> {
public PullResult call() throws GitAPIException, public PullResult call() throws GitAPIException,
WrongRepositoryStateException, InvalidConfigurationException, WrongRepositoryStateException, InvalidConfigurationException,
DetachedHeadException, InvalidRemoteException, CanceledException, DetachedHeadException, InvalidRemoteException, CanceledException,
RefNotFoundException, NoHeadException, RefNotFoundException, RefNotAdvertisedException, NoHeadException,
org.eclipse.jgit.api.errors.TransportException { org.eclipse.jgit.api.errors.TransportException {
checkCallable(); checkCallable();
@ -284,11 +286,13 @@ public class PullCommand extends TransportCommand<PullCommand, PullResult> {
r = fetchRes.getAdvertisedRef(Constants.R_HEADS r = fetchRes.getAdvertisedRef(Constants.R_HEADS
+ remoteBranchName); + remoteBranchName);
} }
if (r == null) if (r == null) {
throw new JGitInternalException(MessageFormat.format(JGitText throw new RefNotAdvertisedException(MessageFormat.format(
.get().couldNotGetAdvertisedRef, remoteBranchName)); JGitText.get().couldNotGetAdvertisedRef, remote,
else remoteBranchName));
} else {
commitToMerge = r.getObjectId(); commitToMerge = r.getObjectId();
}
} else { } else {
try { try {
commitToMerge = repo.resolve(remoteBranchName); commitToMerge = repo.resolve(remoteBranchName);

54
org.eclipse.jgit/src/org/eclipse/jgit/api/errors/RefNotAdvertisedException.java

@ -0,0 +1,54 @@
/*
* Copyright (C) 2015,Matthias Sohn <matthias.sohn@sap.com> and
* other copyright owners as documented in the project's IP log.
*
* This program and the accompanying materials are made available under the
* terms of the Eclipse Distribution License v1.0 which accompanies this
* distribution, is reproduced below, and is available at
* http://www.eclipse.org/org/documents/edl-v10.php
*
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions are met:
*
* - Redistributions of source code must retain the above copyright notice, this
* list of conditions and the following disclaimer.
*
* - Redistributions in binary form must reproduce the above copyright notice,
* this list of conditions and the following disclaimer in the documentation
* and/or other materials provided with the distribution.
*
* - Neither the name of the Eclipse Foundation, Inc. nor the names of its
* contributors may be used to endorse or promote products derived from this
* software without specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
* POSSIBILITY OF SUCH DAMAGE.
*/
package org.eclipse.jgit.api.errors;
/**
* Thrown when a ref is not found in advertised refs
*
* @since 4.0
*/
public class RefNotAdvertisedException extends GitAPIException {
private static final long serialVersionUID = 1L;
/**
* @param message
*/
public RefNotAdvertisedException(String message) {
super(message);
}
}

1
org.eclipse.jgit/src/org/eclipse/jgit/dircache/DirCacheCheckout.java vendored

@ -1192,6 +1192,7 @@ public class DirCacheCheckout {
entry.setLastModified(f.lastModified()); entry.setLastModified(f.lastModified());
} }
@SuppressWarnings("deprecation")
private static void checkValidPath(CanonicalTreeParser t) private static void checkValidPath(CanonicalTreeParser t)
throws InvalidPathException { throws InvalidPathException {
ObjectChecker chk = new ObjectChecker() ObjectChecker chk = new ObjectChecker()

11
org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/file/ObjectDirectoryPackParser.java

@ -366,11 +366,10 @@ public class ObjectDirectoryPackParser extends PackParser {
@Override @Override
protected void onEndThinPack() throws IOException { protected void onEndThinPack() throws IOException {
final byte[] tailHash = this.tailDigest.digest();
final byte[] buf = buffer(); final byte[] buf = buffer();
final MessageDigest origDigest = Constants.newMessageDigest(); final MessageDigest origDigest = Constants.newMessageDigest();
final MessageDigest tailDigest = Constants.newMessageDigest(); final MessageDigest tailDigest2 = Constants.newMessageDigest();
final MessageDigest packDigest = Constants.newMessageDigest(); final MessageDigest packDigest = Constants.newMessageDigest();
long origRemaining = origEnd; long origRemaining = origEnd;
@ -393,15 +392,15 @@ public class ObjectDirectoryPackParser extends PackParser {
origDigest.update(buf, 0, origCnt); origDigest.update(buf, 0, origCnt);
origRemaining -= origCnt; origRemaining -= origCnt;
if (origRemaining == 0) if (origRemaining == 0)
tailDigest.update(buf, origCnt, n - origCnt); tailDigest2.update(buf, origCnt, n - origCnt);
} else } else
tailDigest.update(buf, 0, n); tailDigest2.update(buf, 0, n);
packDigest.update(buf, 0, n); packDigest.update(buf, 0, n);
} }
if (!Arrays.equals(origDigest.digest(), origHash) if (!Arrays.equals(origDigest.digest(), origHash) || !Arrays
|| !Arrays.equals(tailDigest.digest(), tailHash)) .equals(tailDigest2.digest(), this.tailDigest.digest()))
throw new IOException( throw new IOException(
JGitText.get().packCorruptedWhileWritingToFilesystem); JGitText.get().packCorruptedWhileWritingToFilesystem);

1
org.eclipse.jgit/src/org/eclipse/jgit/storage/file/WindowCacheConfig.java

@ -221,6 +221,7 @@ public class WindowCacheConfig {
* *
* @since 3.0 * @since 3.0
*/ */
@SuppressWarnings("deprecation")
public void install() { public void install() {
WindowCache.reconfigure(this); WindowCache.reconfigure(this);
} }

1
org.eclipse.jgit/src/org/eclipse/jgit/transport/TransportSftp.java

@ -227,6 +227,7 @@ public class TransportSftp extends SshTransport implements WalkTransport {
Collection<String> getPackNames() throws IOException { Collection<String> getPackNames() throws IOException {
final List<String> packs = new ArrayList<String>(); final List<String> packs = new ArrayList<String>();
try { try {
@SuppressWarnings("unchecked")
final Collection<ChannelSftp.LsEntry> list = ftp.ls("pack"); //$NON-NLS-1$ final Collection<ChannelSftp.LsEntry> list = ftp.ls("pack"); //$NON-NLS-1$
final HashMap<String, ChannelSftp.LsEntry> files; final HashMap<String, ChannelSftp.LsEntry> files;
final HashMap<String, Integer> mtimes; final HashMap<String, Integer> mtimes;

8
org.eclipse.jgit/src/org/eclipse/jgit/transport/UploadPack.java

@ -810,9 +810,9 @@ public class UploadPack {
adv.advertiseCapability(OPTION_ALLOW_TIP_SHA1_IN_WANT); adv.advertiseCapability(OPTION_ALLOW_TIP_SHA1_IN_WANT);
adv.advertiseCapability(OPTION_AGENT, UserAgent.get()); adv.advertiseCapability(OPTION_AGENT, UserAgent.get());
adv.setDerefTags(true); adv.setDerefTags(true);
Map<String, Ref> refs = getAdvertisedOrDefaultRefs(); Map<String, Ref> advertisedOrDefaultRefs = getAdvertisedOrDefaultRefs();
findSymrefs(adv, refs); findSymrefs(adv, advertisedOrDefaultRefs);
advertised = adv.send(refs); advertised = adv.send(advertisedOrDefaultRefs);
if (adv.isEmpty()) if (adv.isEmpty())
adv.advertiseId(ObjectId.zeroId(), "capabilities^{}"); //$NON-NLS-1$ adv.advertiseId(ObjectId.zeroId(), "capabilities^{}"); //$NON-NLS-1$
adv.end(); adv.end();
@ -1467,7 +1467,7 @@ public class UploadPack {
pckOut.end(); pckOut.end();
} }
private void findSymrefs( private static void findSymrefs(
final RefAdvertiser adv, final Map<String, Ref> refs) { final RefAdvertiser adv, final Map<String, Ref> refs) {
Ref head = refs.get(Constants.HEAD); Ref head = refs.get(Constants.HEAD);
if (head != null && head.isSymbolic()) { if (head != null && head.isSymbolic()) {

Loading…
Cancel
Save