From 0e224e9f9cb5cca91dc9d9210917ec74510564af Mon Sep 17 00:00:00 2001 From: joey Date: Fri, 8 Sep 2006 20:33:49 +0000 Subject: [PATCH] * Add a missing -n to tla undo call. * otl2hmtl can't be safely used with IPC::Open2 since it sometimes writes before consuming its whole stdin. Avoid this deadlock. * Fix alt tags for outline checkboxes. --- IkiWiki/Plugin/otl.pm | 66 ++++++++++++++++++++++++++++++------------- debian/changelog | 9 ++++-- 2 files changed, 52 insertions(+), 23 deletions(-) diff --git a/IkiWiki/Plugin/otl.pm b/IkiWiki/Plugin/otl.pm index 0cd93a3ec..b10d5e807 100644 --- a/IkiWiki/Plugin/otl.pm +++ b/IkiWiki/Plugin/otl.pm @@ -5,7 +5,6 @@ package IkiWiki::Plugin::otl; use warnings; use strict; use IkiWiki; -use IPC::Open2; sub import { #{{{ IkiWiki::hook(type => "filter", id => "otl", call => \&filter); @@ -18,9 +17,9 @@ sub filter (@) { #{{{ # Munge up check boxes to look a little bit better. This is a hack. my $checked=IkiWiki::htmllink($params{page}, $params{page}, - "smileys/star_on.png", 0); + "smileys/star_on.png", 0, 0, "[X]"); my $unchecked=IkiWiki::htmllink($params{page}, $params{page}, - "smileys/star_off.png", 0); + "smileys/star_off.png", 0, 0, "[_]"); $params{content}=~s/^(\s*)\[X\]\s/${1}$checked /mg; $params{content}=~s/^(\s*)\[_\]\s/${1}$unchecked /mg; @@ -30,29 +29,56 @@ sub filter (@) { #{{{ sub htmlize (@) { #{{{ my %params=@_; + # Can't use open2 since otl2html doesn't play nice with buffering. + # Instead, fork off a child process that will run otl2html and feed + # it the content. Then read otl2html's response. + my $tries=10; my $pid; - while (1) { - eval { - $pid=open2(*IN, *OUT, 'otl2html -S /dev/null -T /dev/stdin'); - }; - last unless $@; - $tries--; - if ($tries < 1) { - IkiWiki::debug("failed to run otl2html: $@"); - return $params{content}; + do { + $pid = open(KID_TO_READ, "-|"); + unless (defined $pid) { + $tries--; + if ($tries < 1) { + IkiWiki::debug("failed to fork: $@"); + return $params{content}; + } } + } until defined $pid; + + if (! $pid) { + $tries=10; + $pid=undef; + + do { + $pid = open(KID_TO_WRITE, "|-"); + unless (defined $pid) { + $tries--; + if ($tries < 1) { + IkiWiki::debug("failed to fork: $@"); + print $params{content}; + exit; + } + } + } until defined $pid; + + if (! $pid) { + if (! exec 'otl2html', '-S', '/dev/null', '-T', '/dev/stdin') { + IkiWiki::debug("failed to run otl2html: $@"); + print $params{content}; + exit; + } + } + + print KID_TO_WRITE $params{content}; + close KID_TO_WRITE; + waitpid $pid, 0; + exit; } - # open2 doesn't respect "use open ':utf8'" - binmode (IN, ':utf8'); - binmode (OUT, ':utf8'); - print OUT $params{content}; - close OUT; - local $/ = undef; - my $ret=; - close IN; + my $ret=; + close KID_TO_READ; waitpid $pid, 0; $ret=~s/.*//s; diff --git a/debian/changelog b/debian/changelog index 0e3b0fd47..7be9e9bc3 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,8 +1,11 @@ -ikiwiki (1.26) UNRELEASED; urgency=low +ikiwiki (1.26) unstable; urgency=low - * Add a missing -n to tla undo call. + * Add a missing -n to tla undo call. + * otl2hmtl can't be safely used with IPC::Open2 since it sometimes + writes before consuming its whole stdin. Avoid this deadlock. + * Fix alt tags for outline checkboxes. - -- Joey Hess Fri, 8 Sep 2006 15:10:13 -0400 + -- Joey Hess Fri, 8 Sep 2006 16:24:16 -0400 ikiwiki (1.25) unstable; urgency=low -- 2.26.2