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

Re: [PATCH 0/8] Make a couple of commands builtin

From
Lukas Sandström <lukass@etek.chalmers.se>
Date
Jun 13, 2006, 22:03 UTC
Message-ID
<448F3623.9010005@etek.chalmers.se>
In-Reply-To
<20060614005437.69ff6a62.tihirvon@gmail.com>
Timo Hirvonen wrote:
Show 14 quoted lines
> Lukas Sandström <lukass@etek.chalmers.se> wrote:
> 
>> This patchseries has the ultimate goal of making
>> git-am a builtin.
>>
>> The version of git-am I'm sending out makes quite heavy
>> use of system(), but I think that can be worked around.
>> I just haven't figured out how, yet.
> 
> I don't think git-stripspace needs to be a built-in.  It doesn't even
> depend on git.  It is just a tiny helper program used by git-am,
> git-applymbox, git-commit and git-tag.  If all these commands are made
> built-in then git-stripspace becomes useless.
> 
The reason I made it builtin was to de able to call it easily from git-am.

As you say, once all the users are builtin it could be removed, unless somone is using it in their scripts.

/Lukas
Previous: Timo Hirvonen
Message 3 of 3 in “Make a couple of commands builtin”
  1. 0/8 Make a couple of commands builtinLukas Sandström, Jun 13, 2006
  2. Timo HirvonenJun 13, 2006
  3. Lukas SandströmJun 13, 2006

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.