google / oss-fuzz

OSS-Fuzz - continuous fuzzing for open source software.
https://google.github.io/oss-fuzz
Apache License 2.0
10.13k stars 2.16k forks source link

Upgrade capstone to use python3 #12028

Closed twizmwazin closed 1 week ago

twizmwazin commented 1 month ago

Upstream has ended python2 support. When removing python2 compatibility code, it was noticed that oss-fuzz was still utilizing this support. This fixes that by moving to a supported version of python3.

See this PR: https://github.com/capstone-engine/capstone/pull/2378

google-cla[bot] commented 1 month ago

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

github-actions[bot] commented 1 month ago

twizmwazin is a new contributor to projects/capstone. The PR must be approved by known contributors before it can be merged. The past contributors are: kabeor, vanhauser-thc, inferno-chromium, catenacyber, StalkR, devtty1er, Dor1s

XVilka commented 1 month ago

twizmwazin is a new contributor to projects/capstone. The PR must be approved by known contributors before it can be merged. The past contributors are: kabeor, vanhauser-thc, inferno-chromium, catenacyber, StalkR, devtty1er, Dor1s

@kabeor could you please take a look?

maflcko commented 3 weeks ago

CI failed with: /src/build.sh: ./test_corpus.py: /usr/bin/python: bad interpreter: No such file or directory

kabeor commented 3 weeks ago

Hello, we have merged the relevant PR upstream, so we hope to rerun it to apply the patched code, thanks.

Rot127 commented 1 week ago

@maflcko Could you please trigger the CI again?

maflcko commented 1 week ago

I am not a maintainer here. Maybe @DavidKorczynski can help?

XVilka commented 1 week ago

@kabeor you probably need to approve it again

Rot127 commented 1 week ago

@DavidKorczynski ping

DavidKorczynski commented 1 week ago

retriggered CI