Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

remove YAML_support feature

  • Loading branch information...
commit edf5915e98a3e351bd52f65c25516f356810dea7 1 parent 081f061
@dagolden dagolden authored
View
4 Build.PL
@@ -130,10 +130,6 @@ my $build = ModuleBuildBuilder->new(
'ExtUtils::Installed' => 1.999, # technically 1.999_001 is what's available
},
},
- YAML_support => {
- description => "Use YAML::Tiny to write META.yml files",
- requires => {'YAML::Tiny' => 1.40},
- },
manpage_support => {
description => "Create Unix man pages",
requires => {'Pod::Man' => 0 },
View
2  Changes
@@ -18,6 +18,8 @@ Revision history for Perl extension Module::Build.
- Module::Build::YAML has been deprecated. Module::Build now depends
directly upon YAML::Tiny. M::B::YAML remains as a subclass wrapper.
+ The YAML_support feature has been removed, as YAML is now an ordinary
+ dependency.
0.36_19 - Tue Dec 7 13:43:42 EST 2010
View
7 lib/Module/Build/Base.pm
@@ -4469,9 +4469,9 @@ sub do_create_metafile {
sub read_metafile {
my $self = shift;
my ($metafile) = @_;
- my $yaml;
- $self->_mb_feature('YAML_support') or return;
+ $self->check_installed_status("YAML::Tiny", 1.4) or return;
+ require YAML::Tiny;
my $string = $self->_slurp($metafile, $] < 5.8 ? "" : ":utf8");
my $meta = YAML::Tiny->read_string($string)
@@ -4485,7 +4485,8 @@ sub write_metafile {
my $self = shift;
my ($metafile, $node) = @_;
- return unless $self->_mb_feature('YAML_support');
+ $self->check_installed_status("YAML::Tiny", 1.4) or return;
+ require YAML::Tiny;
my $yaml = YAML::Tiny->new($node);
my $string = $yaml->write_string;
View
4 t/metadata2.t
@@ -13,8 +13,8 @@ use DistGen;
############################## ACTION distmeta works without a MANIFEST file
SKIP: {
- skip( 'YAML_support feature is not enabled', 4 )
- unless Module::Build::ConfigData->feature('YAML_support');
+ skip( 'YAML::Tiny is not installed', 4 )
+ unless eval "require YAML::Tiny; 1";
my $dist = DistGen->new( no_manifest => 1 )->chdir_in->regen;
View
4 t/runthrough.t
@@ -6,7 +6,7 @@ use MBTest tests => 29;
blib_load('Module::Build');
blib_load('Module::Build::ConfigData');
-my $have_yaml = Module::Build::ConfigData->feature('YAML_support');
+my $have_yaml = eval "require YAML::Tiny; 1";
#########################
@@ -107,7 +107,7 @@ ok grep {$_ eq 'save_out' } $mb->cleanup;
}
SKIP: {
- skip( 'YAML_support feature is not enabled', 7 ) unless $have_yaml;
+ skip( 'YAML::Tiny is not available', 7 ) unless $have_yaml;
my $output = eval {
stdout_stderr_of( sub { $mb->dispatch('disttest') } )
View
5 t/script_dist.t
@@ -73,9 +73,8 @@ stdout_stderr_of( sub { $result = $mb->dispatch('distmeta') } );
ok $result;
SKIP: {
- skip( 'YAML_support feature is not enabled', 1 )
- unless Module::Build::ConfigData->feature('YAML_support');
- require YAML::Tiny;
+ skip( 'YAML::Tiny is not installed', 1 )
+ unless eval "require YAML::Tiny; 1";
my $yml = YAML::Tiny::LoadFile('META.yml');
is_deeply($yml->{provides}, \%meta_provides);
}
Please sign in to comment.
Something went wrong with that request. Please try again.