Link to my amended version of Will's plugin
authorSimon McVittie <smcv@ http://smcv.pseudorandom.co.uk/>
Sun, 26 Jul 2009 16:15:01 +0000 (17:15 +0100)
committerSimon McVittie <smcv@ http://smcv.pseudorandom.co.uk/>
Sun, 26 Jul 2009 16:15:01 +0000 (17:15 +0100)
doc/todo/Raw_view_link.mdwn
doc/todo/source_link.mdwn

index 8f412fd095e29b8817ca32a31edfec5f413fc374..fd64074c2a1bda8c924a361aa8c532d7f5af93a8 100644 (file)
@@ -10,7 +10,8 @@ The configuration setting for Mercurial could be something like this:
 > Not that I'm opposed to the idea of a plugin that adds a Raw link
 > --[[Joey]]
 
->> [[todo/source_link]] does this via the CGI instead of delegating
->> to gitweb/etc. --[[smcv]]
+>> In [[todo/source_link]], Will does this via the CGI instead of delegating
+>> to gitweb/etc. I think Will's patch is a good approach, and have improved
+>> on it a bit in a git branch.
 
 [[!tag wishlist]]
index 9d9ec969713b3778b0581ca5a343471d9efee6c9..962e099a6b47fafb47b59e2918e4f8a340e4d090 100644 (file)
@@ -9,6 +9,21 @@ I just implemented this.  There is one [[patch]] to the default page template, a
 > by loading the index with IkiWiki::loadindex, like [[plugins/goto]] does?
 > --[[smcv]]
 
+[[!template id=gitbranch branch=smcv/getsource
+  author="[[Will]]/[[smcv]]"]]
+
+>> I've applied the patch below in a git branch, fixed my earlier criticism,
+>> and also fixed a couple of other issues I noticed:
+>>
+>> * missing pages could be presented better as a real 404 page
+>> * the default Content-type should probably be UTF-8 since the rest of
+>>   IkiWiki tends to assume that
+>> * emitting attachments (images, etc.) as text/plain isn't going to work :-)
+>>
+>> Any opinions on my branch? I think it's ready for merge, if Joey approves.
+>>
+>> --[[smcv]]
+
 ----
 
     diff --git a/templates/page.tmpl b/templates/page.tmpl