Re: [PATCH 1/3] fast-export: add --signed-tags=warn-strip mode
On Mon, Apr 15, 2013 at 09:50:42PM -0700, Sverre Rabbelier wrote: On Mon, Apr 15, 2013 at 9:47 PM, Junio C Hamano gits...@pobox.com wrote: When you see 78 in the output and you know you have 92 tags in the repository, is that sufficient to let you go on, or do we also need an easy way to tell which ones are those 78 that were stripped and the remaining 14 were not stripped? There is no reason to worry about some signed tags are stripped but not others, so it feels that the number alone should be sufficient, I guess. If those remaining 14 weren't stripped, that is (at least at the moment) by definition because they are unsigned, annotated tags. Or because they were not exported? Perhaps 78 tags stripped, 92 exported in total. I think I prefer Jonathan's suggestion to this one if we need to change it. The reason I didn't do this initially was that I assumed that from a remote helper we would, in general, not be pushing any tags which already exist, so the number of tags to push will be small. Printing one message per tag also matches the current behaviour for --signed-tags=warn. I don't want to make the behaviour for warn and warn-strip different, so should warn also print a summary message instead of a message for each tag? -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/3] fast-export: add --signed-tags=warn-strip mode
John Keeping j...@keeping.me.uk writes: Printing one message per tag also matches the current behaviour for --signed-tags=warn. I don't want to make the behaviour for warn and warn-strip different,... That is a valid point. Nobody has complained that the current warning is too noisy, so perhaps the patch is good as-is? What do others think? -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/3] fast-export: add --signed-tags=warn-strip mode
On Tue, Apr 16, 2013 at 9:48 PM, Junio C Hamano gits...@pobox.com wrote: That is a valid point. Nobody has complained that the current warning is too noisy, so perhaps the patch is good as-is? Ah, hadn't realized that. Probably fine then. -- Cheers, Sverre Rabbelier -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/3] fast-export: add --signed-tags=warn-strip mode
On Sun, Apr 14, 2013 at 3:57 AM, John Keeping j...@keeping.me.uk wrote: This issues a warning while stripping signatures from signed tags, which allows us to use it as default behaviour for remote helpers which cannot specify how to handle signed tags. Perhaps it makes sense to instead count the number of signed tags and emit Stripped signature from %d tags? For example, for git.git it would be on the order of a hundred warning lines. -- Cheers, Sverre Rabbelier -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/3] fast-export: add --signed-tags=warn-strip mode
Sverre Rabbelier srabbel...@gmail.com writes: On Sun, Apr 14, 2013 at 3:57 AM, John Keeping j...@keeping.me.uk wrote: This issues a warning while stripping signatures from signed tags, which allows us to use it as default behaviour for remote helpers which cannot specify how to handle signed tags. Perhaps it makes sense to instead count the number of signed tags and emit Stripped signature from %d tags? For example, for git.git it would be on the order of a hundred warning lines. When you see 78 in the output and you know you have 92 tags in the repository, is that sufficient to let you go on, or do we also need an easy way to tell which ones are those 78 that were stripped and the remaining 14 were not stripped? There is no reason to worry about some signed tags are stripped but not others, so it feels that the number alone should be sufficient, I guess. If those remaining 14 weren't stripped, that is (at least at the moment) by definition because they are unsigned, annotated tags. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/3] fast-export: add --signed-tags=warn-strip mode
On Mon, Apr 15, 2013 at 9:47 PM, Junio C Hamano gits...@pobox.com wrote: When you see 78 in the output and you know you have 92 tags in the repository, is that sufficient to let you go on, or do we also need an easy way to tell which ones are those 78 that were stripped and the remaining 14 were not stripped? There is no reason to worry about some signed tags are stripped but not others, so it feels that the number alone should be sufficient, I guess. If those remaining 14 weren't stripped, that is (at least at the moment) by definition because they are unsigned, annotated tags. Or because they were not exported? Perhaps 78 tags stripped, 92 exported in total. -- Cheers, Sverre Rabbelier -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/3] fast-export: add --signed-tags=warn-strip mode
Junio C Hamano wrote: Sverre Rabbelier srabbel...@gmail.com writes: Perhaps it makes sense to instead count the number of signed tags and emit Stripped signature from %d tags? For example, for git.git it would be on the order of a hundred warning lines. When you see 78 in the output and you know you have 92 tags in the repository, is that sufficient to let you go on, or do we also need an easy way to tell which ones are those 78 that were stripped and the remaining 14 were not stripped? I suspect the actually relevant information is warning: stripping signature before pushing signed tags The count and the list of signed tags are not too important. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html