fast-import part is clear and should be safe to apply. The test script is a bit ugly as it uses fsck and prune on a shared (with nearby tests) repo. But it should work, any hints on simplifying it are welcome. fsck part fixes an uncaught bad committer, but also changes error messages for some of bad committer strings. The messages choice is a subject for discussion most likely. Dmitry Ivankov (5): fast-import: add input format tests fast-import: don't fail on omitted committer name fast-import: check committer name more strictly fsck: add a few committer name tests fsck: improve committer/author check fast-import.c | 33 ++++++++++------ fsck.c | 10 +++-- t/t1450-fsck.sh | 24 ++++++++++++ t/t9300-fast-import.sh | 99 ++++++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 150 insertions(+), 16 deletions(-) -- 1.7.3.4 -- 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