From: ori@eigenstate.org
To: 9front@9front.org, philip.silva@protonmail.com
Cc: 9front@9front.org
Subject: Re: [9front] Git multiple remotes
Date: Tue, 04 Jan 2022 11:37:43 -0500 [thread overview]
Message-ID: <7F9053BC16DE03533DACFC3283504681@eigenstate.org> (raw)
In-Reply-To: <87ilv0a9fe.fsf@turtle-trading.net>
Quoth Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net>:
> Hi Philip.
>
> Thanks for confirming what I thought. So this is what I did (slightly
> edited):
>
> term% cat /.git/config
> [core]
> repositoryformatversion = p9.0
> [remote "origin"]
> url=gits://git.9front.org/plan9front/plan9front
> [remote "host"]
> url=ssh://benny@host/home/benny/Projects/plan9front
>
> This URL works from the host with "host" replaced with the actual IP
> address. The SSH access from my 9front VM also works with "ssh
> benny@host hostname".
>
> term% git/pull -u host
> git/query: resolve: invalid ref remotes/host/front
> git/query: resolve: invalid ref remotes/host/front
> git/query: resolve: invalid ref remotes/host/front
> ours: facb0e757ac63f763bd942a2714f979538b99eb0
> git/query: resolve: invalid ref remotes/host/front
> theirs:
> git/query: resolve: invalid ref remotes/host/front
> common:
> git/merge remotes/host/front
>
> term% git/branch -a
> heads/front
> remotes/THEM/front
> remotes/origin/front
>
> I expected remotes/host/front in that list. git/pull also seems to
> expect that, s.a.
>
> term% git/query origin/front front host/front
> git/query: resolve: invalid ref host/front
>
> term% git/query origin/front front
> acc504c319a4b4188479cfa602e40cb6851c0528
> facb0e757ac63f763bd942a2714f979538b99eb0
>
> Note that host/front is actually at f63d1d3ce at this time.
>
> I try an explict fetch:
>
> term% cd /
> term% git/fetch -u host ssh://benny@host/home/benny/Projects/plan9front
> uri: "ssh://benny@host/home/benny/Projects/plan9front"
> symref HEAD refs/heads/front
>
> But this does not change anything.
>
> Anything else I could try or add?
>
> TIA, benny
>
by default, we don't fetch all remote branches;
there was a diff that went in a while ago to improve
this, but it got backed out because it encounters a
bug in upstream git -- the spec says that a single
ACK should be sent when not using multi-ack modes,
but they send back multiple ones. I've got an
apparent workaround to fix that, but it's not yet
tested enough to commit.
Here's the work in progress -- if you can apply it
@{cd / && ape/patch -p1} < $patch
and let me know if that improves things, that'd be
appreciated. There's a chance I may just need to put
in a heuristic for when the pack starts, or bite the
bullet and implement multi-ack.
diff f63d1d3ced81702e0eadf56228a54a467278b0d4 uncommitted
--- a/sys/src/cmd/git/fetch.c
+++ b/sys/src/cmd/git/fetch.c
@@ -186,6 +186,7 @@
int nref, refsz, first;
int i, n, req, pfd;
vlong packsz;
+ Objset hadobj;
Object *o;
nref = 0;
@@ -246,13 +247,19 @@
req = 1;
}
flushpkt(c);
+ osinit(&hadobj);
for(i = 0; i < nref; i++){
- if(hasheq(&have[i], &Zhash))
+ if(hasheq(&have[i], &Zhash) || oshas(&hadobj, have[i]))
continue;
+ if((o = readobject(have[i])) == nil)
+ sysfatal("missing object we should have: %H", have[i]);
+ osadd(&hadobj, o);
+ unref(o);
n = snprint(buf, sizeof(buf), "have %H\n", have[i]);
if(writepkt(c, buf, n + 1) == -1)
sysfatal("could not send have for %H", have[i]);
}
+ osclear(&hadobj);
if(!req)
flushpkt(c);
@@ -260,7 +267,7 @@
if(writepkt(c, buf, n) == -1)
sysfatal("write: %r");
if(!req)
- return 0;
+ goto showrefs;
if(readphase(c) == -1)
sysfatal("read: %r");
if((n = readpkt(c, buf, sizeof(buf))) == -1)
--- a/sys/src/cmd/git/pull
+++ b/sys/src/cmd/git/pull
@@ -7,13 +7,10 @@
upstream=$2
url=$3
dir=$4
- bflag=()
dflag=()
- if(! ~ $#branch 0)
- bflag=(-b $branch)
if(! ~ $#debug 0)
dflag='-d'
- {git/fetch $dflag $bflag -u $upstream $url >[2=3] || die $status} | awk '
+ {git/fetch $dflag -u $upstream $url >[2=3] || die $status} | awk '
/^remote/{
if($2=="HEAD")
next
next prev parent reply other threads:[~2022-01-04 18:27 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-03 15:20 Benjamin Riefenstahl
2022-01-03 16:59 ` Philip Silva
2022-01-03 19:47 ` Benjamin Riefenstahl
2022-01-04 0:20 ` Philip Silva
2022-01-04 12:44 ` Benjamin Riefenstahl
2022-01-04 16:37 ` ori [this message]
2022-01-04 19:14 ` Benjamin Riefenstahl
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7F9053BC16DE03533DACFC3283504681@eigenstate.org \
--to=ori@eigenstate.org \
--cc=9front@9front.org \
--cc=philip.silva@protonmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).