in reply to Re^3: Inline::CUDA : looking for feedback
in thread Inline::CUDA : looking for feedback

apropos Test::CheckManifest being a requirement, I was just scanning all files for external packages and I saw that in t/manifest.t created by module-starter

#!perl use 5.006; use strict; use warnings; use Test::More; unless ( $ENV{RELEASE_TESTING} ) { plan( skip_all => "Author tests not required for installation" ); } my $min_tcm = 0.9; eval "use Test::CheckManifest $min_tcm"; plan skip_all => "Test::CheckManifest $min_tcm required" if $@; ok_manifest();

Replies are listed 'Best First'.
Re^5: Inline::CUDA : looking for feedback
by kcott (Bishop) on Jul 30, 2021 at 09:22 UTC

    The issue isn't so much that module-starter adds a use Test::CheckManifest to t/manifest.t; rather, it's that you added Test::CheckManifest under PREREQ_PM in Makefile.PL (module-starter doesn't do that).

    I was questioning the inclusion of Test::CheckManifest in Makefile.PL, not its presence in t/manifest.t. I could tell (by inspection) that Test::CheckManifest in Makefile.PL would be a problem: I force-installed Test::CheckManifest to circumvent that issue.

    Take a look at "Re: Perl tools for making code better" for an example of my equivalent of manifest.t, along with some related information.

    — Ken