Removed six dependency in tools/git since it breaks hermetic builds

[c4rtune]

git_configure.bzl uses the host python to run tools/git/gen_git_source.py

It looks like in a cross compatibility migration (tensorflow/tensorflow@c396546) a six dependency was added. If building on a completely vanilla system, this will break if six is not installed on the host. Ideally, this should be done with python provided by bazel, but these rules are called during tf_workspace, bind, etc... So I think this approach is acceptable.

Small change, but also adds consistency! See https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/git/gen_git_source.py#L171

[MonnetalX]

🔍 PR Link Analysis

🔗 Ranked Links

#1 tools/git/gen_git_source.py

🔍 The six dependency in gen_git_source.py is unnecessary because the script only uses standard Python libraries.

#2 tensorflow/tensorflow@c396546

🔍 The six dependency was added during the PY3 migration to ensure compatibility between Python 2 and 3, but it can break hermetic builds on systems where six is not installed.

#3 https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/git/gen_git_source.py#L171

🔍 The six dependency in gen_git_source.py was added during a cross-compatibility migration but can break hermetic builds on vanilla systems where six is not available.

#4 git_configure.bzl

🔍 The six dependency in git_configure.bzl can break hermetic builds on vanilla systems because it relies on the host Python environment.