already fixed
authorhttp://smcv.pseudorandom.co.uk/ <smcv@web>
Sat, 4 Jan 2014 14:28:35 +0000 (10:28 -0400)
committeradmin <admin@branchable.com>
Sat, 4 Jan 2014 14:28:35 +0000 (10:28 -0400)
doc/bugs/Webedits_without_comment_don__39__t_make_it_through_git.mdwn

index ac75c441a75ef8670b10798e7260ee48d6d606b5..2bfe71e8715d0599dadc6eaec57be94cba8ab3d4 100644 (file)
@@ -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]]