Skip to content

Include MANIFEST and SKIP in GH repo #184

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jkeenan
Copy link
Contributor

@jkeenan jkeenan commented May 12, 2025

Git should be aware of MANIFEST and MANIFEST.SKIP. Once your code is on github.com, someone may fork it and want to run it through the typical 'perl Makefile.PL -- make -- make test' cycle. Save them the trouble of calling 'make manifest' unless they've modified either of those files.

Git should be aware of MANIFEST and MANIFEST.SKIP.  Once your code is on
github.com, someone may fork it and want to run it through the typical
'perl Makefile.PL -- make -- make test' cycle.  Save them the trouble of
calling 'make manifest' unless they've modified either of those files.
@haarg
Copy link
Contributor

haarg commented May 12, 2025

A MANIFEST file isn't needed to run any of perl Makefile.PL, make, make test, or even make install. It is only used for make dist. It is a generated file only needed when making releases, and including it in the repo is generally just noise.

@khwilliamson
Copy link
Contributor

Any objections to my closing this request without merging?

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.

3 participants