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

[Package Issue]: JRSoftware.InnoSetup #170762

Closed
2 tasks done
bdovaz opened this issue Aug 30, 2024 · 1 comment · Fixed by #170764
Closed
2 tasks done

[Package Issue]: JRSoftware.InnoSetup #170762

bdovaz opened this issue Aug 30, 2024 · 1 comment · Fixed by #170764
Labels
In-PR Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Triage This work item needs to be triaged by a member of the core team.
Milestone

Comments

@bdovaz
Copy link
Contributor

bdovaz commented Aug 30, 2024

Please confirm these before moving forward

  • I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
  • I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue

Installation issue.

Brief description of your issue

#119886 added the missing user scope but also changed the default behavior and is that the installation if you do not pass the --scope machine parameter explicitly, it does it with user and this behavior was not so until it was changed in this PR.

Steps to reproduce

winget install JRSoftware.InnoSetup.

It installs with user scope when before the mentioned PR it did it with machine.

Actual behavior

Do not change the default scope that has been changed.

Expected behavior

Keep the default scope as machine.

Environment

Windows: Windows.Desktop v10.0.22631.4112
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.23.1911.0

Screenshots and Logs

No response

@bdovaz bdovaz added the Issue-Bug It either shouldn't be doing this or needs an investigation. label Aug 30, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Triage This work item needs to be triaged by a member of the core team. label Aug 30, 2024
@Dragon1573
Copy link
Contributor

Tip

Update contents of #170764 (comment) with "closing keyword" Resolve #170762 will help linking the pull request with this issue. When pull request is successfully merged, this issue will also be closed as completed automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In-PR Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Triage This work item needs to be triaged by a member of the core team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants