What is in the patch series looked more or less good to me. Lukas's sirname was still P in [3/5], a patch in [4/5] added an unnecssary blank line before git_verify_tag_config() and also a local variable declaration for "char *fmt_pretty" was indented funnily, but none of these were something I couldn't fix up locally. I however notice that there is no new tests to protect these two new features from future breakages. Perhaps you want to add some in [6/5]? Thanks.