Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

warning is not issued #123

Open
KES777 opened this issue Mar 5, 2017 · 3 comments
Open

warning is not issued #123

KES777 opened this issue Mar 5, 2017 · 3 comments

Comments

@KES777
Copy link

KES777 commented Mar 5, 2017

When create next config:

meta_merge =>  { resources =>  {
	bugtracker =>  'url',
}}

and run ./Build.PL the warning is not issued about url is not valid. and this field silently missed at MYMETA.json.

@karenetheridge
Copy link
Member

This is probably a Module::Build problem, not CPAN::Meta. CPAN::Meta has proper validation on that field:

https://metacpan.org/source/DAGOLDEN/CPAN-Meta-2.150010/lib/CPAN/Meta/Validator.pm#L151

@KES777
Copy link
Author

KES777 commented Mar 6, 2017

Ok. That is probably due to this issue

@KES777 KES777 closed this as completed Mar 6, 2017
@Leont
Copy link
Member

Leont commented Mar 6, 2017

Module::Build uses CPAN::Meta::Converter via CPAN::Meta::Merge. As this fragment shows:

use CPAN::Meta::Converter;
use Data::Dumper;

my $c = CPAN::Meta::Converter->new({ resources =>  { bugtracker => 'url' } });
print Dumper($c->upgrade_fragment(version => 2));

The observed behavior follows out of CPAN::Meta::Converters behavior.

@Leont Leont reopened this Mar 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants