fix permissions on 00_setup_env_native_fuzz_with_msan.sh #21864

pull glozow wants to merge 1 commits into bitcoin:master from glozow:2021-05-lint changing 1 files +0 −0

glozow commented at 6:52 PM on May 5, 2021: member

I think there was a silent merge conflict between #21852 and #21740?

File "ci/test/00_setup_env_native_fuzz_with_msan.sh" contains a shebang line, but has the file permission 644 instead of the expected executable permission 755. Do "chmod 755 ci/test/00_setup_env_native_fuzz_with_msan.sh" (or remove the shebang line).
ERROR: There were 1 failed tests in the lint-files.py lint test. Please resolve the above errors.
^---- failure generated from test/lint/lint-files.sh

fix permissions on 00_setup_env_native_fuzz_with_msan d48565d109
glozow commented at 6:57 PM on May 5, 2021: member

cc @MarcoFalke (Looks like the lint job passed, yay)
MarcoFalke commented at 6:58 PM on May 5, 2021: member

ACK d48565d10968171edeeb75c1a0af8f4119bb99bc

Thanks! :rocket:
MarcoFalke merged this on May 5, 2021
MarcoFalke closed this on May 5, 2021
glozow deleted the branch on May 5, 2021
windsok commented at 7:16 PM on May 5, 2021: contributor

Apologies for not noticing this. Thanks for fixing it.
sidhujag referenced this in commit 415bdb814f on May 5, 2021
gwillen referenced this in commit 086ab9fa9c on Jun 1, 2022
DrahtBot locked this on Aug 16, 2022

Contributors

glozow

MarcoFalke

windsok

Linked (view graph)

#21740 test: add new python linter to check file names and permissions #21852 ci: Add msan fuzz config