removing old patch that was also sent by mail

[ikiwiki.git] / doc / bugs / Monotone_rcs_support.mdwn

diff --git a/doc/bugs/Monotone_rcs_support.mdwn b/doc/bugs/Monotone_rcs_support.mdwn
index 116939722184bc685fd251dedcec78d71f57a40b..866805348eabc7494f6d3d8c2ab54b4155485d03 100644 (file)
--- a/doc/bugs/Monotone_rcs_support.mdwn
+++ b/doc/bugs/Monotone_rcs_support.mdwn
@@ -50,34 +50,7 @@ Here is another patch.  It fixes a FIXME you added.  I was using $file within ba
 I was getting an error trying to do it right.  I've figured out the error, and now do it right.  This
 should also speed things up (very slightly)
 
-    Index: IkiWiki/Rcs/monotone.pm
-    ===================================================================
-    --- IkiWiki/Rcs/monotone.pm        (revision 4234)
-    +++ IkiWiki/Rcs/monotone.pm        (working copy)
-    @@ -239,17 +239,11 @@
-     
-               # Something has been committed, has this file changed?
-               my ($out, $err);
-    -          #$automator->setOpts("-r", $oldrev, "-r", $rev);
-    -          #my ($out, $err) = $automator->call("content_diff", $file);
-    -          #debug("Problem committing $file") if ($err ne "");
-    -          # FIXME: use of $file in these backticks is not wise from a
-    -          # security POV. Probably safe, but should be avoided
-    -          # anyway.
-    -          # At the moment the backticks are used because the above call using the automate
-    -          # interface was failing.  When that bug in monotone is fixed, we should switch
-    -          # back.
-    -          my $diff = `mtn --root=$config{mtnrootdir} au content_diff -r $oldrev -r $rev $file`; # was just $out;
-    -
-    +          $automator->setOpts("r", $oldrev, "r", $rev);
-    +          ($out, $err) = $automator->call("content_diff", $file);
-    +          debug("Problem committing $file") if ($err ne "");
-    +          my $diff = $out;
-    +          
-               if ($diff) {
-                       # Commit a revision with just this file changed off
-                       # the old revision.
-
+> applied
 
 >> BTW, will all the monotone output parsing work if LANG != C?
 
@@ -95,3 +68,11 @@ should also speed things up (very slightly)
 
 >>>> So the rcs_notify support is not just untested, but can't work
 >>>> at all w/o further development.
+
+I've just done this further development...  The following patch adds support for diffurls.
+I've also partially tested the commit message support.  I was unable to get Ikiwiki to send
+change emails at all (cgi or otherwise), so I tested by adding warn() calls.  There were a
+few things that needed to be fixed.  Support is much closer now (including a simple default
+monotone lua hook).
+
+> We handled this patchset by mail, I applied the still-relevant parts. --[[Joey]]