commit f2d981c00b39ac11a4e5e6d0dfec728f944fef5e
Author: Ruediger Landmann <r.landmann@xxxxxxxxxx>
Date: Wed Nov 21 04:23:00 2012 +1000
New upstream
.gitignore | 1 +
RT-74392.patch | 21 +++++++++++++++++++++
col_0_fix.patch | 21 +++++++++++++++++++++
perl-HTML-FormatText-WithLinks-AndTables.spec | 14 +++++++++-----
sources | 2 +-
5 files changed, 53 insertions(+), 6 deletions(-)
---
diff --git a/.gitignore b/.gitignore
index de9afec..6b0dd73 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1 +1,2 @@
/HTML-FormatText-WithLinks-AndTables-0.01.tar.gz
+/HTML-FormatText-WithLinks-AndTables-0.02.tar.gz
diff --git a/RT-74392.patch b/RT-74392.patch
new file mode 100644
index 0000000..e7411da
--- /dev/null
+++ b/RT-74392.patch
@@ -0,0 +1,21 @@
+diff -ur HTML-FormatText-WithLinks-AndTables/lib/HTML/FormatText/WithLinks/AndTables.pm HTML-FormatText-WithLinks-AndTab-patched/lib/HTML/FormatText/WithLinks/AndTables.pm
+--- HTML-FormatText-WithLinks-AndTables/lib/HTML/FormatText/WithLinks/AndTables.pm 2012-06-08 10:10:37.000000000 +1000
++++ HTML-FormatText-WithLinks-AndTab-patched/lib/HTML/FormatText/WithLinks/AndTables.pm 2012-07-24 16:17:19.964364582 +1000
+@@ -134,7 +134,7 @@
+ for my $tr (@trs) { # *** 1st pass over rows
+ $max_col_heights[$row_count] = 0;
+ $col_lines[$row_count] = [];
+- my @cols = $tr->look_down(_tag=>'td'); # no support for <th>. sorry.
++ my @cols = $tr->look_down(_tag=>qr/(td|th)/); # no support for <th>. sorry.
+ for (my $i = 0; $i < scalar @cols; $i++) {
+ my $td = $cols[$i]->clone;
+ my $new_tree = HTML::TreeBuilder->new;
+@@ -167,7 +167,7 @@
+ SECOND_PASS: {
+ my $row_count = 0; # obviously, another counter...
+ for my $tr (@trs) { # *** 2nd pass over rows
+- my @cols = $tr->look_down(_tag=>'td'); # no support for <th>. sorry.
++ my @cols = $tr->look_down(_tag=>qr/(td|th)/); # no support for <th>. sorry.
+
+ my $row_text; # the final string representing each row of reformatted text
+
diff --git a/col_0_fix.patch b/col_0_fix.patch
new file mode 100644
index 0000000..6ff0dda
--- /dev/null
+++ b/col_0_fix.patch
@@ -0,0 +1,21 @@
+diff -ur HTML-FormatText-WithLinks-AndTables/lib/HTML/FormatText/WithLinks/AndTables.pm HTML-FormatText-WithLinks-AndTab-patched/lib/HTML/FormatText/WithLinks/AndTables.pm
+--- HTML-FormatText-WithLinks-AndTables/lib/HTML/FormatText/WithLinks/AndTables.pm 2012-06-08 10:10:37.000000000 +1000
++++ HTML-FormatText-WithLinks-AndTab-patched/lib/HTML/FormatText/WithLinks/AndTables.pm 2012-07-24 16:25:07.631985777 +1000
+@@ -208,13 +208,15 @@
+ $new_line .= "\n" if $j != $max_col_heights[$row_count] - 1; # add a newline to all but the last text row
+ $col_rows[$j] = $new_line; # put the line into the stack for this row
+ }
+- $row_text .= $_ for @col_rows;
++ $row_text .= ($_ || '') for @col_rows; # BUGBUG this breaks on colspan in headers
+ for (my $i = 1; $i < scalar @cols; $i++) {
+ $cols[$i]->delete; # get rid of unneeded <td>'s
+ }
+ # put the fully formatted text into our accumulator
+ $formatted_tables->[$table_count]->[$row_count] = $row_text;
+- $cols[0]->content->[0] = "__TOKEN__${table_count}__${row_count}__"; # place a token into the row at col 0
++ if($cols[0]) { # BUGBUG this breaks on "complex" tables
++ $cols[0]->content->[0] = "__TOKEN__${table_count}__${row_count}__"; # place a token into the row at col 0
++ }
+ $row_count++;
+ }
+ }
diff --git a/perl-HTML-FormatText-WithLinks-AndTables.spec b/perl-HTML-FormatText-WithLinks-AndTables.spec
index 4237470..18fbb21 100644
--- a/perl-HTML-FormatText-WithLinks-AndTables.spec
+++ b/perl-HTML-FormatText-WithLinks-AndTables.spec
@@ -1,11 +1,13 @@
Name: perl-HTML-FormatText-WithLinks-AndTables
-Version: 0.01
-Release: 3%{?dist}
+Version: 0.02
+Release: 2%{?dist}
Summary: Converts HTML to Text with tables in tact
License: GPL+ or Artistic
Group: Development/Libraries
URL: http://search.cpan.org/dist/HTML-FormatText-WithLinks-AndTables/
Source0: http://www.cpan.org/authors/id/S/SF/SFRYER/HTML-FormatText-WithLinks-AndTables-%{version}.tar.gz
+Patch0: RT-74392.patch
+Patch1: col_0_fix.patch
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
BuildArch: noarch
BuildRequires: perl(ExtUtils::MakeMaker)
@@ -28,7 +30,9 @@ preserve multi-line text inside of a <TD> element provided it is broken
using <BR/> tags.
%prep
-%setup -q -n HTML-FormatText-WithLinks-AndTables-%{version}
+%setup -q -n HTML-FormatText-WithLinks-AndTables
+%patch0 -p 1
+%patch1 -p 1
%build
%{__perl} Makefile.PL INSTALLDIRS=vendor
@@ -57,8 +61,8 @@ rm -rf $RPM_BUILD_ROOT
%{_mandir}/man3/*
%changelog
-* Fri Jan 13 2012 Fedora Release Engineering <rel-eng@xxxxxxxxxxxxxxxxxxxxxxx> - 0.01-3
-- Rebuilt for https://fedoraproject.org/wiki/Fedora_17_Mass_Rebuild
+* Tue Jul 24 2012 Jeff fearn <jfearn@xxxxxxxxxx> 0.02-1
+- New Upstream
* Sun Jun 26 2011 Rüdiger Landmann <r.landmann@xxxxxxxxxx> 0.01-2
- rm duplicate Requires
diff --git a/sources b/sources
index ec71c34..dc69fa9 100644
--- a/sources
+++ b/sources
@@ -1 +1 @@
-0f6132d17056f2b2c20d03f31ea0c533 HTML-FormatText-WithLinks-AndTables-0.01.tar.gz
+804146aa9e49a5b1af4f63fecd9b0c28 HTML-FormatText-WithLinks-AndTables-0.02.tar.gz
--
Fedora Extras Perl SIG
http://www.fedoraproject.org/wiki/Extras/SIGs/Perl
perl-devel mailing list
perl-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/perl-devel
