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

Re: What's cooking in git.git (Jun 2012, #01; Sun, 3)

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Jun 6, 2012, 12:22 UTC
Message-ID
<CAMP44s0cQe3jdfzbJmFVoFS50Ho5hVPg9dpZx0KhG3sP_J8=_g@mail.gmail.com>
In-Reply-To
<7vr4twudqm.fsf@alter.siamese.dyndns.org>
On Mon, Jun 4, 2012 at 2:23 AM, Junio C Hamano <gitster@pobox.com> wrote:
Show 10 quoted lines
> * fc/git-prompt-script (2012-05-22) 5 commits
>  - completion: split __git_ps1 into a separate script
>  - completion: remove executable mode
>  - Merge branch 'fc/git-complete-helper' into fc/git-prompt-script
>  - tests: add tests for the bash prompt functions in the completion script
>  - tests: move code to run tests under bash into a helper library
>  (this branch is tangled with sg/bash-prompt.)
>
> The last remaining sticking point is what to do with the duplicated shell
> function.

What is the problem with leaving it as it is; having it as a duplicate function. It's not a *huge* maintenance burden, and it's a big problem if the functions diverge.

I still plan to add a native helper for this, but I don't see what that would block these patches.

-- 
Felipe Contreras
Previous: Junio C HamanoNext: Junio C Hamano
Message 2 of 12 in “What's cooking in git.git (Jun 2012, #01; Sun, 3)”
  1. Junio C HamanoJun 4, 2012
  2. Felipe ContrerasJun 6, 2012
  3. Junio C HamanoJun 6, 2012
  4. Felipe ContrerasJun 6, 2012
  5. Junio C HamanoJun 10, 2012
  6. Felipe ContrerasJun 10, 2012
  7. Junio C HamanoJun 11, 2012
  8. Felipe ContrerasJun 13, 2012
  9. Junio C HamanoJun 13, 2012
  10. Felipe ContrerasJun 13, 2012
  11. René ScharfeJun 10, 2012
  12. Felipe ContrerasJun 11, 2012

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.