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

add support for uploading windows installers #26

Closed
wants to merge 1 commit into from
Closed

add support for uploading windows installers #26

wants to merge 1 commit into from

Conversation

schmir
Copy link
Contributor

@schmir schmir commented Dec 31, 2013

twine is currently not able to upload bdist_wininst generated windows installers. this pull requests adds support for it.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5f52ea5 on schmir:wininstaller into ef5af8d on dstufft:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 6e05425 on schmir:wininstaller into ef5af8d on dstufft:master.

@dstufft
Copy link
Member

dstufft commented Jan 7, 2014

Just a FYI that I'm planning on merging this. I just need to test it still.

@schmir
Copy link
Contributor Author

schmir commented Jan 7, 2014

thanks for the feedback. one problem I noticed is that I couldn't upload greenlet's wininstaller for python 2.4. the zipfile contains just greenlet.pyd and greenlet.h.

.msi files also cannot be uploaded. Do you have any plans to fix that? (something like a switch to "read metadata from another file but upload that .msi/.exe")

@ariddell
Copy link

I just rebased this and it seems to be working fine for .exe files. 👍

@sigmavirus24
Copy link
Member

This looks good to me. If you can give this a rebase @schmir I'll give it a test and merge it.

@sigmavirus24 sigmavirus24 modified the milestone: 1.4 Dec 6, 2014
@sigmavirus24
Copy link
Member

I just noticed that this closes #64 as well

@schmir
Copy link
Contributor Author

schmir commented Dec 6, 2014

Sorry, but I don't use python anymore and I'm not interested in doing that work.

@sigmavirus24
Copy link
Member

Superseded by #68

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants