oops, forgot some changes

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

diff --git a/IkiWiki/Plugin/monotone.pm b/IkiWiki/Plugin/monotone.pm
index 14c22ebca2f02c69f132ea581bb423aa29f20c24..b0bba5661541843bb1440bffe5c466ae88dc238d 100644 (file)
--- a/IkiWiki/Plugin/monotone.pm
+++ b/IkiWiki/Plugin/monotone.pm
@@ -7,8 +7,10 @@ use IkiWiki;
 use Monotone;
 use Date::Parse qw(str2time);
 use Date::Format qw(time2str);
 use Monotone;
 use Date::Parse qw(str2time);
 use Date::Format qw(time2str);

+use URI::Escape q{uri_escape_utf8};

 
 my $sha1_pattern = qr/[0-9a-fA-F]{40}/; # pattern to validate sha1sums
 
 my $sha1_pattern = qr/[0-9a-fA-F]{40}/; # pattern to validate sha1sums

+my $mtn_version = undef;

 
 sub import {
        hook(type => "checkconfig", id => "monotone", call => \&checkconfig);
 
 sub import {
        hook(type => "checkconfig", id => "monotone", call => \&checkconfig);


@@ -40,20 +42,19 @@ sub checkconfig () {
                exec("mtn", "version") || error("mtn version failed to run");
        }
 
                exec("mtn", "version") || error("mtn version failed to run");
        }
 

-       my $version=undef;

        while (<MTN>) {
                if (/^monotone (\d+\.\d+)(?:(?:\.\d+){0,2}|dev)? /) {
        while (<MTN>) {
                if (/^monotone (\d+\.\d+)(?:(?:\.\d+){0,2}|dev)? /) {

-                       $version=$1;
+                       $mtn_version=$1;

                }
        }
 
        close MTN || debug("mtn version exited $?");
 
                }
        }
 
        close MTN || debug("mtn version exited $?");
 

-       if (!defined($version)) {
+       if (!defined($mtn_version)) {

                error("Cannot determine monotone version");
        }
                error("Cannot determine monotone version");
        }

-       if ($version < 0.38) {
-               error("Monotone version too old, is $version but required 0.38");
+       if ($mtn_version < 0.38) {
+               error("Monotone version too old, is $mtn_version but required 0.38");

        }
 
        if (defined $config{mtn_wrapper} && length $config{mtn_wrapper}) {
        }
 
        if (defined $config{mtn_wrapper} && length $config{mtn_wrapper}) {


@@ -309,10 +310,10 @@ sub commitauthor (@) {
 
        if (defined $params{session}) {
                if (defined $params{session}->param("name")) {
 
        if (defined $params{session}) {
                if (defined $params{session}->param("name")) {

-                       return "Web user: " . $params{session}->param("name");
+                       return "Web user: " . IkiWiki::cloak($params{session}->param("name"));

                }
                elsif (defined $params{session}->remote_addr()) {
                }
                elsif (defined $params{session}->remote_addr()) {

-                       return "Web IP: " . $params{session}->remote_addr();
+                       return "Web IP: " . IkiWiki::cloak($params{session}->remote_addr());

                }
        }
        return "Web: Anonymous";
                }
        }
        return "Web: Anonymous";


@@ -593,7 +594,8 @@ sub rcs_recentchanges ($) {
                                my $diffurl=$config{diffurl};
                                $diffurl=~s/\[\[r1\]\]/$parent/g;
                                $diffurl=~s/\[\[r2\]\]/$rev/g;
                                my $diffurl=$config{diffurl};
                                $diffurl=~s/\[\[r1\]\]/$parent/g;
                                $diffurl=~s/\[\[r2\]\]/$rev/g;

-                               $diffurl=~s/\[\[file\]\]/$file/g;
+                               my $efile = uri_escape_utf8($file);
+                               $diffurl=~s/\[\[file\]\]/$efile/g;

                                push @pages, {
                                        page => pagename($file),
                                        diffurl => $diffurl,
                                push @pages, {
                                        page => pagename($file),
                                        diffurl => $diffurl,


@@ -662,9 +664,11 @@ sub rcs_getctime ($) {
                     "--brief", $file) || error("mtn log $file failed to run");
        }
 
                     "--brief", $file) || error("mtn log $file failed to run");
        }
 

+       my $prevRev;

        my $firstRev;
        while (<MTNLOG>) {
                if (/^($sha1_pattern)/) {
        my $firstRev;
        while (<MTNLOG>) {
                if (/^($sha1_pattern)/) {

+                       $prevRev=$firstRev;

                        $firstRev=$1;
                }
        }
                        $firstRev=$1;
                }
        }


@@ -678,6 +682,17 @@ sub rcs_getctime ($) {
        my $automator = Monotone->new();
        $automator->open(undef, $config{mtnrootdir});
 
        my $automator = Monotone->new();
        $automator->open(undef, $config{mtnrootdir});
 

+       # mtn 0.48 has a bug that makes it list the creation of parent
+       # directories as last (first) log entry...  So when we're dealing
+       # with that version, let's check that the file we're looking for
+       # is actually part of the last (first) revision.  Otherwise, pick
+       # the one before (after) that one.
+       if ($mtn_version == 0.48) {
+               my $changes = [get_changed_files($automator, $firstRev)];
+               if (! exists {map { $_ => 1 } @$changes}->{$file}) {
+                       $firstRev = $prevRev;
+               }
+       }

        my $certs = [read_certs($automator, $firstRev)];
 
        $automator->close();
        my $certs = [read_certs($automator, $firstRev)];
 
        $automator->close();