Skip Menu |
 

This queue is for tickets about the Module-Build CPAN distribution.

Report information
The Basics
Id: 43764
Status: resolved
Priority: 0/
Queue: Module-Build

People
Owner: Nobody in particular
Requestors: dolmen [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 0.31012
Fixed in: (no value)



Subject: META.yml should be encoded in UTF-8
Download (untitled) / with headers
text/plain 348b
My author name contains non-ASCII characters. Module::Build::YAML::DumpFile() writes META.yml using the default platform encoding, which is 'windows-1252' on my platform. It should instead force UTF-8 encoding. Module::Build::YAML::LoadFile() should also use UTF-8 for loading. Just add the following call after open(): binmode($OUT, ":utf8");
Download (untitled) / with headers
text/plain 380b
Le Mer. Mai. 06 09:33:48 2009, DOLMEN a écrit : Show quoted text
> I've uploaded a patch: > https://pause.perl.org/pub/PAUSE/authors/id/D/DO/DOLMEN/patches/Module- > Build-0.33-EWILHELM-RT43764.patch.gz
The patch is ok, but is not enough as Module::Build::YAML is not used if module YAML is avalable. And UTF-8 support in YAML is broken too: http://rt.cpan.org/Public/Bug/Display.html?id=25434
Subject: [PATCH] META.yml should be encoded in UTF-8
I've submitted a patch also for RT25434. Once both patches are applied the bug will be resolved.
Download (untitled) / with headers
text/plain 117b
Thanks for the patch. Applied to the M::B trunk. Marking unimportant/stalled pending the fix to YAML.pm. -- David
Download (untitled) / with headers
text/plain 159b
Module::Build has migrated to using YAML::Tiny. This issue now depends on the fix to this upstream bug: http://rt.cpan.org/Public/Bug/Display.html?id=48885
Download (untitled) / with headers
text/plain 186b
The same YAML::Tiny bug also affects MYMETA.yml generation (wider impact as this affect the final user, not just the distribution author). -- Olivier Mengué - http://o.mengue.free.fr/
Download (untitled) / with headers
text/plain 146b
I think I'm going to deal with this by handling reading/writing in Module::Build and just letting M::B::YAML or YAML::Tiny encode/decode strings.
Download (untitled) / with headers
text/plain 150b
YAML::Tiny has been fixed. This ticket can be closed as resolved. -- Olivier Mengué - https://metacpan.org/author/dolmen https://github.com/dolmen/
Thank you for the update. Marking it closed.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.