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

Wheel support #32

Closed
kevin-brown opened this issue Apr 11, 2017 · 2 comments
Closed

Wheel support #32

kevin-brown opened this issue Apr 11, 2017 · 2 comments

Comments

@kevin-brown
Copy link

http://pythonwheels.com/

Right now, only the tar.gz file is being distributed on PyPI for applicationinsights, which isn't that much of an issue, but it does add some time to installing the package.

This package should be compatible with the Wheel format, considering it doesn't appear to have any C dependencies and it is compatible with both Python 2 and 3. As a result, you should only need to generate a universal wheel and then everyone (on all systems) will get the ability to install applicationinsights with just the wheel, without having to do any extra work.

@OsvaldoRosado
Copy link
Member

Interesting. It looks like this repo should already be supporting Wheel from the existance of this file: https://github.com/Microsoft/ApplicationInsights-Python/blob/master/setup.cfg

If my understanding is correct

@kevin-brown
Copy link
Author

It should, yes.

The problem is that the wheels are not being built, so they aren't being sent up to PyPI. I just noticed that this appears to be automatic, using Travis CI, so only a small change is needed. You just need to tell Travis to build the wheels.

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

No branches or pull requests

3 participants