rename hook: instead of modifying the passed-by-name array, return a copy

[git.ikiwiki.info.git] / doc / todo / fortune:_select_options_via_environment.mdwn

diff --git a/doc/todo/fortune:_select_options_via_environment.mdwn b/doc/todo/fortune:_select_options_via_environment.mdwn
index 75b48cd9c026d453a1a081bc94551c43c88a44cb..ddacd91b526d157917bd5cc6a42271bee63d5745 100644 (file)
--- a/doc/todo/fortune:_select_options_via_environment.mdwn
+++ b/doc/todo/fortune:_select_options_via_environment.mdwn
@@ -14,9 +14,9 @@
      package IkiWiki::Plugin::fortune;
 
      use warnings;
-    @@ -12,7 +18,13 @@ sub import { #{{{
+    @@ -12,7 +18,13 @@ sub import {
 
-     sub preprocess (@) { #{{{
+     sub preprocess (@) {
             $ENV{PATH}="$ENV{PATH}:/usr/games:/usr/local/games";
     -       my $f = `fortune 2>/dev/null`;
     +       my $f;
@@ -29,3 +29,6 @@
 
         if ($?) {
                 return "[[".gettext("fortune failed")."]]";
+       
+> An environment variable is not the right approach. Ikiwiki has a setup
+> file, and plugins can use configuration from there. --[[Joey]]