Hi, Johannes Schindelin wrote: > Hi, > > On Fri, 18 Jul 2008, Stephan Beyer wrote: > > > launch_editor() is declared in strbuf.h but defined in builtin-tag.c. > > This patch moves launch_editor() into a new source file editor.c, but > > keeps the declaration in strbuf.h. > > Sorry, but that has been tried before. Junio pointed out that > launch_editor() iss too die()-happy, and that the messages are too bound > to the current callers. Ah, a short note: this is not a real "libify" patch. It is more an "Oh, launch_editor is used for git-commit and git-tag, but it is only defined in builtin_tag.c, but declared (as libgit.a member) in strbuf.h, so let's move it" patch without further changes. But of course you and Junio are right. > You need to at least provide an add-on patch to fix both issues. Tomorrow, I need sleep first :) Regards. -- Stephan Beyer <s-beyer@xxxxxxx>, PGP 0x6EDDD207FCC5040F -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
