Skip to content

Commit

Permalink
Exclude bundled thrift, install_requires it instead
Browse files Browse the repository at this point in the history
Correctly package evernote by excluding any bundled thrift modules from
installation and use install_requires to depend on the (PyPI) package
instead. This prevents package/installation conflicts between other
Python packages that depend on thrift.
  • Loading branch information
koobs authored and iafan committed Mar 28, 2018
1 parent 6c611b7 commit 1ddba82
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ def read(fname):
description='Evernote SDK for Python',
long_description=read('README.md'),
packages=find_packages('lib'),
package_dir={'': 'lib'},
packages=find_packages('lib',exclude=["*.thrift", "*.thrift.*", "thrift.*", "thrift"]),
classifiers=[
'Development Status :: 5 - Production/Stable',
'Intended Audience :: Developers',
Expand All @@ -28,6 +28,7 @@ def read(fname):
],
license='BSD',
install_requires=[
'thrift',
'oauth2',
],
)

0 comments on commit 1ddba82

Please sign in to comment.