aboutsummaryrefslogtreecommitdiff
path: root/doc/patchqueue
diff options
context:
space:
mode:
authorjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>2006-11-08 06:42:08 +0000
committerjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>2006-11-08 06:42:08 +0000
commita2766fa0d7c4ba4bad775afc41545f028658b1d0 (patch)
tree21cb781ce6fd5ed72aa6b86c46106abbdab25b6f /doc/patchqueue
parent11479d71bc5d6ca9618781e1b715b1c129850f64 (diff)
downloadikiwiki-a2766fa0d7c4ba4bad775afc41545f028658b1d0.tar
ikiwiki-a2766fa0d7c4ba4bad775afc41545f028658b1d0.tar.gz
web commit by EthanGlasserCamp: use diff -ur so I can apply this easier in the future
Diffstat (limited to 'doc/patchqueue')
-rw-r--r--doc/patchqueue/pruning_is_too_strict.mdwn125
1 files changed, 61 insertions, 64 deletions
diff --git a/doc/patchqueue/pruning_is_too_strict.mdwn b/doc/patchqueue/pruning_is_too_strict.mdwn
index c9087ae90..0e56ef897 100644
--- a/doc/patchqueue/pruning_is_too_strict.mdwn
+++ b/doc/patchqueue/pruning_is_too_strict.mdwn
@@ -1,83 +1,80 @@
Preliminary patch for a feature wishlist item: [[bugs/pruning_is_too_strict]].
- diff --git a/IkiWiki.pm b/IkiWiki.pm
- index 1a00f2d..0119a26 100644
- --- a/IkiWiki.pm
- +++ b/IkiWiki.pm
- @@ -20,6 +20,8 @@ our $VERSION = 1.00;
+ diff -ur ikiwiki-orig/IkiWiki/CGI.pm ikiwiki/IkiWiki/CGI.pm
+ --- ikiwiki-orig/IkiWiki/CGI.pm 2006-10-27 20:15:17.000000000 -0700
+ +++ ikiwiki/IkiWiki/CGI.pm 2006-11-07 22:32:41.000000000 -0800
+ @@ -405,7 +405,7 @@
+ my ($page)=$form->field('page');
+ $page=titlepage(possibly_foolish_untaint($page));
+ if (! defined $page || ! length $page ||
+ - $page=~/$config{wiki_file_prune_regexp}/ || $page=~/^\//) {
+ + is_prune($page) || $page=~/^\//) {
+ error("bad page name");
+ }
+
+ @@ -495,8 +495,7 @@
+ my $best_loc;
+ if (! defined $from || ! length $from ||
+ $from ne $form->field('from') ||
+ - $from=~/$config{wiki_file_prune_regexp}/ ||
+ - $from=~/^\// ||
+ + is_prune($from) || $from=~/^\// ||
+ $form->submitted eq "Preview") {
+ @page_locs=$best_loc=$page;
+ }
+ diff -ur ikiwiki-orig/IkiWiki/Render.pm ikiwiki/IkiWiki/Render.pm
+ --- ikiwiki-orig/IkiWiki/Render.pm 2006-10-27 20:15:17.000000000 -0700
+ +++ ikiwiki/IkiWiki/Render.pm 2006-11-07 22:36:48.000000000 -0800
+ @@ -189,7 +193,7 @@
+ no_chdir => 1,
+ wanted => sub {
+ $_=decode_utf8($_);
+ - if (/$config{wiki_file_prune_regexp}/) {
+ + if (is_prune($_)) {
+ $File::Find::prune=1;
+ }
+ elsif (! -d $_ && ! -l $_) {
+ @@ -209,7 +213,7 @@
+ no_chdir => 1,
+ wanted => sub {
+ $_=decode_utf8($_);
+ - if (/$config{wiki_file_prune_regexp}/) {
+ + if (is_prune($_)) {
+ $File::Find::prune=1;
+ }
+ elsif (! -d $_ && ! -l $_) {
+ diff -ur ikiwiki-orig/IkiWiki.pm ikiwiki/IkiWiki.pm
+ --- ikiwiki-orig/IkiWiki.pm 2006-10-27 20:15:23.000000000 -0700
+ +++ ikiwiki/IkiWiki.pm 2006-11-07 22:21:17.000000000 -0800
+ @@ -21,6 +21,8 @@
# Optimisation.
use Memoize;
memoize("abs2rel");
+memoize("basefile");
+memoize("is_prune");
memoize("pagespec_translate");
-
+
my $installdir=''; # INSTALLDIR_AUTOREPLACE done by Makefile, DNE
- @@ -318,6 +320,22 @@ sub baseurl (;$) { #{{{
- return $page;
+ @@ -343,6 +352,22 @@
+ return $page;
} #}}}
-
+
+sub basefile ($;$) { #{{{
- + my $file=shift;
- + my $base=shift || $config{srcdir};
+ + my $file=shift;
+ + my $base=shift || $config{srcdir};
+
- + require File::Spec;
- + $base=File::Spec->canonpath($base);
- + my $ret=File::Spec->canonpath($file);
+ + require File::Spec;
+ + $base=File::Spec->canonpath($base);
+ + my $ret=File::Spec->canonpath($file);
+
- + $ret=~s#^$base/*##;
- + return $ret;
+ + $ret=~s#^$base/*##;
+ + return $ret;
+} #}}}
+
+sub is_prune ($;$) { #{{{
- + return basefile($_[0], $_[1])=~m/$config{wiki_file_prune_regexp}/;
+ + return basefile($_[0], $_[1])=~m/$config{wiki_file_prune_regexp}/;
+} #}}}
+
sub abs2rel ($$) { #{{{
- # Work around very innefficient behavior in File::Spec if abs2rel
- # is passed two relative paths. It's much faster if paths are
- diff --git a/IkiWiki/CGI.pm b/IkiWiki/CGI.pm
- index f550b67..5d1991d 100644
- --- a/IkiWiki/CGI.pm
- +++ b/IkiWiki/CGI.pm
- @@ -386,7 +386,7 @@ sub cgi_editpage ($$) { #{{{
- my ($page)=$form->field('page');
- $page=titlepage(possibly_foolish_untaint($page));
- if (! defined $page || ! length $page ||
- - $page=~/$config{wiki_file_prune_regexp}/ || $page=~/^\//) {
- + is_prune($page) || $page=~/^\//) {
- error("bad page name");
- }
-
- @@ -476,8 +476,7 @@ sub cgi_editpage ($$) { #{{{
- my $best_loc;
- if (! defined $from || ! length $from ||
- $from ne $form->field('from') ||
- - $from=~/$config{wiki_file_prune_regexp}/ ||
- - $from=~/^\// ||
- + is_prune($from) || $from=~/^\// ||
- $form->submitted eq "Preview") {
- @page_locs=$best_loc=$page;
- }
- diff --git a/IkiWiki/Render.pm b/IkiWiki/Render.pm
- index 48a25be..a083d3f 100644
- --- a/IkiWiki/Render.pm
- +++ b/IkiWiki/Render.pm
- @@ -202,7 +202,7 @@ sub refresh () { #{{{
- no_chdir => 1,
- wanted => sub {
- $_=decode_utf8($_);
- - if (/$config{wiki_file_prune_regexp}/) {
- + if (is_prune($_)) {
- $File::Find::prune=1;
- }
- elsif (! -d $_ && ! -l $_) {
- @@ -222,7 +222,7 @@ sub refresh () { #{{{
- no_chdir => 1,
- wanted => sub {
- $_=decode_utf8($_);
- - if (/$config{wiki_file_prune_regexp}/) {
- + if (is_prune($_, $config{underlaydir})) {
- $File::Find::prune=1;
- }
- elsif (! -d $_ && ! -l $_) { \ No newline at end of file
+ # Work around very innefficient behavior in File::Spec if abs2rel
+ # is passed two relative paths. It's much faster if paths are