search

BlenderVR / source

Blender Virtual Reality Main Code
52 stars 21 forks source link

Windows #39

Closed dfelinto closed 8 years ago

dfelinto commented 8 years ago

This 3 patches solves the problems I was having with mixing windows and unix machines in the same BlenderVR setup:

  1. Replaces all the calls to os.path.join to "/".join. This prevents mixed / \ in the same filepath.
  2. Remove 'env' from subprocess
  3. Have both Windows and Unix to name their vr files with the period.

The only change I'm not sure about is the second. It would be nice to know why we were passing the env in the first place, to make sure things are still working.

dfelinto commented 8 years ago

Note: the changes to 'docs' don't need to be in the final patch

PyrApple commented 8 years ago

sound fix. merged (I did not have time to test on windows master/slave, I'll take your word on this dalai)