Our user was experiencing issue [1] when using a git repository on NTFS mount
running on Linux. The current check checks if we are running on Windows, but
it should really check if we are on NTFS. And since checking fs type is not
that trivial and not efficient, it is simpler and better to just always apply
NTFS-specific logic, since it works on other filesystems as well.

[1] https://github.com/iterative/dvc/issues/1880#issuecomment-483253764

This should workaround possible permission issues.

Related to https://github.com/gitpython-developers/GitPython/issues/353

For some reason, it gets bytes where it did expect a stream ... .
Probably I should have figured out where this was input, instead
of fixing it the brutal way

autopep8 -v -j 8 --max-line-length 120 --in-place --recursive

It's a nice way of saying that there is still one failing, consistently.

Bytes should always be returned from the streams, so the tests
should be checking against byte strings instead of text strings.

This also fixes the `sha_iter` as it relied on the Python 2
`iterkeys` which has been renamed to `keys` in Python 3.