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)

History Show all quoted text
  Mon Mar 02 04:48:23 2009 dolmen [...] cpan.org - Ticket created
Subject: META.yml should be encoded in UTF-8
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");
  Wed May 06 09:33:48 2009 dolmen [...] cpan.org - Correspondence added
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
  Wed May 06 09:33:49 2009 dolmen [...] cpan.org - Status changed from 'new' to 'open'
  Wed May 06 11:32:08 2009 dolmen [...] cpan.org - Correspondence added
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
  Thu May 07 04:39:20 2009 dolmen [...] cpan.org - Correspondence added
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.
  Thu May 07 20:19:11 2009 dolmen [...] cpan.org - Subject changed from 'META.yml should be encoded in UTF-8' to '[PATCH] META.yml should be encoded in UTF-8'
  Thu Jun 11 20:36:31 2009 dolmen [...] cpan.org - Dependency on ticket #25434 added
  Fri Jun 19 13:08:38 2009 dagolden [...] cpan.org - Severity Important changed to Unimportant
  Fri Jun 19 13:08:39 2009 dagolden [...] cpan.org - Correspondence added
Thanks for the patch. Applied to the M::B trunk. Marking unimportant/stalled pending the fix to YAML.pm. -- David
  Fri Jun 19 13:08:42 2009 dagolden [...] cpan.org - Status changed from 'open' to 'stalled'
  Tue Jun 23 15:22:01 2009 dagolden [...] cpan.org - Subject changed from '[PATCH] META.yml should be encoded in UTF-8' to '[needs upstream fix] META.yml should be encoded in UTF-8'
  Fri Sep 11 13:38:19 2009 dagolden [...] cpan.org - Correspondence added
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
  Fri Sep 11 13:38:19 2009 The RT System itself - Status changed from 'stalled' to 'open'
  Fri Sep 11 13:38:20 2009 dagolden [...] cpan.org - Status changed from 'open' to 'stalled'
  Tue Feb 09 07:14:11 2010 dolmen [...] cpan.org - Dependency on ticket #48885 added
  Thu May 27 08:21:25 2010 dolmen [...] cpan.org - Correspondence added
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/
  Thu May 27 08:21:26 2010 The RT System itself - Status changed from 'stalled' to 'open'
  Thu May 27 09:39:14 2010 dagolden [...] cpan.org - Correspondence added
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.
  Sun Sep 04 05:20:23 2011 dolmen [...] cpan.org - Correspondence added
YAML::Tiny has been fixed. This ticket can be closed as resolved. -- Olivier Mengué - https://metacpan.org/author/dolmen https://github.com/dolmen/
  Tue Sep 06 09:36:45 2011 dagolden [...] cpan.org - Dependency on ticket #48885 deleted
  Tue Sep 06 09:36:45 2011 dagolden [...] cpan.org - Dependency on ticket #25434 deleted
  Tue Sep 06 09:37:07 2011 dagolden [...] cpan.org - Correspondence added
Thank you for the update. Marking it closed.
  Tue Sep 06 09:37:09 2011 dagolden [...] cpan.org - Status changed from 'open' to 'resolved'