Junio C Hamano <gitster@xxxxxxxxx> writes: > Alexander Kuleshov <kuleshovmail@xxxxxxxxx> writes: > >> * "cache.h" - is unnecessary because it already included at "builtin.h" >> >> * "exec_cmd.h" - was added at a47d1813 (Allow a relative builtin template >> directory., 15 Nov 2007). init-db used 'git_exec_path' routine from >> "exec_cmd.h", but later it was removed at 2de9de5e (Move code interpreting >> path relative to exec-dir to new function system_path()., 14 Jul 2008). So >> we no need in it anymore. >> >> Signed-off-by: Alexander Kuleshov <kuleshovmail@xxxxxxxxx> >> --- > > Makes sense. Thanks. Actually, not. 2de9de5e does make it unnecessary to refer to git_exec_path(), but in order to reference the newly introduced system_path(), which is declared in exec_cmd.h, the file still would have needed to include the header, so the justification for the latter does not make sense. And indeed, we still need exec_cmd.h in today's code. -- 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