special case for page.tmpl dependency handling
authorJoey Hess <joey@kitenet.net>
Fri, 23 Apr 2010 20:41:07 +0000 (16:41 -0400)
committerJoey Hess <joey@kitenet.net>
Fri, 23 Apr 2010 20:41:07 +0000 (16:41 -0400)
Rather than wasting resources recording that every page depends on
page.tmpl, add a special case. The special case curretly rebuilds non-page
files too when page.tmpl changes, but that's minor.

IkiWiki/Render.pm

index 7cf19645e17c07fa0b0833e161bb12a88ac3e4bc..8ae0cbd4f64fe4f8b450114dc1cdb1ba0caad400 100644 (file)
@@ -74,9 +74,16 @@ sub genpage ($$) {
                        $templatefile=$file;
                }
        });
-       my $template=template_depends(
-               defined $templatefile ? $templatefile : 'page.tmpl', $page,
-               blind_cache => 1);
+       my $template;
+       if (defined $templatefile) {
+               $template=template_depends($templatefile, $page,
+                       blind_cache => 1);
+       }
+       else {
+               # no explicit depends as special case
+               $template=template('page.tmpl', 
+                       blind_cache => 1);
+       }
        my $actions=0;
 
        if (length $config{cgiurl}) {
@@ -763,8 +770,14 @@ sub refresh () {
        foreach my $file (@$new, @$del) {
                render_linkers($file);
        }
-       
-       if (@$changed || @$internal_changed ||
+
+       if ($rendered{"templates/page.tmpl"}) {
+               foreach my $f (@$files) {
+                       next if $f eq "templates/page.tmpl";
+                       render($f, sprintf(gettext("building %s, which depends on %s"), $f, "templates/page.tmpl"));
+               }
+       }
+       elsif (@$changed || @$internal_changed ||
            @$del || @$internal_del || @$internal_new) {
                1 while render_dependent($files, $new, $internal_new,
                        $del, $internal_del, $internal_changed,