aboutsummaryrefslogtreecommitdiff
path: root/doc/bugs/RecentChanges_broken_with_empty_svnpath.mdwn
blob: c852df5e93e5bfdcf63aa414fe53e0206d8a8903 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
The [[RecentChanges]] page is broken (doesn't show any history at all) when used with an empty svnpath in the ikiwiki.setup file.

Say you have the following configuration:

    rcs => "svn",
    svnrepo => "ssh+svn://foo.bar.com/wiki",
    svnpath => "",

In the above, $svnpath need to be either empty or "/" - both trigger the 'next unless' check in IkiWiki/Rcs/svn.pm:rcs_recentchanges() as shown in the patch below, thus causing all files to be ignored for [[RecentChanges]].

I can not see why this check is needed in the first place, so here's a patch for removing it :)

    diff -upr ikiwiki-1.49.orig/IkiWiki/Rcs/svn.pm ikiwiki-1.49/IkiWiki/Rcs/svn.pm
    --- ikiwiki-1.49.orig/IkiWiki/Rcs/svn.pm	Mon Apr 16 15:15:09 2007
    +++ ikiwiki-1.49/IkiWiki/Rcs/svn.pm	Mon Apr 16 15:15:47 2007
    @@ -176,7 +176,6 @@ sub rcs_recentchanges ($) {
     		}
     
     		foreach (keys %{$logentry->{paths}}) {
    -			next unless /^\/\Q$config{svnpath}\E\/([^ ]+)(?:$|\s)/;
     			my $file=$1;
     			my $diffurl=$config{diffurl};
     			$diffurl=~s/\[\[file\]\]/$file/g;

> It's necessary for wikis, such as this one, that keep other things in the
> same svn repository. Bug [[fixed|done]]. --[[Joey]]