releasing version 3.20100623

[git.ikiwiki.info.git] / IkiWiki / Plugin / bzr.pm

diff --git a/IkiWiki/Plugin/bzr.pm b/IkiWiki/Plugin/bzr.pm
index 3712302cea5d7fdbdfe1738073940b4b2476d50b..9bee0c4b27611eda8966a05c773cd6e7930802d9 100644 (file)
--- a/IkiWiki/Plugin/bzr.pm
+++ b/IkiWiki/Plugin/bzr.pm
@@ -286,8 +286,8 @@ sub rcs_diff ($) {
        }
 }
 
        }
 }
 

-sub extract_timestamp ($) {
-       my ($out) = @_;
+sub extract_timestamp (@) {
+       open (my $out, "-|", @_);

        my @log = bzr_log($out);
 
        if (length @log < 1) {
        my @log = bzr_log($out);
 
        if (length @log < 1) {


@@ -297,28 +297,22 @@ sub extract_timestamp ($) {
        eval q{use Date::Parse};
        error($@) if $@;
        
        eval q{use Date::Parse};
        error($@) if $@;
        

-       my $ctime = str2time($log[0]->{"timestamp"});
-       return $ctime;
+       my $time = str2time($log[0]->{"timestamp"});
+       return $time;

 }
 
 sub rcs_getctime ($) {
        my ($file) = @_;
 
 }
 
 sub rcs_getctime ($) {
        my ($file) = @_;
 

-       # XXX filename passes through the shell here, should try to avoid
-       # that just in case

        my @cmdline = ("bzr", "log", "--forward", "--limit", '1', "$config{srcdir}/$file");
        my @cmdline = ("bzr", "log", "--forward", "--limit", '1', "$config{srcdir}/$file");

-       open (my $out, "@cmdline |");
-       return extract_timestamp($out);
+       return extract_timestamp(@cmdline);

 }
 
 sub rcs_getmtime ($) {
        my ($file) = @_;
 
 }
 
 sub rcs_getmtime ($) {
        my ($file) = @_;
 

-       # XXX filename passes through the shell here, should try to avoid
-       # that just in case

        my @cmdline = ("bzr", "log", "--limit", '1', "$config{srcdir}/$file");
        my @cmdline = ("bzr", "log", "--limit", '1', "$config{srcdir}/$file");

-       open (my $out, "@cmdline |");
-       return extract_timestamp($out);
+       return extract_timestamp(@cmdline);

 }
 
 1
 }
 
 1