Re: [PATCH] One Git To Rule Them All

>>>>> "JM" == McMullan, Jason <jason.mcmullan@timesys.com> writes:

JM> Am I missing something here?

No, but one thing I failed to mention was that I am not a big
fan of busybox.  It is a cute hack to save "disk" space in an
embedded environment, but I find the way many commands are
bundled together a bit too hacky for my taste to use on my
regular work box.  I also suspect the change makes the
maintenance of the result harder than having them separate.

You said with this change there is only one file to install, but
you are not bundling the git-* scripts in the "single git", so
that claim is not true either.  If your patch handled the git-*
script parts as well, then I would have admired that part of the
change ;-).

Personally I find the mandatory documentation requirements this
patch brings us is a good thing ;-).

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html