This just struck me as odd that we sometimes used a setter method, and sometimes used direct assignment to the field. Now we use the setter method if it is available. The fetch and push refspecs do not have setters, so must still be done by direct field assignment. Signed-off-by: Shawn O. Pearce <spearce@xxxxxxxxxxx> --- .../src/org/spearce/jgit/transport/Transport.java | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/org.spearce.jgit/src/org/spearce/jgit/transport/Transport.java b/org.spearce.jgit/src/org/spearce/jgit/transport/Transport.java index 1068f50..c36ccdd 100644 --- a/org.spearce.jgit/src/org/spearce/jgit/transport/Transport.java +++ b/org.spearce.jgit/src/org/spearce/jgit/transport/Transport.java @@ -568,9 +568,9 @@ public void setRemoveDeletedRefs(final boolean remove) { */ public void applyConfig(final RemoteConfig cfg) { setOptionUploadPack(cfg.getUploadPack()); - fetch = cfg.getFetchRefSpecs(); + setOptionReceivePack(cfg.getReceivePack()); setTagOpt(cfg.getTagOpt()); - optionReceivePack = cfg.getReceivePack(); + fetch = cfg.getFetchRefSpecs(); push = cfg.getPushRefSpecs(); } -- 1.6.3.2.416.g04d0 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html