On Sat, 27 Jan 2007, Junio C Hamano wrote: > > I would think we probably should reuse the --porcelain output, > perhaps enhancing it even more. I looked at using "emit_porcelain()" directly, but that format doesn't seem to actually be usable for incremental blame. For example: the porcelain depends on things like the MORE_THAN_ONE_PATH flag having been computed, which simply isn't known incrementally. Also, for the incremental blame, it makes no sense to actually print out the actual blame buffer: anybody who uses the incremental blame thing really needs to get the original buffer separately set up anyway. So one one hand, I agree: the output really should probably share a lot of the ideas with --porcelain. At the same time, the porcelain output as it is now is actually very non-sensible for the incremental case. (The "METAINFO_SHOWN" kind of logic works fine for --incremental, though. It's only the MORE_THAN_ONE_PATH things that don't really make sense until the end, since they are part of the discovery logic rather than part of the actual print-out logic. I guess it _works_, but still). I think the people who will care are the people who actually write some nice gui around it.. Linus - 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