Skip to content
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

Creating MANIFEST.in file #13

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

Creating MANIFEST.in file #13

wants to merge 1 commit into from

Conversation

Jakidxav
Copy link
Contributor

@Jakidxav Jakidxav commented Mar 8, 2021

  • Changing setup.py file to check for MANIFEST.in file

  • MANIFEST.in file now includes and excludes correct files and directories for building source distribution

To test the source build, please run:

python3 setup.py sdist

This will create a folder dist/ which includes a tar file. From a Linux terminal, you can run:

tar -xvzf varsomedata-1.0.0.tar.gz

or to unpack it.

You should notice a couple of things:

  1. The examples/, utilities/, varsomdata/, and varsomscripts/ directories are included in the build. This means that the output/ and resources/ directories are not included, which was the point of setting up the MANIFEST.in file.

  2. Additionally, the experimental/, config, test directories and the setenvironment.py file are not included. The config files are in the .gitignore file, so it did not make sense to include the setenvironment.py file which uses those files. However, this would be easy to include if we wanted any of these files or directories for the first release to PyPI.

MANIFEST.in file now includes and excludes correct files and directories for building source distribution
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.

2 participants