Re: [Perl-workflow-devel] RT #5943
Brought to you by:
jonasbn
|
From: Jonas B. N. <jo...@gm...> - 2008-09-04 07:43:17
|
Hi Grant,
After much thinking and examination I have come to the conclusion that
XML::SAX::ParserFactory has an issue.
The following:
if (@{$self->{KnownParsers}}) {
return $self->{KnownParsers}[-1]{Name};
}
else {
return "XML::SAX::PurePerl"; # backup plan!
}
Yields unpredictable behavior based on installation order of parsers.
I would suggest addressing this and documenting how to control parser
load.
I would rewrite it to:
if (@{$self->{KnownParsers}}) {
return $self->{KnownParsers}[-1]{Name};
}
else {
return "XML::SAX::PurePerl"; # backup plan!
}
But this breaks existing installtions and require people to revisit
their: XML/SAX/ParserDetails.ini files.
Perhaps the best solution would be to look into the feature system,
but this will require multiple patches to a lot of distributions in
order to get properly propagated - and I currently do not have the
knowledge on XML::SAX/XML::Simple to conclude that it can be
controlled that way.
Ref: http://use.perl.org/~jonasbn/journal/37344
jonasbn
Ps. I have Cc'ed the Workflow development list to keep my fellow
developers on workflow aware of my progress, you probably cannot post
to the list, but I will see to that a possible response from you gets
relayed.
On 20/08/2008, at 11.59, Grant McLean wrote:
> Hi Jonas
>
> The only thing that's holding this up is lack of time. I will get
> to it
> but I have more important problems to deal with first. If you want to
> get co-maintainership of XML::SAX::RTF and do a release then go right
> ahead.
>
> Cheers
> Grant
>
> On Wed, 2008-08-20 at 09:07 +0200, Jonas Brømsø Nielsen wrote:
>> Hello Grant,
>>
>> I have been chasing this bug:
>>
>> http://www.perlmonks.org/?node_id=512040
>>
>> And I ended up at this ticket:
>>
>> http://rt.cpan.org/Ticket/Display.html?id=5943
>>
>> How are things doing with this, do you need assistance, help, test,
>> patches??
>>
>> Looking forward to hearing from you,
>>
>> jonasbn
>>
>>
>> --
>> pauseid: JONASBN
>> email: jo...@cp...
>> blog: http://use.perl.org/~jonasbn/journal/
>>
>>
>
|