From 57f2d6fa3136f1a599612832f7aeba6370bf3758 Mon Sep 17 00:00:00 2001
From:
"https://www.google.com/accounts/o8/id?id=AItOawmUWmB1M35_jviFvGPYDIH-a-_Al-7OrXM"
<Daniel@web>
Date: Fri, 15 Jul 2011 21:28:00 -0400
Subject: [PATCH]
---
...ve_commit_messages_for_removing_pages.mdwn | 22 +++++++++++++++++++
1 file changed, 22 insertions(+)
create mode 100644 doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn
diff --git a/doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn b/doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn
new file mode 100644
index 000000000..f000878b7
--- /dev/null
+++ b/doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn
@@ -0,0 +1,22 @@
+As [[Add instructive commit messages for add _47_ edit pages]], but for `remove.pm`.
+
+I use a `join()` since it at least looks like the plugin is able to remove several pages at once (`foreach` looping over file parameters), thus holding multiple entries in `@pages`. I haven't seen this happen, though.
+
+Diff follows. --[[Daniel Andersson]]
+
+[[!tag patch]]
+
+---
+
+ diff -r 4f2ad3a5377e Plugin/remove.pm
+ --- a/Plugin/remove.pm Fri Jul 15 17:39:04 2011 +0200
+ +++ b/Plugin/remove.pm Sat Jul 16 03:20:35 2011 +0200
+ @@ -228,7 +228,7 @@
+ IkiWiki::rcs_remove($file);
+ }
+ IkiWiki::rcs_commit_staged(
+ - message => gettext("removed"),
+ + message => sprintf(gettext("remove %s"), join(', ', @files)),
+ session => $session,
+ );
+ IkiWiki::enable_commit_hook();
--
2.39.5