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

Include the test suite in the sourcetarball but do not install it. #363

Merged
merged 1 commit into from
Feb 25, 2016

Conversation

fladi
Copy link
Contributor

@fladi fladi commented Feb 9, 2016

I reworked the MANIFEST.in to include the whole test suite so it can be used
by distribution packages during build time. It is excluded from the installed
packages automatically.

The inclusion rules for the documentation were also made more verbose to prevent
build artifacts from entering the source tarball (think .pyc files).

@vstoykov
Copy link
Collaborator

@fladi I made some changes to travis configuration to finally include Python 3.5 correctly. Can you rebase your branch?

I reworked the `MANIFEST.in` to include the whole test suite so it can be used
by distribution packages during build time. It is excluded from the installed
packages automatically.

The inclusion rules for the documentation were also made more verbose to prevent
build artifacts from entering the source tarball (think .pyc files).
@fladi fladi force-pushed the include_tests_in_source branch from 5f0eafa to f2255a5 Compare February 25, 2016 07:40
@fladi
Copy link
Contributor Author

fladi commented Feb 25, 2016

@vstoykov thanks, rebased my branch as requested.

vstoykov added a commit that referenced this pull request Feb 25, 2016
Include the test suite in the sourcetarball but do not install it.
@vstoykov vstoykov merged commit 52ad8a0 into matthewwithanm:develop Feb 25, 2016
@vstoykov
Copy link
Collaborator

Thanks @fladi for your contribution! I'm really glad that this project will be packaged in Debian!

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