|
List Info
Thread: XML::SAX::Expat bugfix
|
|
| XML::SAX::Expat bugfix |
|
2007-01-24 13:44:11 |
|
I am a developer for the BioPerl distribution and have run into an
issue with XML::SAX::Expat 0.37 and external entity resolution.
Apparently there is no direct way to turn off external entity
resolution, yet there is an apparent (2-yr old) fix filed as a bug
report:
http://rt.cpan.org/Public/Bug/Display.html?id=7434
However, it looks like XML::SAX::Expat development has languished
(the latest CPAN distro is almost 4 yrs old) and the fix, as simple
as it is, hasn't been committed. The problem we have run into is
many BioPerl users have this module as their default XML::SAX
parser. Should I press to get this fixed, or dissuade users from
using XML::SAX::Expat altogether?
chris
_______________________________________________
Perl-XML mailing list
Perl-XML listserv.ActiveState.com
To unsubscribe: http://listserv.ActiveState.com/mailman/mysubs
|
| Re: XML::SAX::Expat bugfix |
|
2007-01-24 13:49:28 |
|
| * Chris Fields wrote:
>However, it looks like XML::SAX::Expat development has languished
>(the latest CPAN distro is almost 4 yrs old) and the fix, as simple
>as it is, hasn't been committed. The problem we have run into is
>many BioPerl users have this module as their default XML::SAX
>parser. Should I press to get this fixed, or dissuade users from
>using XML::SAX::Expat altogether?
Send the patch, I'll apply it and upload the new version. Generally
though XML::SAX::ExpatXS is the better alternative.
--
Björn Höhrmann · mailto:bjoernhoehrmann.de · http://bjoern.hoehrmann.de
Weinh. Str. 22 · Telefon: +49(0)621/4309674 · http://www.bjoernsworld.de
68309 Mannheim · PGP Pub. KeyID: 0xA4357E78 · http://www.websitedev.de/
_______________________________________________
Perl-XML mailing list
Perl-XMLlistserv.ActiveState.com
To unsubscribe: http://listserv.ActiveState.com/mailman/mysubs
|
| Re: XML::SAX::Expat bugfix |
|
2007-01-24 14:13:21 |
|
|
On Jan 24, 2007, at 1:49 PM, Bjoern Hoehrmann wrote:
> * Chris Fields wrote:
>> However, it looks like XML::SAX::Expat development has languished
>> (the latest CPAN distro is almost 4 yrs old) and the fix, as simple
>> as it is, hasn't been committed. The problem we have run into is
>> many BioPerl users have this module as their default XML::SAX
>> parser. Should I press to get this fixed, or dissuade users from
>> using XML::SAX::Expat altogether?
>
> Send the patch, I'll apply it and upload the new version. Generally
> though XML::SAX::ExpatXS is the better alternative.
> --
> Björn Höhrmann · mailto:bjoernhoehrmann.de · http://
> bjoern.hoehrmann.de
> Weinh. Str. 22 · Telefon: +49(0)621/4309674 · http://
> www.bjoernsworld.de
> 68309 Mannheim · PGP Pub. KeyID: 0xA4357E78 · http://
> www.websitedev.de/
I agree (about using ExpatXS over Expat). I should also point out
there are more unclosed bug reports present on CPAN (some with fixes).
The patch for this is attached.
chris
|
 Approximate file size 538 bytes |
| Re: XML::SAX::Expat bugfix |
|
2007-01-31 10:12:28 |
On Jan 24, 2007, at 1:49 PM, Bjoern Hoehrmann wrote:
> * Chris Fields wrote:
>> However, it looks like XML::SAX::Expat development
has languished
>> (the latest CPAN distro is almost 4 yrs old) and
the fix, as simple
>> as it is, hasn't been committed. The problem we
have run into is
>> many BioPerl users have this module as their
default XML::SAX
>> parser. Should I press to get this fixed, or
dissuade users from
>> using XML::SAX::Expat altogether?
>
> Send the patch, I'll apply it and upload the new
version. Generally
> though XML::SAX::ExpatXS is the better alternative.
Just wanted to say thanks for applying the patch and making
the
release. It works great!
Cheers!
chris
_______________________________________________
Perl-XML mailing list
Perl-XMLlistserv.ActiveState.com
To unsubscribe: http:/
/listserv.ActiveState.com/mailman/mysubs
|
|
[1-4]
|
|
|
about | contact Other archives ( Real Estate discussion Medical topics )
|