In my koha development system, I was doing testing: Valid Cases to check ------------------------- ./koha_perl_deps ./koha_perl_deps -h ./koha_perl_deps -a ./koha_perl_deps -m ./koha_perl_deps -i ./koha_perl_deps -u ./koha_perl_deps -a -c ./koha_perl_deps -m -c ./koha_perl_deps -i -c ./koha_perl_deps -u -c ./koha_perl_deps -m -u ./koha_perl_deps -m -u -c ------------------------- Invalid Cases to check ------------------------- ./koha_perl_deps -s ./koha_perl_deps -s -u ./koha_perl_deps -s -u -c ------------------------- Cases 3, 4, 7, 8, 11, and 12 (Numbered from 1 sequentially in the above list) would fail on Test::Strict which is does not have a defined version required. I thought of patching koha_perl_deps.pl, which bypasses the problem. However, I think the problem lies in the C4::Installer::PerlDependencies module. Only Test::Strict is lacking a 'min_ver'. The 'required' is set to the version number. All other 'required' values are '1' or '0'. This leads me to believe that 'required' should be 'min_ver' and 'required' => '1' be added.
Created attachment 11060 [details] [review] Added 'require' => '1', and moved old value to a 'min_ver' => '0.14' line Before the patch: koha_perl_deps.pl -a should generate an error on Test::Strict After the patch: koha_perl_deps.pl -a should not generate an error on Test::Strict
(In reply to comment #1) > Created attachment 11060 [details] [review] > Added 'require' => '1', and moved old value to a 'min_ver' => '0.14' line > > Before the patch: > koha_perl_deps.pl -a should generate an error on Test::Strict > After the patch: > koha_perl_deps.pl -a should not generate an error on Test::Strict I'm not sure Test::Strict is required, I think it could be 0. Its only used by one database dependent test, so will not be run when calling make test. If you rework the patch to make it not required, I'll sign off.
See also bug 7368 latest patch
Okay, I see that the patch for bug 7368 does fix this problem as Chris desired. I think I'll mark this as DUPLICATE as a result. Thanks for pointing this out, M. de Rooy. *** This bug has been marked as a duplicate of bug 7368 ***