X-Git-Url: https://sipb.mit.edu/gitweb.cgi/ikiwiki.git/blobdiff_plain/a7b8e35bf90fa9e2df8be78ac636b158760ecf9c..ab7d4e6af5289ad64dc6661c1a608913f15cc17e:/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn diff --git a/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn b/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn index 9eca41faf..bc80125ad 100644 --- a/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn +++ b/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn @@ -3,19 +3,23 @@ The graphviz.pm plug-in currently attempts to read PNG data in UTF-8 mode, which utf8 "\x89" does not map to Unicode at /usr/local/lib/perl5/site_perl/5.8.8/IkiWiki/Plugin/graphviz.pm line 53, chunk 1. Wide character in subroutine entry at /usr/local/lib/perl5/site_perl/5.8.8/IkiWiki/Plugin/graphviz.pm line 68. -> Ok, will remove the binmode IN then. [[done]] --[[Joey]] +> Ok, will remove the binmode IN then. done --[[Joey]] + +>> Thanks --[[HenrikBrixAndersen]] It also generates image URLs relative to the page being rendered, which means the URLs wont work when previewing a graph from the CGI script. -> You seem to be using an old version of ikiwiki, these preview isues have -> been fixed for some time, and the code doesn't look like what you patch -> in your second hunk. --[[Joey]] +(preview bug split to [[Problems_with_graphviz.pm_plug-in_previews]]) + +>> Here is an updated patch againt ikiwiki-2.5: + +>>> [[Applied|done]], thanks. --[[Joey]] The patch below fixes these two issues. --- graphviz.pm.orig Thu Jun 7 15:45:16 2007 +++ graphviz.pm Fri Jun 8 12:03:38 2007 - @@ -41,7 +41,6 @@ sub render_graph (\%) { #{{{ + @@ -41,7 +41,6 @@ sub render_graph (\%) { $pid=open2(*IN, *OUT, "$params{prog} -Tpng"); # open2 doesn't respect "use open ':utf8'" @@ -23,7 +27,7 @@ The patch below fixes these two issues. binmode (OUT, ':utf8'); print OUT $src; - @@ -70,7 +69,12 @@ sub render_graph (\%) { #{{{ + @@ -70,7 +69,12 @@ sub render_graph (\%) { } } @@ -34,6 +38,6 @@ The patch below fixes these two issues. + else { + return "\n"; + } - } #}}} + } - sub graph (@) { #{{{ + sub graph (@) {