Skip to content
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

[Bug]: ani-cli cmd shim calls wsl bash instead of git's bash, causing errors #13385

Closed
3 tasks done
ykhan21 opened this issue Jun 1, 2024 · 2 comments · Fixed by ScoopInstaller/Scoop#6035
Closed
3 tasks done
Labels

Comments

@ykhan21
Copy link

ykhan21 commented Jun 1, 2024

Prerequisites

  • I have written a descriptive issue title.
  • I have searched all issues/PRs to ensure it has not already been reported or fixed.
  • I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name

ani-cli

Expected/Current Behaviour

On cmd and powershell, on certain Windows computers, running ani-cli will cause WSL errors. This is because the ani-cli cmd shim, calls bash, which may link WSL bash, depending on if the computer used WSL.
It used to be that the cmd shim with call git for Windows' bash, which did not cause an error.
Running ani-cli on git for windows' bash works, because the ani-cli sh shim is called instead of the cmd shim.

Steps to Reproduce

1. Get a Windows computer that has used WSL previously.
2. Run `scoop buckets add extras` and `scoop install ani-cli`.
3. Run `ani-cli` in cmd or powershell.

Possible Solution

Replace bash with git for windows' bash in the cmd shim. PR #13342 makes this change after scoop install ani-cli.

Scoop and Buckets Version

Current Scoop version:
d337bb1f chore(release): Bump to version 0.4.2 (#5964)

'main' bucket:
da6185d09 shadowsocks-rust: Update to version 1.19.2

'extras' bucket:
74721d5bd upscayl: Update to version 2.11.5

'versions' bucket:
7962a2188 zig-dev: Update to version 0.13.0-dev.349

'nerd-fonts' bucket:
a6b9261c SarasaGothic: Update to version 1.0.13

'nonportable' bucket:
fe9ee98 k-lite-codec-pack-standard-np: Update to version 18.3.5

Scoop Config

last_update          scoop_repo                              scoop_branch
-----------          ----------                              ------------
6/1/2024 11:35:36 AM https://github.com/ScoopInstaller/Scoop master

PowerShell Version

Name                           Value
----                           -----
PSVersion                      7.3.9
PSEdition                      Core
GitCommitId                    7.3.9
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0

Additional Softwares

No response

@ykhan21
Copy link
Author

ykhan21 commented Jun 2, 2024

A better solution would be if you could specify in the scoop manifest whether to use a native bash (from git for windows or msys) or a wsl bash (C:\Windows\System32\bash.exe) in generated cmd shims.

@ykhan21
Copy link
Author

ykhan21 commented Aug 15, 2024

@niheaven, ScoopInstaller/Scoop#6006 doesn't work to fix this issue.
ani-cli, and apps like it, are expecting to use Git Bash's bash.exe, even when ani-cli is run from cmd.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant