summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--IkiWiki/Plugin/otl.pm66
-rw-r--r--debian/changelog9
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=<IN>;
- close IN;
+ my $ret=<KID_TO_READ>;
+ close KID_TO_READ;
waitpid $pid, 0;
$ret=~s/.*<body>//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 <joeyh@debian.org> Fri, 8 Sep 2006 15:10:13 -0400
+ -- Joey Hess <joeyh@debian.org> Fri, 8 Sep 2006 16:24:16 -0400
ikiwiki (1.25) unstable; urgency=low