git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 1/1] mingw: optionally disable side-band-64k for transport

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Apr 29, 2019, 23:17 UTC
Message-ID
<nycvar.QRO.7.76.6.1904291916510.45@tvgsbejvaqbjf.bet>
In-Reply-To
<CAPig+cT1EP+HC5h4e0NGCu=1i6cZXPxERRgnnAdZJn+XeqpbFw@mail.gmail.com>
Hi Eric,
On Mon, 29 Apr 2019, Eric Sunshine wrote:
Show 9 quoted lines
> On Mon, Apr 29, 2019 at 6:04 PM Thomas Braun via GitGitGadget
> <gitgitgadget@gmail.com> wrote:
> > Since commit 0c499ea60f (send-pack: demultiplex a sideband stream with
> > status data, 2010-02-05) the built-in send-pack uses the side-band-64k
> > capability if advertised by the server.
> >
> > Unfortunately this breaks pushing over the dump git protocol if used
>
> s/dump/dumb/
Of course!
Show 24 quoted lines
> > over a network connection when using MinGW (but *not* when using
> > mingw-w64).
> > [...]
> > The new config option "sendpack.sideband" allows to override the
> > side-band-64k capability of the server, and thus makes the dump git
> > protocol work.
> > [...]
> > Signed-off-by: Thomas Braun <thomas.braun@byte-physics.de>
> > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> > ---
> > diff --git a/Documentation/config/sendpack.txt b/Documentation/config/sendpack.txt
> > @@ -0,0 +1,5 @@
> > +sendpack.sideband::
> > +       Allows to disable the side-band-64k capability for send-pack even
> > +       when it is advertised by the server. Makes it possible to work
> > +       around a limitation in the git for windows implementation together
> > +       with the dump git protocol. Defaults to true.
>
> s/dump/dumb/
>
> For someone who hasn't read the commit message of this patch, "work
> around a limitation in ... git for windows" doesn't mean much. Perhaps
> this documentation could explain in more precise terms under what
> circumstances this option should be used?

You're right, this is confusing, especially since Git for Windows 2.x does not have that bug.

I simply dropped that sentence.

Thanks! Dscho

Previous: Eric SunshineNext: Johannes Sixt
Message 4 of 10 in “Support git:// with old MinGW”
  1. 0/1 Support git:// with old MinGWJohannes Schindelin via GitGitGadget, Apr 29, 2019
  2. 1/1 mingw: optionally disable side-band-64k for transportThomas Braun via GitGitGadget, Apr 29, 2019
  3. Eric SunshineApr 29, 2019
  4. Johannes SchindelinApr 29, 2019
  5. Johannes SixtApr 30, 2019
  6. Johannes SchindelinApr 30, 2019
  7. Johannes SixtApr 30, 2019
  8. Johannes SchindelinMay 3, 2019
  9. brian m. carlsonApr 29, 2019
  10. Johannes SchindelinApr 30, 2019

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.