Perl-Email-Project / Email-Valid

perl library to validate email addresses
19 stars 19 forks source link

Problem with dependency specification (1.199) #27

Closed eserte closed 8 years ago

eserte commented 8 years ago

t/valid.t fails because of a missing dependency:

Net::Domain::TLD not available at /home/cpansand/.cpan/build/2016032715/Email-Valid-1.199-JHmWve/blib/lib/Email/Valid.pm line 214.
# Looks like you planned 39 tests but ran 5.
# Looks like your test exited with 2 just after 5.
t/valid.t .............. 
Dubious, test returned 2 (wstat 512, 0x200)
Failed 34/39 subtests 

It seems that t/00-report-prereqs.t knew this:

# 
# Versions for all modules listed in MYMETA.json (including optional ones):
# 
...
# === Runtime Requires ===
# 
#     Module            Want     Have
#     ----------------- ---- --------
...
#     Net::Domain::TLD  1.65  missing
...
# *** WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING ***
# 
# The following REQUIRED prerequisites were not satisfied:
# 
# Net::Domain::TLD is not installed (version '1.65' required)
# 

... but it's not listed in the prerequisites section of the report:

------------------------------
PREREQUISITES
------------------------------

Prerequisite modules loaded:

requires:

    Module              Need  Have    
    ------------------- ----- --------
    Mail::Address       0     2.14    
    Net::DNS            0     1.05    
    perl                5.006 5.023009
    Scalar::Util        0     1.44    
    Test::More          0     1.001014

build_requires:

    Module              Need  Have    
    ------------------- ----- --------
    ExtUtils::MakeMaker 0     7.10_01 
    File::Spec          0     3.63    
    Test::More          0.96  1.001014
rjbs commented 8 years ago

Thanks. This was fixed and uploaded, but not yet indexed, at the time of your report. 1.200 should be okay.