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

Re: [PATCH] git-pull: don't complain about branch merge config if only fetching tags

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 27, 2007, 10:39 UTC
Message-ID
<7vwsr0ifqi.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<20071227093016.18337.qmail@e6d9fb481b7087.315fe32.mid.smarden.org>
Gerrit Pape <pape@smarden.org> writes:
>> I've seen this patch on this list in the past, but isn't "git
>> pull -t" a user-error?
>
> If so, we shouldn't advertise the -t switch in the git-pull(1) manpage.
Perhaps.  Patches are welcomed as always.
> As it stands, I'd understand 'git pull -t' as 'fetch _all_ tags and
> branch heads and merge according to the configuration'.

Yeah, but that would mean that the current error message needs to be kept, as that is the logical conclusion of "merge according to the configuration". Which is not so nice.

Perhaps using the same logic as your patch to detect the case of "pull --tags and nothing else" case, and exit with non-zero saying "you probably meant fetch --tags" would be a sane compromise? I dunno.

Previous: Gerrit PapeNext: Gerrit Pape
Message 4 of 9 in “git-pull: don't complain about branch merge config if only fetching tags”
  1. git-pull: don't complain about branch merge config if only fetching tagsGerrit Pape, Dec 21, 2007
  2. Junio C HamanoDec 21, 2007
  3. Gerrit PapeDec 27, 2007
  4. Junio C HamanoDec 27, 2007
  5. git-pull: warn if only fetching tags with the -t switchGerrit Pape, Dec 27, 2007
  6. Junio C HamanoDec 28, 2007
  7. Junio C HamanoDec 28, 2007
  8. Daniel BarkalowDec 28, 2007
  9. Junio C HamanoDec 28, 2007

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.