abseil/abseil-py

Python files are marked executable

gagath opened this issue · 0 comments

Most of the test Python files in the repository are marked executable:

$ find . -name '*.py' -perm 755 
./absl/tests/app_test_helper.py
./absl/logging/tests/logging_functional_test.py
./absl/logging/tests/logging_functional_test_helper.py
./absl/testing/tests/flagsaver_test.py
./absl/testing/tests/xml_reporter_test.py
./absl/testing/tests/absltest_sharding_test.py
./absl/testing/tests/parameterized_test.py
./absl/testing/tests/absltest_sharding_test_helper.py
./absl/testing/parameterized.py
./absl/testing/xml_reporter.py
./absl/testing/flagsaver.py

But they do not have any shebang:

$ find . -name '*.py' -perm 755 | xargs -n1 head -n2
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#
# Copyright 2017 The Abseil Authors.
#

Which means that the files should not be executable because the executable bit is not used.