I have a mixed feeling about this. From a longer-term perspective, do you really want this to be a part of git.git repository? I do not mind having notes to endorse and advocate "stats" as one of the "Third party packages that may make your git life more pleasuable", just like tig, stgit, guilt and topgit, but I cannot convince myself that merging it as a subtree is the right thing to do at this point. The "stats" tool, at least at the conceptual level, shares one important property with tools like gitk and gitweb: it could be useful to people whose sources are not in git repositories but in say Hg or Bzr, with some effort. The code may need refactoring to make it easier to plug in different backends and writing actual backends (aka "porting"), but that is something you can expect people with different backends to help you with. However, it would be awkward for the contrib/ area in git.git to carry a lot of code that are only needed to produce stat data from non-git repositories, once such a porting effort begins. It's perfectly fine if you are not interested in any of the other backends, and tell the people that they are welcome to fork it never to merge back. But if this were my brainchild, I'd imagine I'd be wishing to be able to buy back the improvements to the "core stats" parts that are done by people with other backends. I would imagine binding the current code as part of git.git would make such improvements harder to manage, both for you (who wants to buy back the changes made by others on different backends) and for others on different backends (who want to merge the changes made by you to their forks). Perhaps pointing at your tree as a submodule would be the right thing to do; then git.git proper will be just one of the users of "stats" tool. How about making that as a mid-to-longer term goal? When we eject git-gui and gitk from git.git and make them a submodule (wasn't that supposed to happen in 1.8 or 2.0 timeframe?), we may also add "stats" as a submodule? -- 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
