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

Upload to PyPi and make installation hella easier #4

Open
cmutel opened this issue May 20, 2015 · 4 comments
Open

Upload to PyPi and make installation hella easier #4

cmutel opened this issue May 20, 2015 · 4 comments

Comments

@cmutel
Copy link
Contributor

cmutel commented May 20, 2015

Its not that tough, here is a simple tutorial:

http://peterdowns.com/posts/first-time-with-pypi.html

@cmutel
Copy link
Contributor Author

cmutel commented May 20, 2015

I am even emailing you the source package and wheels to make your life easier!

@stefanpauliuk
Copy link
Member

Hi Chris,

thanks for taking the time and checking the 2.7 compatibility! I carefully checked your modifications and merged the pull request, plus some other minor cleanup.

I will closely look at pypi and probably upload soon. Would be cool to just write 'pip install pysut' and be done with it :)

One question: You capitalized pydsm.py to pdDSM.py because of the repository name, right? I accepted that change and tried to to the same for pySUT.py, but my capitalisation on Windows does not translate to Github. Plus, a student of mine had problems installing pydsm previously when it was capitalized for the first time. What's your take on this?

Cheers,Stefan

@stefanpauliuk
Copy link
Member

I just re-uploaded pySUT.py and it is capitalized now. So is the call in init.py.
That should do the job.

@cmutel
Copy link
Contributor Author

cmutel commented May 25, 2015

There are three approaches to fix this; all three would work, and the one you choose just depends on your personal style:

  1. Change the name of the file to pysut.py (i.e. make it lower-case)
  2. Do what you already did, change the import line to pySUT in __init__.py
  3. Move the code into the __init__.py file - as there is only one file, you don't need anything else.

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

2 participants