tag 560562 confirmed patch upstream
thanks

On Fri, Dec 11, 2009 at 09:57:54AM +0100, Lucas Nussbaum wrote:
> Source: smokeping
> Version: 2.3.6-4
> Severity: serious
> User: debian...@lists.debian.org
> Usertags: qa-ftbfs-20091210 qa-ftbfs
> Justification: FTBFS on amd64

> > cd doc/; top=""; top=${top:-.}; pod2html 
> > --infile=/build/user-smokeping_2.3.6-4-amd64-KJCN1z/smokeping-2.3.6/doc/smokeping_extend.pod
> >  --noindex --htmlroot=. --podroot=. --podpath=${top} 
> > --title=smokeping_extend | ${top}/../util/fix-pod2html.pl > 
> > smokeping_extend.html
> > parse: No such file or directory at ./../util/fix-pod2html.pl line 25.
> > make[1]: *** [doc/smokeping_extend.html] Error 2

Thanks. This is an upstream change in HTML::Parser 3.64:
"-" no longer means STDIN in the util/fix-pod2html.pl code

 $p->parse_file(shift||"-") or die("parse: $!");

Trivial patch attached. I'll try to find the time and upload this weekend.

Tobi, could you please confirm that this works for you too? I didn't
commit it yet.
-- 
Niko Tyni   nt...@debian.org
>From 662af2f88ff409f82cfa5688dbeaf1326fd36a47 Mon Sep 17 00:00:00 2001
From: Niko Tyni <nt...@debian.org>
Date: Fri, 11 Dec 2009 13:12:46 +0200
Subject: [PATCH] Accommodate to HTML::Parser 3.64 change in the file open interface.

2009-10-25  Release 3.64
[...]
      Make parse_file() method use 3-arg open [RT#49434]

Using \*STDIN should work with both the old and the new versions.
---
 util/fix-pod2html.pl |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/util/fix-pod2html.pl b/util/fix-pod2html.pl
index fa51400..ff0176e 100755
--- a/util/fix-pod2html.pl
+++ b/util/fix-pod2html.pl
@@ -22,7 +22,7 @@ my $p = HTML::Parser->new(api_version => 3);
 $p->handler(start => \&startsub, 'tagname, text');
 $p->handler(end => \&endsub, 'tagname, text');
 $p->handler(default => sub { print shift() }, 'text');
-$p->parse_file(shift||"-") or die("parse: $!");
+$p->parse_file(shift||\*STDIN) or die("parse: $!");
 
 my @stack;
 my $a=0;
-- 
1.6.5.4

Reply via email to