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

BUG: Avoid trying to graft missing examples directory #15373

Merged
merged 1 commit into from
Feb 12, 2017
Merged

BUG: Avoid trying to graft missing examples directory #15373

merged 1 commit into from
Feb 12, 2017

Conversation

neirbowj
Copy link
Contributor

The examples directory was removed by #9391. Remove it from the MANIFEST too.

@jreback
Copy link
Contributor

jreback commented Feb 12, 2017

good point

out of curiosity does this have an effect when installing? (e.g. that it's not there)

@neirbowj
Copy link
Contributor Author

With some combinations of python (3.4.6) and setuptools (at least 32.1.0), graft traces with FileNotFoundError instead of a warning when a directory is missing. For example, see devel/py-setuptools: egg_info fails under python 3.4.6 on nonexistent MANIFEST graft.

I wouldn't expect it to make any difference in other cases.

@neirbowj
Copy link
Contributor Author

The problem exists for setuptools < v34.0.1.

Adopting globs avoids the underlying trigger in 3.4.6: pypa/setuptools@9cb83c3

@jreback jreback added the Build Library building on various platforms label Feb 12, 2017
@jreback jreback added this to the 0.20.0 milestone Feb 12, 2017
@jreback jreback merged commit 2f971a2 into pandas-dev:master Feb 12, 2017
@jreback
Copy link
Contributor

jreback commented Feb 12, 2017

thanks @neirbowj

@neirbowj neirbowj deleted the clean_manifest branch February 14, 2017 13:04
AnkurDedania pushed a commit to AnkurDedania/pandas that referenced this pull request Mar 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Library building on various platforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants