-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[python] prepare for PyPI #635
Conversation
typo: PyPI (pypi), not pipy :) |
I can do refactoring |
@henry0312 can you refactor it base on this branch, then creat a PR to that branch? |
@guolinke yes, I can, but I'll have been busy until the end of June. |
@wxchan any comments? |
Is it possible to remove |
MANIFEST.in can be simplified a little:
build_pip.py seems not necessary? we can run |
@wxchan |
ok, hope it wouldn't be confused to old users. what is the getopt error you talked about yesterday? |
ok, no other issue. |
@wxchan Do you have any idea about this ? |
python-package/setup.py
Outdated
use_precompile = True | ||
except getopt.GetoptError as err: | ||
pass | ||
sys.argv = sys.argv[0:2] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got the same error on my mac, fix by removing this line
* add make command to the python package. * Update README.rst * Update README.rst * Update README.rst * fix tests. * fix unix build * update readme * fix setup.py * update travis * Update .travis.yml * Update test.py * some fixes. * check the 64-bit python * fix build. * refine MANIFEST.in * update Manifest.in * add more build options. * Add fatal in cmake * fix a endif. * fix bugs. * fix pep8 * add test for the pip package build * add test pip install in travis. * fix version with pre-compile dll * fix readme.rst * update readme
No description provided.