]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commitdiff
* Apply patch from hb that uses the full path to a file when adding a file
authorjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sun, 29 Apr 2007 03:25:09 +0000 (03:25 +0000)
committerjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sun, 29 Apr 2007 03:25:09 +0000 (03:25 +0000)
  using the mercurial backend. Not 100% sure why it failed w/o the full
  path, but this still passes the test suite, and indeed, is how the test
  suite calls hg add.

IkiWiki/Rcs/mercurial.pm
debian/changelog
doc/bugs/mercurial_fail_to_add.mdwn [new file with mode: 0644]
doc/patchqueue/mercurial_fail_to_add.mdwn [deleted file]

index 2e15085ecca097c518b3525297833a10ada82c99..3a75b40c5bbdc564877f48fc87e2a2fdc3207334 100644 (file)
@@ -95,7 +95,7 @@ sub rcs_commit ($$$;$$) { #{{{
 sub rcs_add ($) { # {{{
        my ($file) = @_;
 
-       my @cmdline = ("hg", "-q", "-R", "$config{srcdir}", "add", "$file");
+       my @cmdline = ("hg", "-q", "-R", "$config{srcdir}", "add", "$config{srcdir}/$file");
        if (system(@cmdline) != 0) {
                warn "'@cmdline' failed: $!";
        }
index 22de088545c1dc097daf37a8bb945b51c4fc81bf..1a14021c4166b9b6d0f51ce1f853f27f8c38db5a 100644 (file)
@@ -44,8 +44,12 @@ ikiwiki (1.51) UNRELEASED; urgency=low
   * Updated Polish translation from Pawel Tecza.
   * Updated Czech translation from Miroslav Kure.
   * Updated Spanish translation from Victor Moral.
+  * Apply patch from hb that uses the full path to a file when adding a file
+    using the mercurial backend. Not 100% sure why it failed w/o the full
+    path, but this still passes the test suite, and indeed, is how the test
+    suite calls hg add.
 
- -- Joey Hess <joeyh@debian.org>  Sat, 28 Apr 2007 22:47:38 -0400
+ -- Joey Hess <joeyh@debian.org>  Sat, 28 Apr 2007 23:51:35 -0400
 
 ikiwiki (1.50) unstable; urgency=low
 
diff --git a/doc/bugs/mercurial_fail_to_add.mdwn b/doc/bugs/mercurial_fail_to_add.mdwn
new file mode 100644 (file)
index 0000000..d47960e
--- /dev/null
@@ -0,0 +1,26 @@
+I don't know what's wrong but I can't add a file with the mercurial backend (the file is created but not added).
+
+Here is a patch that's seems to work, although I'm not quite sure what's wrong with current code :
+
+    hbernard@tactic:/usr/share/perl5/IkiWiki/Rcs$ diff mercurial.pm /home/hbernard/mercurial.pm -Nau
+    --- mercurial.pm        2007-03-24 16:14:35.000000000 +0100
+    +++ /home/hbernard/mercurial.pm 2007-04-19 19:05:47.000000000 +0200
+    @@ -95,7 +95,7 @@
+     sub rcs_add ($) { # {{{
+            my ($file) = @_;
+     
+    -       my @cmdline = ("hg", "-q", "-R", "$config{srcdir}", "add", "$file");
+    +       my @cmdline = ("hg", "-q", "-R", "$config{srcdir}", "add", "$config{srcdir}/$file");
+            if (system(@cmdline) != 0) {
+                    warn "'@cmdline' failed: $!";
+           }
+
+My srcdir path has some symbolics links and hidden directorys... maybe that's it ?
+
+> Interesting, the mercurial test suite shows the add without the path
+> working ok. OTOH, it also continues to work if I apply your patch, so I
+> guess it's safe enough. It would be good to know why it's failing w/o the
+> path in your case. --[[Joey]]
+
+> As I applied the patch, I'm moving this out of the patchqueue to bugs and
+> marking it [[done]]. --[[Joey]]
diff --git a/doc/patchqueue/mercurial_fail_to_add.mdwn b/doc/patchqueue/mercurial_fail_to_add.mdwn
deleted file mode 100644 (file)
index 86ee6ab..0000000
+++ /dev/null
@@ -1,18 +0,0 @@
-I don't know what's wrong but I can't add a file with the mercurial backend (the file is created but not added).
-
-Here is a patch that's seems to work, although I'm not quite sure what's wrong with current code :
-
-    hbernard@tactic:/usr/share/perl5/IkiWiki/Rcs$ diff mercurial.pm /home/hbernard/mercurial.pm -Nau
-    --- mercurial.pm        2007-03-24 16:14:35.000000000 +0100
-    +++ /home/hbernard/mercurial.pm 2007-04-19 19:05:47.000000000 +0200
-    @@ -95,7 +95,7 @@
-     sub rcs_add ($) { # {{{
-            my ($file) = @_;
-     
-    -       my @cmdline = ("hg", "-q", "-R", "$config{srcdir}", "add", "$file");
-    +       my @cmdline = ("hg", "-q", "-R", "$config{srcdir}", "add", "$config{srcdir}/$file");
-            if (system(@cmdline) != 0) {
-                    warn "'@cmdline' failed: $!";
-           }
-
-My srcdir path has some symbolics links and hidden directorys... maybe that's it ?