From ce1f43822a68c341b820af61445a18725a6e5230 Mon Sep 17 00:00:00 2001 From: "http://smcv.pseudorandom.co.uk/" Date: Sat, 4 Jan 2014 10:28:35 -0400 Subject: [PATCH] already fixed --- ...omment_don__39__t_make_it_through_git.mdwn | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/doc/bugs/Webedits_without_comment_don__39__t_make_it_through_git.mdwn b/doc/bugs/Webedits_without_comment_don__39__t_make_it_through_git.mdwn index ac75c441a..2bfe71e87 100644 --- a/doc/bugs/Webedits_without_comment_don__39__t_make_it_through_git.mdwn +++ b/doc/bugs/Webedits_without_comment_don__39__t_make_it_through_git.mdwn @@ -29,3 +29,22 @@ to push @opts, '--allow-empty-message'; [[!tag bugs patch]] + +> This is already [[fixed|done]] since 3.20130711. git versions since 1.7.2 +> are affected. Here's the commit if you want to backport it: +> [[b162563|http://source.ikiwiki.branchable.com/?p=source.git;a=commitdiff;h=b162563dc1c6126953e66cdcc508f389b9d39d8e]]. +> +> As a general comment on synthesizing commit messages, I personally don't +> think ikiwiki should invent an untranslated English commit message +> if the user didn't provide one - using an obviously trivial commit message, +> ".", seems more honest. OTOH, the `bzr` and `mercurial` plugins both use +> an untranslated "no message given", and `darcs` uses "empty message". +> It should either consistently use ".", or consistently use gettext(x) +> for some standardized value of x, perhaps "no message given". Joey, +> any preference? +> +> The other RCS plugins (`cvs`, `svn`, `tla`) never need to deal with an +> empty commit message because they prepend something like "web commit +> from smcv", so the message can never be empty. +> +> --[[smcv]] -- 2.39.2