web commit by http://per.bothner.myopenid.com/: translate foo/index.html to foo using...
[ikiwiki.git] / doc / patchqueue / index.html_allowed.mdwn
1 Instead of having files foo.html "in front of" foo/, I prefer to have 
2 foo/index.html.
3
4 I independently implemented a similar, but smaller patch.
5 (It's smaller because I only care about rendering; not CGI, for example.)
6 The key to this patch is that "A/B/C" is treated as equivalent
7 to "A/B/C/index".
8 Here it is:  --Per Bothner
9
10     --- IkiWiki/Render.pm~  2007-01-11 15:01:51.000000000 -0800
11     +++ IkiWiki/Render.pm   2007-02-02 22:24:12.000000000 -0800
12     @@ -60,9 +60,9 @@
13             foreach my $dir (reverse split("/", $page)) {
14                     if (! $skip) {
15                             $path.="../";
16     -                       unshift @ret, { url => $path.htmlpage($dir), page => pagetitle($dir) };
17     +                       unshift @ret, { url => abs2rel(htmlpage(bestlink($page, $dir)), dirname($page)), page => pagetitle($dir) };
18                     }
19     -               else {
20     +               elsif ($dir ne "index") {
21                             $skip=0;
22                     }
23             }
24
25     --- IkiWiki.pm~ 2007-01-12 12:47:09.000000000 -0800
26     +++ IkiWiki.pm  2007-02-02 18:02:16.000000000 -0800
27     @@ -315,6 +315,12 @@
28                     elsif (exists $pagecase{lc $l}) {
29                             return $pagecase{lc $l};
30                      }
31     +               else {
32     +                   my $lindex = $l . "/index";
33     +                   if (exists $links{$lindex}) {
34     +                       return $lindex;
35     +               }
36     +               }
37              } while $cwd=~s!/?[^/]+$!!;
38      
39             if (length $config{userdir} && exists $links{"$config{userdir}/".lc($link)}) {
40
41 Note I handle setting the url; slightly differently.
42 Also note that an initial "index" is ignored.  I.e. a
43 page "A/B/index.html" is treated as "A/B".
44
45 > Actually, your patch is shorter because it's more elegant and better :)
46 > I'm withdrawing my old patch, because yours is much more in line with
47 > ikiwiki's design and architecture.
48 > I would like to make one suggestion to your patch, which is:
49
50     diff -urX ignorepats clean-ikidev/IkiWiki/Plugin/inline.pm ikidev/IkiWiki/Plugin/inline.pm
51     --- clean-ikidev/IkiWiki/Plugin/inline.pm   2007-02-25 12:26:54.099113000 -0800
52     +++ ikidev/IkiWiki/Plugin/inline.pm 2007-02-25 14:55:21.163340000 -0800
53     @@ -154,7 +154,7 @@
54                         $link=htmlpage($link) if defined $type;
55                         $link=abs2rel($link, dirname($params{destpage}));
56                         $template->param(pageurl => $link);
57     -                   $template->param(title => pagetitle(basename($page)));
58     +                   $template->param(title => titlename($page));
59                         $template->param(ctime => displaytime($pagectime{$page}));
60
61                         if ($actions) {
62     @@ -318,7 +318,7 @@
63                 my $pcontent = absolute_urls(get_inline_content($p, $page), $url);
64
65                 $itemtemplate->param(
66     -                   title => pagetitle(basename($p), 1),
67     +                   title => titlename($p, 1),
68                         url => $u,
69                         permalink => $u,
70                         date_822 => date_822($pagectime{$p}),
71     diff -urX ignorepats clean-ikidev/IkiWiki/Render.pm ikidev/IkiWiki/Render.pm
72     --- clean-ikidev/IkiWiki/Render.pm  2007-02-25 12:26:54.745833000 -0800
73     +++ ikidev/IkiWiki/Render.pm        2007-02-25 14:54:01.564715000 -0800
74     @@ -110,7 +110,7 @@
75         $template->param(
76                 title => $page eq 'index'
77                         ? $config{wikiname}
78     -                   : pagetitle(basename($page)),
79     +                   : titlename($page),
80                 wikiname => $config{wikiname},
81                 parentlinks => [parentlinks($page)],
82                 content => $content,
83     diff -urX ignorepats clean-ikidev/IkiWiki.pm ikidev/IkiWiki.pm
84     --- clean-ikidev/IkiWiki.pm 2007-02-25 12:26:58.812850000 -0800
85     +++ ikidev/IkiWiki.pm       2007-02-25 15:05:22.328852000 -0800
86     @@ -192,6 +192,12 @@
87         return $untainted;
88      } #}}}
89
90     +sub titlename($;@) { #{{{
91     +   my $page = shift;
92     +   $page =~ s!/index$!!;
93     +   return pagetitle(basename($page), @_);
94     +} #}}}
95     +
96      sub basename ($) { #{{{
97         my $file=shift;
98
99
100 > This way foo/index gets "foo" as its title, not "index". --Ethan
101
102 ---
103
104 How about doing the index stuff only on the output side? (Or does the latter patch do it? I haven't tried them.) That is, render every `foo.type` for the rendered types (mdwn etc.) as `foo/index.html`, generating links to `foo/` instead of `foo.html`, but not earlier than the point where the .html as presently appended to the page name. Then you just flip a build time option on an existing wiki without any changes to that, and the pages appear elsewhere. The `index.type` files might be left out of this scheme, though (and the top-level one, of course, has to). --[[tuomov]]
105
106 > Well, get around to wasting time on it after all, and [here's the patch](http://iki.fi/tuomov/use_dirs.diff). The `-use_dirs` option will cause everything to be rendered inside directories. There may still be some problems with it, that need looking into (it doesn't e.g. check for conflicts between foo/index.mdwn and foo.mdwn), but seems to work well enough for me... The patch also improves, I think, the parentlinks code a little, as it uses generic routines to actually find the target location now. The only places where the `use_dirs` option is used is `htmlpage`, in fact, although other specific kludges needed to be removed from other points in the code.
107
108 >> FWIW, [use_dirs.diff](http://iki.fi/tuomov/use_dirs.diff) applies cleanly, and works well for me. Given that it makes this behaviour optional, how about merging it? I have some follow-up patches which I'm sitting on for now. ;-) -- Ben
109
110 >>> How do you apply a patch created by svn diff? I've been curious about this for a long time. The use_dirs patch looks OK but I'd like to play with it. --Ethan
111
112 >>>> Just do `svn co svn://ikiwiki.kitenet.net/ikiwiki/trunk ikiwiki` then `cd ikiwiki && patch -p0 <use_dirs.diff`. :-) Same would work with a tarball as well.   
113
114 >>>>> Sorry, I'm dumb. I'm so used to doing -p1 that doing -p0 never occurred to me; I thought the patch format generated by svn diff was just "wrong". --Ethan
115
116 ----
117
118 First pass over Tumov's patch -- which doesn't cleanly apply anymore, so
119 I'll attach an updated and modified version below. --[[Joey]]
120
121 * As we discussed in email, this will break handling of `foo/index.mdwn`
122   pages. Needs to be changed to generate `foo/index/index.html` for such
123   pages (though not for the toplevel `index`).
124
125   >> Can someone elaborate on this? What's broken about it? Will pages
126   >> foo/index/index.html include foo/index in their parentlinks? --Ethan
127
128   >>> Presently the patch does not move `foo/index.type` as `foo/index/index.html`, but renders
129   >>> it as `foo/index.html`, not because I particularly want that (except for the top-level one, of
130   >>> course), but because it could be done :). This, however, conflicts with a `foo.mdwn`
131   >>> rendered as `foo/index.html`. The easiest and cleanest way to fix this, is to simply
132   >>> not handle `index` in such a special manner -- except for the top-level one. --[[tuomov]]
133
134   >>>> Oh, I see, this patch doesn't address wanting to use foo/index.mdwn as 
135   >>>> an input page. Hmm. --Ethan
136
137   >>>>> No, it doesn't. I originally also was after that, but after discussing the
138   >>>>> complexities of supporting that with Joey, came up with this simpler scheme
139   >>>>> without many of those issues. It is the output that I primarily care about, anyway,
140   >>>>> and I do, in fact, find the present input file organisation quite nice. The output
141   >>>>> locations just aren't very good for conversion of an existing site to ikiwiki, and do
142   >>>>> make for rather ugly URLs with the .html extensions. (I do often type some URLs
143   >>>>> out of memory, when they're gone from the browser's completion history, and the
144   >>>>> .html makes that more laboursome.)
145
146   >>>>>> I support your decision, but now this wiki page serves two different patches :).
147   >>>>>> Can we split them somehow?
148   >>>>>> What are the complexities involved?
149   >>>>>> I think I overcomplicated it a little with my patch, and Per Bothner's gets 
150   >>>>>> much closer to the heart of it. --Ethan
151
152 * This does make the resulting wikis much less browsable directly on the
153   filesystem, since `dir` to `dir/index.html` conversion is only handled by web
154   servers and so you end up browsing to a directory index all the time.
155   Wouldn't it be better to make the links themselves include the index.html?
156   (Although that would mean that [[bugs/broken_parentlinks]] would not be
157   fixed en passant by this patch..)
158
159      > Yes, the sites are not that browsable on the FS (I blame the browsers
160      > for being stupid!), but linking to the directory produces so much
161      > cleaner URLs for the Web, that I specifically want it. This is,
162      > after all, an optional arrangement. 
163
164      >> It's optional for *now* ... I suppose that I could make adding the
165      >> index.html yet another option. I'm not _that_ fond of optioons
166      >> however. --[[Joey]]
167
168      >>> It is worth noting, that with this patch, you _can_ render the local
169      >>> copy in the present manner, while rendering the Web copy under
170      >>> directories. So no extra options are really needed for local browsing, 
171      >>> unless you also want to serve the same copy over the Web, which I
172      >>> doubt. --[[tuomov]]
173
174 * I suggest keeping the links using foo/index.html in the html file, but use
175   JavaScript to fix the links onload time - but only if the protocol is http or https.
176   This provides nice links without the "index.html" when served by a
177   web server, but degrades nicely when using a file: url, or when JavaScript
178   is disabled.  --Per Bothner
179
180          function fixLinks ( ) {
181            var scheme = location.protocol;
182            if (scheme!="http:" && scheme!="https:") return;
183            var links = document.getElementsByTagName("a");
184            for (var i = links.length; --i >= 0; )
185              links[i].href = links[i].href.replace(/[/]index.html/,"");
186          }
187
188 * Some of the generated links are missing the trailing / , which is
189   innefficient since it leads to a http redirect when clicking on that
190   link. Seems to be limited to ".." links, and possibly only to
191   parentlinks. (Already fixed it for "." links.)
192
193       > The solution seems to be to add to `urlto` the following snippet,
194       > which might also help with the next point. (Sorry, no updated patch
195       > yet. Should be on my way out in the cold anyway...)
196
197         if ( !length $to ) {
198                 return baseurl($from);
199         }
200  
201       >> Indeed, this brings the number of abs2rels closer to par, as well
202       >> as fixing the .. links. --[[Joey]]
203
204 * It calles abs2rel about 16% more often with the patch, which makes it
205   a bit slower, since abs2rel is not very efficient. (This omits abs2rel
206   calls that might be memoized away already.) This seems to be due to one
207   extra abs2rel for the toplevel wiki page due to the nicely cleaned up code
208   in `parentlinks` -- so I'm not really complaining.. Especially since the
209   patch adds a new nice memoizable `urlto`.
210 * The rss page name generation code seems unnecesarily roundabout, I'm sure
211   that can be cleaned up somehow, perhaps by making `htmlpage` more
212   generic.
213
214      > Something like `targetpage(basename, extension)`?
215
216      >> Yes exactly. It might also be possible to remove htmlpage from the
217      >> plugin interface entirely (in favour of urlto), which would be a
218      >> good time to make such a changes. Not required to accept this patch
219      >> though.
220
221      >>> [...] in fact, all uses of htmlpage in the plugins are used to
222      >>> construct an absolute address: the absolute url in most cases, so an `absurl`
223      >>> call could be added to be used instead of htmlpage
224      >>> --[[tuomov]]
225
226      >>>> Or it could use urlto("index", $page) instead. --[[Joey]]
227
228      >>>>> That is, however, a relative URL, and maybe an absolute one
229      >>>>> is wanted. Perhaps `urlto($targetpage)` should return the
230      >>>>> absolute version --[[tuomov]]
231
232 * > and something else in the
233   > aggregate plugin (above), that I also think isn't what's wanted:
234   > aren't `foo.html` pages also "rendered", so that they get moved as `foo/index.html`?
235   > --[[tuomov]]
236
237   >> Yes, the aggregate plugin will save the files as foo.html in the
238   >> sourcedir, and that will result in foo/index.html in the web site, same
239   >> as any other page. --[[Joey]]
240
241 * `img.pm` makes some assumptions about name of the page that will be
242   linking to the image, which are probably broken.
243
244 * The changes to htmlpage's behavior probably call for the plugin
245   interface version number to be changed.
246
247 Latest version of my patch... with most of the stuff that's been discussed, including `targetpage`.
248 Also available [here](http://iki.fi/tuomov/use_dirs-20070221.diff). (BTW, this posting, applying, and
249 updating of plain-old-diffs containing all the previous changes is starting to be painful. Reminds
250 me why I use darcs..) --[[tuomov]]
251
252 <pre>
253 Index: IkiWiki.pm
254 ===================================================================
255 --- IkiWiki.pm  (revision 2806)
256 +++ IkiWiki.pm  (working copy)
257 @@ -14,7 +14,7 @@
258  use Exporter q{import};
259  our @EXPORT = qw(hook debug error template htmlpage add_depends pagespec_match
260                   bestlink htmllink readfile writefile pagetype srcfile pagename
261 -                 displaytime will_render gettext
262 +                 displaytime will_render gettext urlto targetpage
263                   %config %links %renderedfiles %pagesources);
264  our $VERSION = 1.02; # plugin interface version, next is ikiwiki version
265  our $version='unknown'; # VERSION_AUTOREPLACE done by Makefile, DNE
266 @@ -73,6 +73,7 @@
267         sslcookie => 0,
268         httpauth => 0,
269         userdir => "",
270 +       usedirs => 0
271  } #}}}
272     
273  sub checkconfig () { #{{{
274 @@ -224,10 +225,21 @@
275         return $page;
276  } #}}}
277  
278 +sub targetpage ($$) { #{{{
279 +       my $page=shift;
280 +       my $ext=shift;
281 +       
282 +       if (! $config{usedirs} || $page =~ /^index$/ ) {
283 +               return $page.".".$ext;
284 +       } else {
285 +               return $page."/index.".$ext;
286 +       }
287 +} #}}}
288 +
289  sub htmlpage ($) { #{{{
290         my $page=shift;
291 -
292 -       return $page.".html";
293 +       
294 +       return targetpage($page, "html");
295  } #}}}
296  
297  sub srcfile ($) { #{{{
298 @@ -393,6 +405,7 @@
299  
300         return "$config{url}/" if ! defined $page;
301         
302 +       $page=htmlpage($page);
303         $page=~s/[^\/]+$//;
304         $page=~s/[^\/]+\//..\//g;
305         return $page;
306 @@ -422,6 +435,32 @@
307                         $config{timeformat}, localtime($time)));
308  } #}}}
309  
310 +sub beautify_url ($) { #{{{
311 +       my $url=shift;
312 +
313 +       $url =~ s!/index.html$!/!;
314 +       $url =~ s!^$!./!; # Browsers don't like empty links...
315 +
316 +       return $url;
317 +} #}}}
318 +
319 +sub urlto ($$) { #{{{
320 +       my $to=shift;
321 +       my $from=shift;
322 +
323 +       if (! length $to) {
324 +               return beautify_url(baseurl($from));
325 +       }
326 +
327 +       if (! grep { $_ eq $to } map { @{$_} } values %renderedfiles) {
328 +               $to=htmlpage($to);
329 +       }
330 +
331 +       my $link = abs2rel($to, dirname(htmlpage($from)));
332 +
333 +       return beautify_url($link);
334 +} #}}}
335 +
336  sub htmllink ($$$;@) { #{{{
337         my $lpage=shift; # the page doing the linking
338         my $page=shift; # the page that will contain the link (different for inline)
339 @@ -457,7 +496,8 @@
340                         "\">?</a>$linktext</span>"
341         }
342         
343 -       $bestlink=abs2rel($bestlink, dirname($page));
344 +       $bestlink=abs2rel($bestlink, dirname(htmlpage($page)));
345 +       $bestlink=beautify_url($bestlink);
346         
347         if (! $opts{noimageinline} && isinlinableimage($bestlink)) {
348                 return "<img src=\"$bestlink\" alt=\"$linktext\" />";
349 Index: IkiWiki/Render.pm
350 ===================================================================
351 --- IkiWiki/Render.pm   (revision 2806)
352 +++ IkiWiki/Render.pm   (working copy)
353 @@ -32,8 +32,8 @@
354         my @links;
355         return unless $backlinks{$page};
356         foreach my $p (keys %{$backlinks{$page}}) {
357 -               my $href=abs2rel(htmlpage($p), dirname($page));
358 -                       
359 +               my $href=urlto($p, $page);
360 +                
361                 # Trim common dir prefixes from both pages.
362                 my $p_trimmed=$p;
363                 my $page_trimmed=$page;
364 @@ -55,18 +55,14 @@
365         my @ret;
366         my $pagelink="";
367         my $path="";
368 -       my $skip=1;
369 +       my $title=$config{wikiname};
370 +       
371         return if $page eq 'index'; # toplevel
372 -       foreach my $dir (reverse split("/", $page)) {
373 -               if (! $skip) {
374 -                       $path.="../";
375 -                       unshift @ret, { url => $path.htmlpage($dir), page => pagetitle($dir) };
376 -               }
377 -               else {
378 -                       $skip=0;
379 -               }
380 +       foreach my $dir (split("/", $page)) {
381 +               push @ret, { url => urlto($path, $page), page => $title };
382 +               $path.="/".$dir;
383 +               $title=pagetitle($dir);
384         }
385 -       unshift @ret, { url => length $path ? $path : ".", page => $config{wikiname} };
386         return @ret;
387  } #}}}
388  
389 Index: IkiWiki/Plugin/inline.pm
390 ===================================================================
391 --- IkiWiki/Plugin/inline.pm    (revision 2806)
392 +++ IkiWiki/Plugin/inline.pm    (working copy)
393 @@ -110,8 +110,8 @@
394  
395         add_depends($params{page}, $params{pages});
396  
397 -       my $rssurl=rsspage(basename($params{page}));
398 -       my $atomurl=atompage(basename($params{page}));
399 +       my $rssurl=basename(rsspage($params{page}));
400 +       my $atomurl=basename(atompage($params{page}));
401         my $ret="";
402  
403         if (exists $params{rootpage} && $config{cgiurl}) {
404 @@ -150,10 +150,7 @@
405                         # Don't use htmllink because this way the
406                         # title is separate and can be overridden by
407                         # other plugins.
408 -                       my $link=bestlink($params{page}, $page);
409 -                       $link=htmlpage($link) if defined $type;
410 -                       $link=abs2rel($link, dirname($params{destpage}));
411 -                       $template->param(pageurl => $link);
412 +                       $template->param(pageurl => urlto(bestlink($params{page}, $page), $params{destpage}));
413                         $template->param(title => pagetitle(basename($page)));
414                         $template->param(ctime => displaytime($pagectime{$page}));
415  
416 @@ -205,15 +202,17 @@
417                 }
418         
419                 if ($rss) {
420 -                       will_render($params{page}, rsspage($params{page}));
421 -                       writefile(rsspage($params{page}), $config{destdir},
422 +                       my $rssp=rsspage($params{page});
423 +                       will_render($params{page}, $rssp);
424 +                       writefile($rssp, $config{destdir},
425                                 genfeed("rss", $rssurl, $desc, $params{page}, @list));
426                         $toping{$params{page}}=1 unless $config{rebuild};
427                         $feedlinks{$params{destpage}}=qq{<link rel="alternate" type="application/rss+xml" title="RSS" href="$rssurl" />};
428                 }
429                 if ($atom) {
430 -                       will_render($params{page}, atompage($params{page}));
431 -                       writefile(atompage($params{page}), $config{destdir},
432 +                       my $atomp=atompage($params{page});
433 +                       will_render($params{page}, $atomp);
434 +                       writefile($atomp, $config{destdir},
435                                 genfeed("atom", $atomurl, $desc, $params{page}, @list));
436                         $toping{$params{page}}=1 unless $config{rebuild};
437                         $feedlinks{$params{destpage}}=qq{<link rel="alternate" type="application/atom+xml" title="Atom" href="$atomurl" />};
438 @@ -288,16 +287,21 @@
439         return $content;
440  } #}}}
441  
442 -sub rsspage ($) { #{{{
443 +sub basepage ($) { #{{{
444         my $page=shift;
445 +       
446 +       $page=htmlpage($page);
447 +       $page =~ s/\.html$//;
448 +       
449 +       return $page;
450 +} #}}}
451  
452 -       return $page.".rss";
453 +sub rsspage ($) { #{{{
454 +       return targetpage(shift, "rss");
455  } #}}}
456  
457  sub atompage ($) { #{{{
458 -       my $page=shift;
459 -
460 -       return $page.".atom";
461 +       return targetpage(shift, "atom");
462  } #}}}
463  
464  sub genfeed ($$$$@) { #{{{
465 Index: IkiWiki/Plugin/aggregate.pm
466 ===================================================================
467 --- IkiWiki/Plugin/aggregate.pm (revision 2806)
468 +++ IkiWiki/Plugin/aggregate.pm (working copy)
469 @@ -320,7 +320,7 @@
470                 # NB: This doesn't check for path length limits.
471                 eval q{use POSIX};
472                 my $max=POSIX::pathconf($config{srcdir}, &POSIX::_PC_NAME_MAX);
473 -               if (defined $max && length(htmlpage($page)) >= $max) {
474 +               if (defined $max && length(htmlfn($page)) >= $max) {
475                         $c="";
476                         $page=$feed->{dir}."/item";
477                         while (exists $IkiWiki::pagecase{lc $page.$c} ||
478 @@ -356,7 +356,7 @@
479         if (ref $feed->{tags}) {
480                 $template->param(tags => [map { tag => $_ }, @{$feed->{tags}}]);
481         }
482 -       writefile(htmlpage($guid->{page}), $config{srcdir},
483 +       writefile(htmlfn($guid->{page}), $config{srcdir},
484                 $template->output);
485  
486         # Set the mtime, this lets the build process get the right creation
487 @@ -434,4 +434,8 @@
488         return "$config{srcdir}/".htmlpage($page);
489  } #}}}
490  
491 +sub htmlfn ($) { #{{{
492 +       return shift().".html";
493 +} #}}}
494 +
495  1
496 Index: IkiWiki/Plugin/linkmap.pm
497 ===================================================================
498 --- IkiWiki/Plugin/linkmap.pm   (revision 2806)
499 +++ IkiWiki/Plugin/linkmap.pm   (working copy)
500 @@ -49,9 +49,7 @@
501         my %mapitems = ();
502         foreach my $item (keys %links) {
503                 if (pagespec_match($item, $params{pages}, $params{page})) {
504 -                       my $link=htmlpage($item);
505 -                       $link=IkiWiki::abs2rel($link, IkiWiki::dirname($params{page}));
506 -                       $mapitems{$item}=$link;
507 +                       $mapitems{$item}=urlto($item, $params{destpage});
508                 }
509         }
510  
511 Index: doc/usage.mdwn
512 ===================================================================
513 --- doc/usage.mdwn      (revision 2806)
514 +++ doc/usage.mdwn      (working copy)
515 @@ -244,6 +244,10 @@
516  
517    Log to syslog(3).
518  
519 +* --usedirs
520 +
521 +  Create output files named page/index.html instead of page.html.
522 +
523  * --w3mmode, --no-w3mmode
524  
525    Enable [[w3mmode]], which allows w3m to use ikiwiki as a local CGI script,
526 Index: doc/plugins/write.mdwn
527 ===================================================================
528 --- doc/plugins/write.mdwn      (revision 2806)
529 +++ doc/plugins/write.mdwn      (working copy)
530 @@ -412,6 +412,10 @@
531  
532  This is the standard gettext function, although slightly optimised.
533  
534 +#### `urlto($$)`
535 +
536 +Construct a relative url to the first parameter from the second.
537 +
538  ## RCS plugins
539  
540  ikiwiki's support for revision control systems also uses pluggable perl
541 Index: doc/ikiwiki.setup
542 ===================================================================
543 --- doc/ikiwiki.setup   (revision 2806)
544 +++ doc/ikiwiki.setup   (working copy)
545 @@ -94,6 +94,8 @@
546         syslog => 0,
547         # To link to user pages in a subdirectory of the wiki.
548         #userdir => "users",
549 +       # To enable alternate output filenames.
550 +       #usedirs => 1,
551  
552         # To add plugins, list them here.
553         #add_plugins => [qw{goodstuff openid search wikitext camelcase
554 </pre>