diff options
| author | Yuki Okushi <huyuumi.dev+love@gmail.com> | 2022-11-01 12:03:40 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-01 12:03:40 +0900 |
| commit | a6ac6b20f35f4d35d54a686ec6410bbb91373923 (patch) | |
| tree | 037208ce9ae2d6b9b2a6bdcb7cfe1bf5d48fe1db | |
| parent | 024207ab43aceb49f2ca957509c503ccf12089d7 (diff) | |
| parent | c83ddaef948e5aadf3e64169dd3ab4268834fbff (diff) | |
| download | rust-a6ac6b20f35f4d35d54a686ec6410bbb91373923.tar.gz rust-a6ac6b20f35f4d35d54a686ec6410bbb91373923.zip | |
Rollup merge of #103007 - albertlarsan68:better-python-discovery, r=jyn514
Add better python discovery The Microsoft Store version of Python installs itself as `pythonM.m`, with `M` being the major version and `m` the minor. The `x.ps1` script will now search for python executables whose command matches the regex `python\d`. The `\d` at the end is to protect from using the `pythonw` versions, which do not work as standard python.
| -rwxr-xr-x | x | 6 | ||||
| -rwxr-xr-x | x.ps1 | 13 |
2 files changed, 18 insertions, 1 deletions
diff --git a/x b/x index 704d0f791f3..4309b82627c 100755 --- a/x +++ b/x @@ -29,5 +29,11 @@ for SEARCH_PYTHON in py python3 python python2; do exec "$python" $extra_arg "$xpy" "$@" fi done + +python=$(bash -c "compgen -c python" | grep '^python[2-3]\.[0-9]\+$' | head -n1) +if ! [ "$python" = "" ]; then + exec "$python" "$xpy" "$@" +fi + echo "$0: error: did not find python installed" >&2 exit 1 diff --git a/x.ps1 b/x.ps1 index 86cea606591..81b98919f43 100755 --- a/x.ps1 +++ b/x.ps1 @@ -10,11 +10,15 @@ foreach ($arg in $args) { $xpy_args += """$arg""" } +function Get-Application($app) { + return Get-Command $app -ErrorAction SilentlyContinue -CommandType Application +} + foreach ($python in "py", "python3", "python", "python2") { # NOTE: this only tests that the command exists in PATH, not that it's actually # executable. The latter is not possible in a portable way, see # https://github.com/PowerShell/PowerShell/issues/12625. - if (Get-Command $python -ErrorAction SilentlyContinue) { + if (Get-Application $python) { if ($python -eq "py") { # Use python3, not python2 $xpy_args = @("-3") + $xpy_args @@ -24,5 +28,12 @@ foreach ($python in "py", "python3", "python", "python2") { } } +$found = (Get-Application "python*" | Where-Object {$_.name -match '^python[2-3]\.[0-9]+(\.exe)?$'}) +if (($null -ne $found) -and ($found.Length -ge 1)) { + $python = $found[0] + $process = Start-Process -NoNewWindow -Wait -PassThru $python $xpy_args + Exit $process.ExitCode +} + Write-Error "${PSCommandPath}: error: did not find python installed" Exit 1 |
