Skip to content

Commit

Permalink
Don't use IO::Scalar but in-memory open(). Using the former breaks PO…
Browse files Browse the repository at this point in the history
…D + UTF-8
  • Loading branch information
avar committed Mar 8, 2010
1 parent e2939a8 commit 3ec9008
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
5 changes: 5 additions & 0 deletions Changes
@@ -1,5 +1,10 @@
Revision history for Dist-Zilla-Plugin-ReadmeFromPod

0.05
Using IO::Scalar instead of Perl's built-in open() to open
in-memory filehandles breaks in-memory Pod::Text processing of
POD documents containing UTF-8 (who knows why) (avar)

0.04 2009.06.16
require Pod::Text 3.13

Expand Down
1 change: 0 additions & 1 deletion dist.ini
Expand Up @@ -11,7 +11,6 @@ remove = BumpVersion

[Prereq]
Dist::Zilla = 1.007
IO::Scalar = 0
Pod::Text = 3.13

[PodWeaver]
Expand Down
6 changes: 2 additions & 4 deletions lib/Dist/Zilla/Plugin/ReadmeFromPod.pm
Expand Up @@ -25,12 +25,10 @@ sub setup_installer {

require Dist::Zilla::File::InMemory;

require IO::Scalar;
my $content;
my $out_fh = new IO::Scalar \$content;
open(my $out_fh, ">", \my $content);

my $mmcontent = $self->zilla->main_module->content;

require Pod::Text;
my $parser = Pod::Text->new();
$parser->output_fh( $out_fh );
Expand Down

1 comment on commit 3ec9008

@avar
Copy link
Owner Author

@avar avar commented on 3ec9008 Mar 8, 2010

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Before:

$ rm -rf Hailo-0.20*; perl -I ~/g/dist-zilla/lib -I utils/developer $(which dzil) build ; grep Bjarm Hailo-0.20/README
    �var Arnfj�r� Bjarmason <avar@cpan.org>
    Copyright 2010 Hinrik �rn Sigur�sson and �var Arnfj�r� Bjarmason

After:

$ rm -rf Hailo-0.20*; perl -I ~/g/dist-zilla-plugin-readmefrompod/lib -I ~/g/dist-zilla/lib -I utils/developer $(which dzil) build ; grep Bjarm Hailo-0.20/README
    Ævar Arnfjörð Bjarmason <avar@cpan.org>
    Copyright 2010 Hinrik Örn Sigurðsson and Ævar Arnfjörð Bjarmason

Please sign in to comment.