-
-
Notifications
You must be signed in to change notification settings - Fork 933
watch out for .pyc files in uploads to pypi #419
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
Comments
Quite amazing that you can see the username and groupname by extracting a tar archive. Should I be worried ? |
Thanks for bringing this to our attention! I can verify this behaviour. To replicate, I cleaned all of my |
(The username and group name are quite normal to find in tarballs, btw.) |
I believe this is now fixed by 25c95ac. |
Thanks for reporting! |
I'd like to introduce you two, just in case you don't know each other yet. @nvie is the one contributor of GitPrime that I happen to know by name, who by now can do anything from pushing to all repos to making new releases. @yarikoptic has been with the project for quite a while as maintainer of the debian packages. |
thanks @Byron for the introduction ;) @nvie, "I am not a whiner but that is my life" would be my self-introduction to clarify my role here I guess. We also use GitPython somewhat in our http://github.com/datalad/datalad project, so I am quite often to whine first ;) as for uploads to pypi, if you are up for it, you could automate them even further, see e.g. https://docs.travis-ci.com/user/deployment/pypi and http://5minutes.youkidea.com/howto-deploy-python-package-on-pypi-with-github-and-travis.html . I did that for duecredit project. Seems to work quite nicely. The only work of caution is that both tag and state of the branch (i.e. master tagging/release from it) should be uploaded at the same time. |
Thanks, @Byron! Good to meet you, @yarikoptic. We're using GitPython extensively at www.gitprime.com, and we're investing in its maintenance and further feature development. Thanks for the links, I'll have a look. |
atm, tarball I have fetched from pypi contains some .pyc's I believe
The text was updated successfully, but these errors were encountered: