On Thu, 18 Feb 2010, Junio C Hamano wrote: > > I personally like the general direction you are going. One minor detail I > do not know about is whether to move "builtin.h" with others, or leave it > at the top. Also I don't particularly find "-I. -Ibuiltin/" ugly. It > would make sense to use "-I." and rewrite our '#include "xyzzy.h"' into > '#include <xyzzy.h>' even if we were not doing this change. I was actually thinking of just changing everybody to use -#include "builtin.h" +#include "builtin/cmd.h" and I even did that with a sed script, but then the renames aren't pure renames any more, so I decided that in order to make the patch be a minimal rename thing, that would be a second phase. 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