Skip to content

Filter out perl from listdeps --omit-core - #739

Merged
rjbs merged 1 commit into
rjbs:mainfrom
Leont:listdeps-perl
Jan 25, 2026
Merged

rjbs merged 1 commit into
rjbs:mainfrom
Leont:listdeps-perl

Conversation

@Leont

@Leont Leont commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

This will filter out perl from listed dependencies when --omit-core satisfies the requirement.

E.g. previously dzil listdeps --json --omit-core=5.042 would look something like:

{
   "configure" : {
      "requires" : {
         "perl" : "v5.13.7"
      }
   },
   "develop" : {
      "requires" : {
         "Crypt::Argon2" : "0",
         "Crypt::Bcrypt" : "0",
         "Test::Pod" : "1.41"
      }
   },
   "runtime" : {
      "requires" : {
         "Crypt::SysRandom" : "0",
         "perl" : "v5.13.7"
      }
   },
   "test" : {
      "requires" : {
         "perl" : "5.006"
      }
   }
}

that will now look like:

{
   "develop" : {
      "requires" : {
         "Crypt::Argon2" : "0",
         "Crypt::Bcrypt" : "0",
         "Test::Pod" : "1.41"
      }
   },
   "runtime" : {
      "requires" : {
         "Crypt::SysRandom" : "0"
      }
   }
}

@karenetheridge

Copy link
Copy Markdown
Contributor

This would be a handy option to use on the PrereqSource role -- when finalizing all prereqs, omit those that are in core as of the minimum required perl version.

@Grinnz

Grinnz commented Dec 9, 2025 •

Copy link
Copy Markdown
Contributor

Looks like someone had that idea as part of their plugin: https://metacpan.org/pod/Dist::Zilla::Plugin::PrereqsClean

I think it would make sense as an AutoPrereqs option, but otherwise, prefer to do that determination in my manual prereq specification.

(anyway getting a bit off topic for this PR)

@karenetheridge

Copy link
Copy Markdown
Contributor

yeah, filtering it in AutoPrereqs sounds fine.

@Leont

Leont commented Dec 9, 2025

Copy link
Copy Markdown
Contributor Author

anyway getting a bit off topic for this PR

Yeah, kind of.

This will filter out perl from listed dependencies when --omit-core
satisfies the requirement.
@rjbs
rjbs merged commit 0523a9a into rjbs:main Jan 25, 2026
1 check failed
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

Successfully merging this pull request may close these issues.

4 participants