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

Re: [PATCH 2/2] transport-helper: ask the helper to set the same options for import as for fetch

From
Mike Hommey <mh@glandium.org>
Date
Feb 13, 2015, 22:14 UTC
Message-ID
<20150213221450.GA24285@glandium.org>
In-Reply-To
<xmqqwq3lmwnb.fsf@gitster.dls.corp.google.com>
On Fri, Feb 13, 2015 at 11:36:24AM -0800, Junio C Hamano wrote:
Show 11 quoted lines
> Mike Hommey <mh@glandium.org> writes:
> 
> > A remote helper is currently only told about the 'check-connectivity',
> > 'cloning', and 'update-shallow' options when it supports the 'fetch'
> > command, but not when it supports 'import' instead.
> 
> Sounds sensible.
> 
> Does the same issue exist for export vs push or do they happen to be
> coded to pass similar enough set of options already by copied and
> pasted code?
The issue exists:
- export is given dry-run, pushcert and force.
- push is given cas, dry-run and pushcert.

(note: cas and pushcert are both not documented in gitremote-helpers.txt)

Force is actually not necessary for push, because the push syntax itself includes the force instruction in the refspec given as argument.

I haven't looked exactly what cas does and if it makes sense for export. (FWIW, I'm using push and import at the moment, so it's not a direct issue for me ; I don't support cas anyways)

Mike
Previous: Junio C HamanoNext: Junio C Hamano
Message 10 of 11 in “transport-helper: emit progress and verbosity options after asking for capabilities”
  1. 1/3 transport-helper: emit progress and verbosity options after asking for capabilitiesMike Hommey, Feb 12, 2015
  2. 2/3 transport-helper: emit check-connectivity, cloning, and update-shallow options for importMike Hommey, Feb 12, 2015
  3. 3/3 transport-helper: fix typo in error message when --signed is not supportedMike Hommey, Feb 12, 2015
  4. Mike HommeyFeb 12, 2015
  5. Junio C HamanoFeb 12, 2015
  6. Junio C HamanoFeb 12, 2015
  7. 1/2 transport-helper: ask the helper to set progress and verbosity options after asking for its capabilitiesMike Hommey, Feb 13, 2015
  8. 2/2 transport-helper: ask the helper to set the same options for import as for fetchMike Hommey, Feb 13, 2015
  9. Junio C HamanoFeb 13, 2015
  10. Mike HommeyFeb 13, 2015
  11. Junio C HamanoFeb 14, 2015

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.