Ticket #168 (closed task: fixed)

Opened 5 months ago

Last modified 5 months ago

Update Package In Trunk: YAML::XS

Reported by: kindlund Assigned to: xkovah
Priority: normal Milestone: 1.2
Component: Installation Version: none
Severity: none Keywords: package, yaml, xs, cpan
Cc:

Description

Hi Xeno,

So apparently in the course of running the proxy logs feed, I ran across a repeatable issue with the YAML::XS library — it currently causes Segmentation Faults when parsing any URLs that contain characters ranging from \x80 to \xFF in the UTF-8 character set.

The issue has been reported, as described here:

http://rt.cpan.org/Public/Bug/Display.html?id=36734

Anyway, the reporter had included a patch that fixes this problem; I've verified that the fix works as expected, but the reporter's patch does not fix any of the unit tests that subsequently fail (because those unit tests don't take into account the patched code).

As such, I've uploaded a custom patch file to this ticket that fixes both the source code and the unit tests. I need you to update our existing Manager bundle with a patched version of the YAML::XS library. When you extract the LibYAML library to update, just apply the patch using this command in the base directory: 

patch -p0 < LibYAML.patch

Then run the usual perl Makefile.PL, make, and make test to verify everything still works properly. Once verified, update the packaged library back into trunk/cpan.

Please close this ticket when you've completed this task.

Thanks,

— Darien

Attachments

LibYAML.patch (1.7 kB) - added by kindlund on 06/18/08 22:02:38.
YAML::XS Patch to support UTF-8 charset.

Change History

06/18/08 22:02:38 changed by kindlund

  • attachment LibYAML.patch added.

YAML::XS Patch to support UTF-8 charset.

06/25/08 12:26:25 changed by xkovah

  • status changed from new to closed.
  • resolution set to fixed.

A patched version of YAML-LibYAML .27 added to the appropriate directory, and the packages file updated accordingly to point to it. The cited problem link was updated to say that .28 will have the patch built in, but it is not yet available on cpan.

Xeno

Add/Change #168 (Update Package In Trunk: YAML::XS)




Change Properties
Action