On Wed, Aug 12, 2015 at 03:44:19PM +0200, Jason A. Donenfeld wrote:
> On Wed, Aug 12, 2015 at 3:03 PM, John Keeping <[email protected]> wrote:
> > This is an attempt at adding "blame" support to CGit.  Because git.git
> > doesn't include the blame implementation in libgit.a, I decided to
> > simply invoke git-blame(1) and parse its porcelain output.
> 
> Could you possibly just do:
> 
> #include "git/builtin/blame.c"
> 
> And then call the static functions?

I was trying to stay to a well defined interface.  It looks like we'd
have to copy reasonable chunks of cmd_blame() and emit_other() in order
to drive the static functions.

I'll give it a go and see what it looks like, but I expect it to result
in more pain during future maintenance than using a separate process.
_______________________________________________
CGit mailing list
[email protected]
http://lists.zx2c4.com/mailman/listinfo/cgit

Reply via email to