]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/bugs/markdown_module_location.mdwn
an order-of-magnitude optimization which also improves correctness
[git.ikiwiki.info.git] / doc / bugs / markdown_module_location.mdwn
1 If the Markdown module is installed via CPAN rather than apt then
2 the module is actually Text::Markdown.
4 I had to edit the source to change this on my old server. I have filed
5 a [bug](http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=387687) against
6 the Debian markdown, which I guess you can consider a blocking bug of this.
8 I tried to come up with the magical invocation to allow either location 
9 to be used by ikiwiki, but I couldn't do it.
11 -- [[JamesWestby]]
13 Fixed, I think --[[Joey]]
15 > Fraid not. The import works ok, but I get
16      `Undefined subroutine &Markdown::Markdown called at IkiWiki/Plugin/mdwn.pm line 41.`
17 > This is what stumped me, I was trying to import as an alias, but couldn't work
18 > out how to do it. A flag if you use the second import would be an ugly solution.
19 > -- [[JamesWestby]]
21 Ok, the markdown in CPAN must be an entirely different version then if it
22 doesn't has a Markdown::Markdown. Interesting, I'll have a look at it.
23 --[[Joey]]
25 > It works if you use Text::Markdown::Markdown, sorry, I forgot to mention that.
26 > --JamesWestby
28 I think what I've committed now will work in all cases. Well, unless there
29 are even more forks of markdown out there (the CPAN module is a fork
30 apparently...) 
32  --[[Joey]]
34 > It now compiles here, thanks. --JamesWestby
36 > It's back open in the latest incarnation of Text::Markdown ... the fix is to use the
37 > lowercase function name (Text::Markdown::markdown) however w/ this setup 
38 > it causes a segfault on my system.... down while compiling
40     todo/calendar_--_archive_browsing_via_a_calendar_frontend.mdwn
41     *** glibc detected *** double free or corruption (!prev): 0x08bced80 ***
43  -- [[harningt]]
45 > What version of Text::Markdown are you referring to? If it crashes perl
46 > then perhaps you need to find a less evil version... --[[Joey]]
48 >> The patch in [[todo/Add_support_for_latest_Text::Markdown_as_found_on_CPAN]] adds support for Text::Markdown::markdown(). -- [[HenrikBrixAndersen]]
49 >> Doesn't fix the above double free though. Nevertheless, I think I'm going to call this [[done]] since I already added support for Text::Markdown::markdown in git earlier this week. --[[Joey]]