seomoz / shovel

Rake, for Python
MIT License
660 stars 49 forks source link

Windows support #14 #32

Closed recfab closed 9 years ago

recfab commented 10 years ago

In setup.py, switch scripts to entry_points so that a shovel-script.py and shovel.exe will be generated.

In shovel/runner.py, explicitly set the prog argument to the argparser so that the usage shows shovel instead of shovel-script.py.

In the tests, normalize the paths to fix forward slash vs backslash conflicts.

NOTE: There is still an error in one of the tests. That test doesn't seem special, so I'm not sure why that one in particular is causing trouble. I will look into it further.

recfab commented 9 years ago

I apologize for the time between posts. Work got in the way, and I forgot to check back in.

Anyway, I just started looking into it again, and the errors I was seeing are gone. Perhaps one of the dependencies was updated in the meantime?

I created a 2.7 virtualenv and a 3.4 virtualenv. python setup nosetests passes in both of those environments.

Is there anything additional you would like to see? Perhaps Windows CI? I've never used Travis, but my understanding is that it is Linux only. I use Appveyor for my .NET projects and would be happy to look into that.

dlecocq commented 9 years ago

This works for me. It didn't bork my shovel binary and didn't break the tests, which is all I really care about :-)