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

Re: [PATCH v2 try2 3/4] completion: bash: fix for multiple dash commands

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 8, 2021, 05:15 UTC
Message-ID
<xmqq35ttq6na.fsf@gitster.g>
In-Reply-To
<60be6f7fa4435_db80d208f2@natae.notmuch>
Felipe Contreras <felipe.contreras@gmail.com> writes:
Show 7 quoted lines
> SZEDER Gábor wrote:
>> On Sun, Jun 06, 2021 at 01:47:25PM -0500, Felipe Contreras wrote:
>> > Otherwise commands like 'for-each-ref' are not completed.
>> 
>> I think you meant that options of commands like 'for-each-ref' are not
>> completed.
>> ...
> Yes, I meant commands like 'for-each-ref' are not completed correctly...
> by __gitcomp_builtin. Which in practical tearms means what you just
> described above.

I couldn't exactly tell if that's a roundabout way to say "yes, yours is better and I'll use it in an update, thanks". I'll assume that is the case and will mark the topic as expecting the final update (as it seemed that all the other patches in the series were received favourably).

Or was it meant as "no, the version you commented on was already correct (if you squint your eyes)"?

Thanks.
Previous: Felipe ContrerasNext: Felipe Contreras
Message 7 of 16 in “completion: bash: a bunch of fixes”
  1. 0/4 completion: bash: a bunch of fixesFelipe Contreras, Jun 6, 2021
  2. 2/4 completion: bash: fix for suboptions with valueFelipe Contreras, Jun 6, 2021
  3. 1/4 completion: bash: fix prefix detection in branch.*Felipe Contreras, Jun 6, 2021
  4. 3/4 completion: bash: fix for multiple dash commandsFelipe Contreras, Jun 6, 2021
  5. SZEDER GáborJun 7, 2021
  6. Felipe ContrerasJun 7, 2021
  7. Junio C HamanoJun 8, 2021
  8. Felipe ContrerasJun 8, 2021
  9. 4/4 completion: bash: add correct suffix in variablesFelipe Contreras, Jun 6, 2021
  10. David AguilarJun 7, 2021
  11. Junio C HamanoJun 7, 2021
  12. 0/4 completion: bash: a bunch of fixesFelipe Contreras, Jun 8, 2021
  13. 2/4 completion: bash: fix for suboptions with valueFelipe Contreras, Jun 8, 2021
  14. 3/4 completion: bash: fix for multiple dash commandsFelipe Contreras, Jun 8, 2021
  15. 1/4 completion: bash: fix prefix detection in branch.*Felipe Contreras, Jun 8, 2021
  16. 4/4 completion: bash: add correct suffix in variablesFelipe Contreras, Jun 8, 2021

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.