Use python3 from $PATH instead of absolute paths

facebook / mysql-5.6

Facebook's branch of the Oracle MySQL database. This includes MyRocks.

http://myrocks.io

Other

2.46k stars 711 forks source link

Use python3 from $PATH instead of absolute paths #1466

Open laurynas-biveinis opened 3 weeks ago

laurynas-biveinis commented 3 weeks ago

This allows using Python virtualenv directly. This partially reverts changes in 9d0dda0c6565f1a7d926e5c82628cb134d4b1ac9, but its commit message is not stating anything about absolute paths.

facebook-github-bot commented 3 weeks ago

@luqun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

luqun commented 1 day ago

after change from /usr/bin/python3 to pythohn3, some of MTR failed to find due to "ModuleNotFoundError: No module named 'MySQLdb'".

laurynas-biveinis commented 1 day ago

Do you know what python3 are the scripts picking up? Is it possible to adjust PATH to put /usr/bin before that?

luqun commented 5 hours ago

Do you know what python3 are the scripts picking up? Is it possible to adjust PATH to put /usr/bin before that?

the pyhon3 in /usr/local/sbin instead of /usr/bin/python3.. need to do some investigation since this PATH variable comes from SYSTEM.