On Nov 12, 2007 5:45 PM, Johannes Sixt <j.sixt@xxxxxxxxxxxxx> wrote: > Ping Yin schrieb: > > On Nov 12, 2007 11:59 PM, Johannes Sixt <j.sixt@xxxxxxxxxxxxx> wrote: > > > >> But at this time git-commit is about to be made a builtin, and since your > >> implementation contains a lot of non-portable constructs ($'', >&) and a new > >> dependency on awk (and, hence, has little chances of being accepted), I > >> suggest that you stay tuned, and implement this in the forth-coming > >> builtin-commit.c. > > Implement this in shell scripts is just a piece of cake, but not so > > easy in builtin-commit.c. > > I'm with you. But git-commit.sh is a dead horse, no matter how hard you beat it. > > BTW, maybe you can keep the log generation in a helper script, > git-status--submodulesummary, and invoke that from builtin-commit.c. This > way you get both: integration and ease of implementation. Sorry for repeating myself, but IMVHO this belong in git-submodule.sh: It can be useful as a standalone command, we get all submodule-related commands isolated in a single place, and builtin-commit.c can of course exec git-submodule to get the summary. -- larsh - 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
