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] short option -a for shovel install broken, long option --arch still works #271

Open
mrplumber opened this issue Jan 29, 2022 · 0 comments
Labels
bug Something isn't working in-NEW Changes are already available in NEW branch. Configure SCOOP_BRANCH to NEW to beta-test changes
Milestone

Comments

@mrplumber
Copy link

mrplumber commented Jan 29, 2022

  • Branch: main
  • Powershell Version: 7.2.1

Bug Report

Current Behavior

shovel install -a 32bit <app> installs 64bit version of the app which also has 32bit (tested with custom manifest, then with main/dirhash)

Expected Behavior

shovel install -a 32bit <app> should install 32bit version of the app (provided app manifest supports it)

Additional context/output

Here is the example output when trying to install 32bit main/dirhash:

PS C:\>shovel install -a 32bit dirhash
Installing 'dirhash' (1.23.0) [64bit] [main]
WARN  By installing you accept following license: BSD-3-Clause (https://spdx.org/licenses/BSD-3-Clause.html)
Loading DirHash-1.23.0-x64.zip from cache
Checking hash of DirHash-1.23.0-x64.zip ... ok.
Extracting DirHash-1.23.0-x64.zip ... done.
Linking ~\scoop\apps\dirhash\current => ~\scoop\apps\dirhash\1.23.0
Creating shim for 'DirHash'.
'dirhash' (1.23.0) was installed successfully!

Output when installing the same app with long option --arch 32bit:

PS C:\>shovel install --arch 32bit dirhash
Installing 'dirhash' (1.23.0) [32bit] [main]
WARN  By installing you accept following license: BSD-3-Clause (https://spdx.org/licenses/BSD-3-Clause.html)
Loading DirHash-1.23.0-x86.zip from cache
Checking hash of DirHash-1.23.0-x86.zip ... ok.
Extracting DirHash-1.23.0-x86.zip ... done.
Linking ~\scoop\apps\dirhash\current => ~\scoop\apps\dirhash\1.23.0
Creating shim for 'DirHash'.
'dirhash' (1.23.0) was installed successfully!
@mrplumber mrplumber added the bug Something isn't working label Jan 29, 2022
@Ash258 Ash258 added the in-NEW Changes are already available in NEW branch. Configure SCOOP_BRANCH to NEW to beta-test changes label Jan 29, 2022
@Ash258 Ash258 added this to the 0.6.5 milestone Jan 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working in-NEW Changes are already available in NEW branch. Configure SCOOP_BRANCH to NEW to beta-test changes
Projects
None yet
Development

No branches or pull requests

2 participants