aboutsummaryrefslogtreecommitdiff
path: root/doc/todo/Add_support_for_latest_Text::Markdown_as_found_on_CPAN.mdwn
blob: 6b9fa05351aafeb1174705d7f812494c317dc2fe (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
Recent versions of Text::Markdown as found on CPAN (e.g. 1.0.16) no longer contains a Text::Markdown::Markdown() routine, but instead contains a Text::Markdown::markdown() routine (notice the difference in capitalization).

It seems that the Text::Markdown module as found on CPAN is now identical to Text::MultiMarkdown - hence the subtle change.

This patch allows IkiWiki to work with either of the two:

> I already wrote such a patch a few days ago and applied it to git. Might
> be a good idea to check out current git master before spending time on
> patches in the future. Thanks for the work anyway.. --[[Joey]]

[[!tag done]]

    --- IkiWiki/Plugin/mdwn.pm.orig	2008-03-08 11:33:50.000000000 +0100
    +++ IkiWiki/Plugin/mdwn.pm	2008-03-08 13:37:21.000000000 +0100
    @@ -28,14 +28,20 @@ sub htmlize (@) {
     			$markdown_sub=\&Markdown::Markdown;
     		}
     		else {
    -			eval q{use Text::Markdown};
    +			eval q{use Text::Markdown 'Markdown'};
     			if (! $@) {
     				$markdown_sub=\&Text::Markdown::Markdown;
     			}
     			else {
    -				do "/usr/bin/markdown" ||
    -					error(sprintf(gettext("failed to load Markdown.pm perl module (%s) or /usr/bin/markdown (%s)"), $@, $!));
    -				$markdown_sub=\&Markdown::Markdown;
    +				eval q{use Text::Markdown 'markdown'};
    +				if (! $@) {
    +					$markdown_sub=\&Text::Markdown::markdown;
    +				}
    +				else {
    +					do "/usr/bin/markdown" ||
    +						error(sprintf(gettext("failed to load Markdown.pm perl module (%s) or /usr/bin/markdown (%s)"), $@, $!));
    +					$markdown_sub=\&Markdown::Markdown;
    +				}
     			}
     		}
     		require Encode;

The above patch, which is against ikiwiki-2.40, should fix [[bugs/markdown_module_location]].

-- [[HenrikBrixAndersen]]

[[!tag patch]]