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.

Sign up for GitHub

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

Jump to bottom

Installing/uninstalling system-wide packages fails without error #1243

Closed
sebastian-pop opened this issue Sep 13, 2023 · 5 comments · Fixed by #1244
Closed

Installing/uninstalling system-wide packages fails without error #1243

sebastian-pop opened this issue Sep 13, 2023 · 5 comments · Fixed by #1244
Assignees
@alnlarsen
Labels
bug Something isn't working
Milestone
9.22.0

Comments

@sebastian-pop
Copy link
Collaborator

Describe the bug

A clear and concise description of what the bug is.

Environment (please complete the following information):

  • OS:Windows 10
  • OpenTAP 9.21.1

To Reproduce

Steps to reproduce the behavior:

  1. `tap package install "Keysight Fixed License Manager"

  2. Click Continue on the elevated prompt

Actual behavior

No error displayed in the CLI

Keysight Fixed License Manager is not installed

Expected behavior

In case the installation fails, an error should be displayed

Keysight Fixed License Manager should be installed

Additional context

There are also issues when uninstalling
@sebastian-pop sebastian-pop added the bug Something isn't working label Sep 13, 2023
@alnlarsen alnlarsen self-assigned this Sep 13, 2023
@alnlarsen alnlarsen added this to the 9.22.0 milestone Sep 13, 2023
@alnlarsen alnlarsen mentioned this issue Sep 13, 2023
Merged
@github-actions
Copy link

A fix for this is in OpenTAP version 9.22.0-beta.40+2ba3aa29 or later.

@sebastian-pop
Copy link
Collaborator Author

Verified on OpenTAP 9.22.0-beta.43

Install works

Uninstall does not work. Reopening
image

@sebastian-pop sebastian-pop reopened this Sep 26, 2023
@alnlarsen
Copy link
Collaborator

@sebastian-pop does uninstall work if you are already running elevated? I don't think we ever implemented elevation for uninstall

@alnlarsen
Copy link
Collaborator

Elevating for uninstall actions was never implemented in the first place. The issue you are describing is not relevant to this issue. Uninstalling from a manually elevated shell works correctly. Closing this.

@sebastian-pop
Copy link
Collaborator Author

Uninstall issue here #627

Install works now on 9.22.0-rc.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alnlarsen alnlarsen

Labels
bug Something isn't working
Projects
None yet
Milestone
9.22.0
Development

Successfully merging a pull request may close this issue.

Add system-wide packages to list of packages to be installed opentap/opentap
2 participants
@alnlarsen @sebastian-pop