diff options
| -rw-r--r-- | IkiWiki/Plugin/bzr.pm | 4 | ||||
| -rw-r--r-- | IkiWiki/Plugin/mercurial.pm | 10 | ||||
| -rw-r--r-- | debian/changelog | 1 |
3 files changed, 6 insertions, 9 deletions
diff --git a/IkiWiki/Plugin/bzr.pm b/IkiWiki/Plugin/bzr.pm index e7c1b8d8e..9bee0c4b2 100644 --- a/IkiWiki/Plugin/bzr.pm +++ b/IkiWiki/Plugin/bzr.pm @@ -287,9 +287,7 @@ sub rcs_diff ($) { } sub extract_timestamp (@) { - # XXX filename passes through the shell here, should try to avoid - # that just in case - open (my $out, "@_ |"); + open (my $out, "-|", @_); my @log = bzr_log($out); if (length @log < 1) { diff --git a/IkiWiki/Plugin/mercurial.pm b/IkiWiki/Plugin/mercurial.pm index 34e009c7a..a80bb2da5 100644 --- a/IkiWiki/Plugin/mercurial.pm +++ b/IkiWiki/Plugin/mercurial.pm @@ -236,15 +236,13 @@ sub rcs_diff ($) { sub rcs_getctime ($) { my ($file) = @_; - # XXX filename passes through the shell here, should try to avoid - # that just in case my @cmdline = ("hg", "-R", $config{srcdir}, "log", "-v", - "--style", "default", "$config{srcdir}/$file"); - open (my $out, "@cmdline |"); + "--style", "default", $file); + open (my $out, "-|", @cmdline); - my @log = mercurial_log($out); + my @log = (mercurial_log($out)); - if (length @log < 1) { + if (@log < 1) { return 0; } diff --git a/debian/changelog b/debian/changelog index 0d49df676..1afc620ab 100644 --- a/debian/changelog +++ b/debian/changelog @@ -23,6 +23,7 @@ ikiwiki (3.20100611) UNRELEASED; urgency=low * attachment: Support Windows paths when taking basename of client-supplied file name. * theme: New plugin, allows easily theming a site via the underlay. + * mercurial: Fix buggy getctime code. Closes: #586279 -- Joey Hess <joeyh@debian.org> Fri, 11 Jun 2010 13:39:15 -0400 |