Minor script cleanup - Githubissues

Mysterie / uncompyle2

uncompyle2

642 stars 148 forks source link

Minor script cleanup #13

Closed r-moeritz closed 11 years ago

r-moeritz commented 11 years ago

Fix inconsistencies in uncompyle2 script, rename to scripts/uncompyle2 & add to setup.py (these are all changes from wibiti).
Change script headers so they work on Windows.
Remove build folder from repo & add .gitignore so it doesn't inadvertently get added again.

Mysterie commented 11 years ago

Hello!

Thx for the pull request, I'll change the file header for windows. And I'll add your .gitignore. But I like the idea of uncompyle2.py in the root directory and not in a script directory. I think it's simplier don't you ?

r-moeritz commented 11 years ago

I don't really mind if it's in a scripts sub-directory or in the root, but what I like about dropping the .py suffix is that you can then invoke it like a regular command from the shell. Also, I usually do a python setup.py install so I find it no less convenient if it's in a sub-directory.