Fixes OSError: File name too long when environment variable is too long

[tomaarsen]

Resolves #35

Hello!

Pull request overview

• Fixes OSError: File name too long when environment variable is too long by try-catching for an OSError, and ignoring it when it causes an error due to File name too long.

Details

This PR complies with the EAFP "easier to ask for forgiveness than permission" style as recommended in the Python glossary. It simply writes out the set comprehension in full and wraps each path.exists() call with a try-except which simply considers a candidate path to be non-existent whenever it throws File name too long.

After this PR, the output of the same program as #35 outputs only

...
bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('-};\n do\n if [ "${_m [[removed]]
...

This is the expected behaviour here.

Lastly, the changes should be equivalently efficient as before.

Note

It may be possible that path.exists() can throw other exceptions that maybe should be ignored as well, but this PR only ignores the File name too long exception.

Let me know if you need anything from me at this point!

• Tom Aarsen

[TimDettmers]

Perfect issue and pull request. Thank you for the explanation and the work!