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

Re: [PATCH v5 06/10] fast-export: add new --refspec option

From
Junio C Hamano <gitster@pobox.com>
Date
Nov 6, 2013, 22:41 UTC
Message-ID
<xmqqmwlh6tlh.fsf@gitster.dls.corp.google.com>
In-Reply-To
<xmqqr4at6u24.fsf@gitster.dls.corp.google.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 26 quoted lines
> Jeff King <peff@peff.net> writes:
>
>> I think that is much nicer for the simple cases, but how do we handle
>> more complex rev expressions? One can say:
>>
>>   git fast-export master ^origin
>>
>> or even:
>>
>>   git fast-export origin..master
>>
>> The "^origin" is not a refspec, and finding the refspec in the
>> dot-expression would involve parsing it into two components. I think you
>> can come up with a workable system by parsing the arguments as revision
>> specifiers and then applying some rules. E.g., a positive ref "foo" is a
>> refspec "foo:foo", but negative "^bar" does not impact refspecs at all,
>> and the same rules are applied for "bar..foo". There is a syntactic
>> conflict where "foo:bar" would be interpreted as a tree:path by the
>> revision code, though, whereas in this context it means a refspec.
>>
>> So I think it is possible to go that route (and perhaps preferable,
>> even, because it keeps the simple and common cases easy for the user),
>> but the implementation is not as simple as just treating the arguments
>> as refspecs.
>
> I forgot about the ranges, but thinking about it further, I think ...

And I am an idiot; you've spelled out everything I wanted to say, so I should have just dropped this ;-)

Previous: Junio C HamanoNext: Junio C Hamano
Message 24 of 45 in “transport-helper: add support to delete branches”
  1. 10/10 transport-helper: add support to delete branchesFelipe Contreras, Oct 31, 2013
  2. 00/10 transport-helper: updatesFelipe Contreras, Oct 31, 2013
  3. Max HornOct 31, 2013
  4. Junio C HamanoOct 31, 2013
  5. Felipe ContrerasOct 31, 2013
  6. Junio C HamanoOct 31, 2013
  7. Max HornOct 31, 2013
  8. 11/10 test-hg.sh: tests are now expected to passRichard Hansen, Nov 11, 2013
  9. 12/10 remote-bzr: support the new 'force' optionRichard Hansen, Nov 11, 2013
  10. Felipe ContrerasNov 11, 2013
  11. Richard HansenNov 11, 2013
  12. Felipe ContrerasNov 11, 2013
  13. 01/10 transport-helper: fix extra linesFelipe Contreras, Oct 31, 2013
  14. Junio C HamanoOct 31, 2013
  15. 08/10 fast-import: add support to delete refsFelipe Contreras, Oct 31, 2013
  16. Max HornOct 31, 2013
  17. 05/10 fast-export: improve argument parsingFelipe Contreras, Oct 31, 2013
  18. 06/10 fast-export: add new --refspec optionFelipe Contreras, Oct 31, 2013
  19. Junio C HamanoOct 31, 2013
  20. Felipe ContrerasOct 31, 2013
  21. Junio C HamanoNov 6, 2013
  22. Jeff KingNov 6, 2013
  23. Junio C HamanoNov 6, 2013
  24. Junio C HamanoNov 6, 2013
  25. Junio C HamanoNov 8, 2013
  26. Jeff KingNov 9, 2013
  27. Felipe ContrerasNov 7, 2013
  28. 03/10 transport-helper: add 'force' to 'export' helpersFelipe Contreras, Oct 31, 2013
  29. Junio C HamanoOct 31, 2013
  30. Felipe ContrerasOct 31, 2013
  31. Junio C HamanoOct 31, 2013
  32. Junio C HamanoNov 1, 2013
  33. Felipe ContrerasNov 1, 2013
  34. Junio C HamanoNov 1, 2013
  35. Eric SunshineNov 1, 2013
  36. fixup! transport-helper: add 'force' to 'export' helpersRichard Hansen, Nov 11, 2013
  37. 09/10 fast-export: add support to delete refsFelipe Contreras, Oct 31, 2013
  38. Max HornOct 31, 2013
  39. Felipe ContrerasOct 31, 2013
  40. Max HornOct 31, 2013
  41. Felipe ContrerasOct 31, 2013
  42. Max HornOct 31, 2013
  43. 07/10 transport-helper: add support for old:new refspecFelipe Contreras, Oct 31, 2013
  44. 04/10 transport-helper: check for 'forced update' messageFelipe Contreras, Oct 31, 2013
  45. 02/10 transport-helper: don't update refs in dry-runFelipe Contreras, Oct 31, 2013

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.