fix missing close tag issue (bz 525587)

This commit is contained in:
Tom Callaway 2009-09-28 14:33:10 +00:00
parent 97466a1328
commit 19a26c27ef
2 changed files with 23 additions and 1 deletions

12
missing_close_tag.patch Normal file
View File

@ -0,0 +1,12 @@
diff -ur HTML-Tree-3.23/lib/HTML/Element.pm HTML-Tree-3.23-fix/lib/HTML/Element.pm
--- HTML-Tree-3.23/lib/HTML/Element.pm 2006-11-13 03:13:33.000000000 +1000
+++ HTML-Tree-3.23-fix/lib/HTML/Element.pm 2009-09-23 14:22:12.000000000 +1000
@@ -2041,7 +2041,7 @@
and ref($this) # sanity
and not(
$this->{'_empty_element'}
- || $empty_element_map->{$this->{'_tag'} || ''}
+ || ($empty_element_map->{$this->{'_tag'} || ''} && !@{$this->{'_content'}})
) # things that don't get post-order callbacks
) {
shift @I;

View File

@ -1,11 +1,14 @@
Name: perl-HTML-Tree
Version: 3.23
Release: 7%{?dist}
Release: 9%{?dist}
Summary: HTML tree handling modules for Perl
Group: Development/Libraries
License: GPL+ or Artistic
URL: http://search.cpan.org/dist/HTML-Tree/
Source0: http://search.cpan.org/CPAN/authors/id/P/PE/PETEK/HTML-Tree-%{version}.tar.gz
# Upstream bug filed:
# https://rt.cpan.org/Ticket/Display.html?id=49932
Patch0: missing_close_tag.patch
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
BuildArch: noarch
BuildRequires: perl(HTML::Parser) >= 2.19
@ -25,6 +28,7 @@ a separate development track.
%prep
%setup -q -n HTML-Tree-%{version}
%patch0 -p1 -b .missing-close-tag
%{__perl} -pi -e 's|/usr/local/bin/perl|%{__perl}|' htmltree
cat << \EOF > %{name}-prov
@ -59,6 +63,12 @@ rm -rf $RPM_BUILD_ROOT
%{_mandir}/man3/HTML::*3*
%changelog
* Mon Sep 28 2009 Tom "spot" Callaway <tcallawa@redhat.com> - 1:3.23-9
- apply Jeff Fearn's fix for the missing close tag bug (bz 535587)
* Sat Jul 25 2009 Fedora Release Engineering <rel-eng@lists.fedoraproject.org> - 1:3.23-8
- Rebuilt for https://fedoraproject.org/wiki/Fedora_12_Mass_Rebuild
* Thu Feb 26 2009 Fedora Release Engineering <rel-eng@lists.fedoraproject.org> - 1:3.23-7
- Rebuilt for https://fedoraproject.org/wiki/Fedora_11_Mass_Rebuild