eecs485staff / madoop

A light weight MapReduce framework for education
MIT License
9 stars 4 forks source link

Fix directories containing spaces #73

Closed aarmaane closed 8 months ago

aarmaane commented 8 months ago

Fixes #72. This could have also been fixed by adding quotations around the path or replacing spaces with their escape character versions. But this seems best since there's no need for any special shell syntax. It's just running a file from a path as-is.

Unintended side effect: subprocess.run will now throw OSError exceptions that are much more descriptive than the previous error codes (i.e. Missing a shebang will throw errno.ENOEXEC and a missing file will throw FileNotFoundError).