Page MenuHomePhabricator

fix more file permissions and shebangs
AbandonedPublic

Authored by PiRK on Feb 13 2024, 09:01.

Details

Reviewers
None
Group Reviewers
Restricted Project
Summary

This makes the file permissions and shebangs conform to the linter added in the next commit.

  • executable files must have a shebang
  • Electrum ABC source files are generally not supposed to be executable, even if they have a if __name__ == "__main__": section (running individual test suites can still be done with python -m electrumabc.tests.<filename>)

Related to D15421

Test Plan

Run the new linter in the next diff

Event Timeline

PiRK requested review of this revision.Feb 13 2024, 09:01

Maybe we could apply the same rules as Electrum ABC to the test_framework/*.py modules that are not supposed to be executables and should not have a shebang, but that can be done in another diff. Currently they are all executable and have a shebang, so the linter will be happy.

PiRK planned changes to this revision.Feb 13 2024, 10:09

Still working on the linter script

PiRK edited the summary of this revision. (Show Details)

rebase

PiRK planned changes to this revision.Feb 13 2024, 16:52

checking something