New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade Python tools to Python3 #5846
Conversation
|
cd3246b
to
e59a81e
Compare
@theopolis apparently the failed checks are issue with |
Thanks for the work here, do you mind rebasing onto master to fix the CI false positive? |
I can trigger the rebase too apparently. |
and remove codegen files
@theopolis I think this is now ready for review whenever you have time! |
A work in progress PR
This PR fixes #5762
Note: Commits will be squashed after all the changes are done!
The Python shebang lines should be #!/usr/bin/env python3 unless the code is compatible with both Python 2.x and 3.x. If our script says it requires Python2 but the shebang line just says python, it will fail if the system resolves that to Python 3.
Python in the osquery build and test systems:
osquery/tools/analysis/*
osquery/tools/build_defs/oss/osquery/*
osquery/tools/codegen/*
osquery/tools/deployment/*
osquery/tools/get_platform.py
osquery/tools/hooks/pre_commit.py
osquery/tools/tests/*
osquery/tools/cmake/downloader.py
osquery/tools/formatting/*