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

(GH-1859) Fix error message for missing 32-bit URL #1867

Closed
wants to merge 1 commit into from

Conversation

easybe
Copy link
Contributor

@easybe easybe commented Jun 27, 2019

For 64-bit-only packages, when running Get-ChocolateyWebFile.ps1 on a
64-bit system having $Env:ChocolateyForceX86 = $true, the error
message would read: "This package does not support 64 bit architecture."
Instead, the message should inform that the package does not support
32-bit systems.

Closes #1859

For 64-bit-only packages, when running Get-ChocolateyWebFile.ps1 on a
64-bit system having `$Env:ChocolateyForceX86 = $true`, the error
message would read: "This package does not support 64 bit architecture."

Instead, we should inform that the package does not support 32-bit
systems.
@easybe
Copy link
Contributor Author

easybe commented Jun 27, 2019

@@ -262,7 +262,7 @@ param(
# If we're on 32 bit or attempting to force 32 bit and there is no
# 32 bit url, we need to throw an error.
if ($url -eq $null -or $url -eq '') {
throw "This package does not support $bitWidth bit architecture."
throw "This package does not support $bitPackage architecture."
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Member

@ferventcoder ferventcoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@ferventcoder
Copy link
Member

Rebased and merged into stable at 34151e5. Thanks so much!

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

Successfully merging this pull request may close these issues.

Wrong error message shown when using Get-ChocolateyWebFile and forcex86
2 participants