# [PATCH 0/8] Make a couple of commands builtin 3 messages from 2006-06-13 to 2006-06-13. Participants: Lukas Sandström, Timo Hirvonen. Thread: https://gitlist.dev/t/4500 ## Lukas Sandström, 2006-06-13 20:21 Subject: [PATCH 0/8] Make a couple of commands builtin Message-ID: <448F1E41.1040607@etek.chalmers.se> URL: https://gitlist.dev/e/448F1E41.1040607%40etek.chalmers.se ``` 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. /Lukas ``` ## Timo Hirvonen, 2006-06-13 21:54 Subject: Re: [PATCH 0/8] Make a couple of commands builtin Message-ID: <20060614005437.69ff6a62.tihirvon@gmail.com> URL: https://gitlist.dev/e/20060614005437.69ff6a62.tihirvon%40gmail.com In-Reply-To: <448F1E41.1040607@etek.chalmers.se> ``` Lukas Sandström 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. -- http://onion.dynserv.net/~timo/ ``` ## Lukas Sandström, 2006-06-13 22:03 Subject: Re: [PATCH 0/8] Make a couple of commands builtin Message-ID: <448F3623.9010005@etek.chalmers.se> URL: https://gitlist.dev/e/448F3623.9010005%40etek.chalmers.se In-Reply-To: <20060614005437.69ff6a62.tihirvon@gmail.com> ``` Timo Hirvonen wrote: > Lukas Sandström 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 ```