-
Notifications
You must be signed in to change notification settings - Fork 67
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
Support universal wheels #29
Conversation
Thank you very much ! I hope getting wheels support can be as trivial for smmap and gitpython too. |
Already done https://github.com/gitpython-developers/smmap/blob/master/setup.cfg#L2 https://github.com/gitpython-developers/GitPython/blob/master/setup.cfg#L2 they all just need a new release with |
Thanks for the hint ! Unfortunately, this command-line
It's a bit odd, and I have no explanation for that :/. |
Ah I know the issue. Please await a PR... On 11 Sep 2016 17:48, wrote:
|
python3 -m venv /tmp/gitpython/ On 11 Sep 2016 17:44, "Sebastian Thiel" notifications@github.com wrote: Thanks for the hint ! Unfortunately, this command-line pip3 install wheel Requirement already satisfied (use --upgrade to upgrade): wheel in error: invalid command 'bdist_wheel' It's a bit odd, and I have no explanation for that :/. — |
Thanks again !
Maybe it's something odd with my system installation, which still seems to be part of the mix. |
@Byron I dont' know what's the status of it, but actually the "official" guide does not recommend this command. |
You don't need to bother with twine on python3 On 2 Oct 2016 21:44, "Kostis Anagnostopoulos" notifications@github.com
|
@graingert wrote:
Would you mind explaining a bit more on that point? |
Ah sorry I thought this was a different thread. This package can't do On 2 Oct 2016 22:05, "Kostis Anagnostopoulos" notifications@github.com @graingert https://github.com/graingert wrote: You don't need to bother with twine on python3 Would you mind explaining a bit more on that point? — Reply to this email directly, view it on GitHub |
But there are no C extensions, are they? |
I thought there was... On 2 Oct 2016 22:18, "Kostis Anagnostopoulos" notifications@github.com
|
There are Line 92 in 847a21e
|
Thank you, I'm involved in this project the last days so I miss a lost of stuff. I have not managed to build them - trying also in Appveyor without success. [edit: ankostis] environment:
PYPI_USER: "FOO"
PYPI_PSWD:
# See https://www.appveyor.com/docs/build-configuration/#secure-variables
secure: <paste-your-encrypted-value>
#... regular build here
on_success:
- ps: >-
$cmtid = git rev-parse HEAD
if (git describe --exact-match $cmtid) {
twine upload -u PYPI_USER -p PYPI_PSWD dist/*
} |
On another issue, actually I never really liked git modules for managing project dependencies - they complicate things needlesly. They may have been useful back when python packaging was imature and a major cause of hurdles. Now, eating our wn dogfood, pushng wheels into PyPi and re-using them aterwards should be a better alternative. I'm interesting to here any feedback on a change along these lines (drop submodules from gitdb and gitpython and depend on regulat python wheels. |
It would be good to extract the optional c extension into an optional, On 3 Oct 2016 7:35 pm, "Kostis Anagnostopoulos" notifications@github.com
|
Any PR is welcome. |
No description provided.