Bug #122674 for MarpaX-Languages-M4: Undeclared dependency Data::Record

Tue Aug 01 13:37:49 2017 SREZIC [...] cpan.org - Ticket created

Subject:

Undeclared dependency Data::Record

Tests fail if Data::Record is not installed: ... Can't locate Data/Record.pm in @INC (you may need to install the Data::Record module) (@INC contains: ... .) at /usr/perl5.27.2p/lib/site_perl/5.27.2/Module/Runtime.pm line 317, <DATA> line 1683. # Tests were run but no plan was declared and done_testing() was not seen. # Looks like your test exited with 2 just after 1. t/m4.t ................. Dubious, test returned 2 (wstat 512, 0x200) All 1 subtests passed ...

Tue Aug 01 14:00:53 2017 JDDPAUSE [...] cpan.org - Correspondence added

Thanks for the report. This is an implicit dependency of MooX::Options, the kind of things that IMHO should be depended automatically by this package but well, man page of MooX::Options says: autosplit This feature allowes one to split option arguments at a defined character and always return an array (implicit flag "repeatable"). option "search_path" => ( is => "ro", required => 1, autosplit => ":", format => "s" ); However, this feature requires following modules are provided: * Data::Record * Regexp::Common and MooX::Options code has: MooX/Options.pm : # XXX maybe we should warn here since a previously beloved feature isn't enabled automatically eval { use_module("Data::Record"); use_module("Regexp::Common"); } and $options{autosplit} = ','; } MooX/Options/Role.pm: use_module("Data::Record"); So okay, despite I formally disagree having to depend on this since I do not care about MooX::Options implementation, I am pretty sure that MooX::Options will not change to have that as an explicit dependency. Regards

Tue Aug 01 14:00:54 2017 The RT System itself - Status changed from 'new' to 'open'

Wed Aug 02 00:00:42 2017 JDDPAUSE [...] cpan.org - Status changed from 'open' to 'resolved'

Wed Aug 02 00:00:43 2017 JDDPAUSE [...] cpan.org - Fixed in 0.019 added