Re: [PATCH v2 1/4] Consistently use the term "builtin" instead of "internal command"

2014-01-22 Thread Sebastian Schuberth
On 02.01.2014 22:05, Sebastian Schuberth wrote: would just leave me wondering "I never claimed it was built-in; what's going on?" I think it would be simplest to keep it as $ git whatever fatal: cannot handle "whatever" internally which at least makes it clear that this is a

Re: [PATCH v2 1/4] Consistently use the term "builtin" instead of "internal command"

2014-01-02 Thread Sebastian Schuberth
On Thu, Jan 2, 2014 at 9:31 PM, Jonathan Nieder wrote: > would just leave me wondering "I never claimed it was built-in; what's > going on?" I think it would be simplest to keep it as > > $ git whatever > fatal: cannot handle "whatever" internally > > which at least makes it clea

Re: [PATCH v2 1/4] Consistently use the term "builtin" instead of "internal command"

2014-01-02 Thread Jonathan Nieder
Hi, Sebastian Schuberth wrote: [...] > --- a/Documentation/technical/api-builtin.txt > +++ b/Documentation/technical/api-builtin.txt > @@ -14,7 +14,7 @@ Git: > > . Add the external declaration for the function to `builtin.h`. > > -. Add the command to `commands[]` table in `handle_internal_c