git.oblomov.eu Git - ikiwiki/blob - 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.
   3
   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.
   7
   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.
  10
  11 -- [[JamesWestby]]
  12
  13 Fixed, I think --[[Joey]]
  14
  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]]
  20
  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]]
  24
  25 > It works if you use Text::Markdown::Markdown, sorry, I forgot to mention that.
  26 > --JamesWestby
  27
  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...)
  31
  32  --[[Joey]]
  33
  34 > It now compiles here, thanks. --JamesWestby
  35
  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
  39
  40     todo/calendar_--_archive_browsing_via_a_calendar_frontend.mdwn
  41     *** glibc detected *** double free or corruption (!prev): 0x08bced80 ***
  42
  43  -- [[harningt]]
  44
  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]]
  47
  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]]