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

Re: [PATCH] Do not call built-in aliases from scripts

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jun 28, 2013, 20:32 UTC
Message-ID
<alpine.DEB.1.00.1306282231300.28957@s15462909.onlinehome-server.info>
In-Reply-To
<CAHGBnuOE6vYzM2bKs2SV1qJVoCOX=t-XWnd_Xbs1JRD+sY4dVQ@mail.gmail.com>
Hi Sebastian,
On Fri, 28 Jun 2013, Sebastian Schuberth wrote:
Show 20 quoted lines
> On Thu, Jun 27, 2013 at 8:52 PM, Johannes Schindelin
> <Johannes.Schindelin@gmx.de> wrote:
> 
> >> --- a/git-merge-octopus.sh
> >> +++ b/git-merge-octopus.sh
> >> @@ -97,7 +97,7 @@ do
> >>       if test $? -ne 0
> >>       then
> >>               echo "Simple merge did not work, trying automatic merge."
> >> -             git-merge-index -o git-merge-one-file -a ||
> >> +             git merge-index -o git-merge-one-file -a ||
> >
> > This is a problem. 'git-merge-one-file' cannot be split here AFAICT.
> >
> > Of course, we could teach merge-index to read *two* parameters instead of
> > one when it encounters "git" as the <merge-program>. But that would be as
> > hacky as the whole dashed-form business to begin with.
> 
> I agree to all of your comments except this one: I did not split
> 'git-merge-one-file' here ...

I know. That is what I pointed out. git-merge-one-file was *not* "un-dashed". And I explained the reason, too.

But if we really want to solve the problem you described earlier, we also must not rely on git-merge-one-file to be present in libexec.

Ciao, Dscho

-- 
-- 
*** Please reply-to-all at all times ***
*** (do not pretend to know who is subscribed and who is not) ***
*** Please avoid top-posting. ***
The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.

You received this message because you are subscribed to the Google
Groups "msysGit" group.
To post to this group, send email to msysgit@googlegroups.com
To unsubscribe from this group, send email to
msysgit+unsubscribe@googlegroups.com
For more options, and view previous threads, visit this group at
http://groups.google.com/group/msysgit?hl=en_US?hl=en

--- 
You received this message because you are subscribed to the Google Groups "msysGit" group.
To unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
Previous: Sebastian SchuberthNext: Junio C Hamano
Message 12 of 14 in “Do not call built-in aliases from scripts”
  1. Do not call built-in aliases from scriptsSebastian Schuberth, Jun 27, 2013
  2. Junio C HamanoJun 27, 2013
  3. Sebastian SchuberthJun 27, 2013
  4. Junio C HamanoJun 27, 2013
  5. John SzakmeisterJun 27, 2013
  6. Junio C HamanoJun 27, 2013
  7. Junio C HamanoJun 27, 2013
  8. Sebastian SchuberthJun 28, 2013
  9. John KeepingJun 27, 2013
  10. Johannes SchindelinJun 27, 2013
  11. Sebastian SchuberthJun 28, 2013
  12. Johannes SchindelinJun 28, 2013
  13. Junio C HamanoJun 28, 2013
  14. Johannes SixtJun 27, 2013

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.