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] bug in scoop reset #4833

Closed
hgkamath opened this issue Mar 22, 2022 · 1 comment · Fixed by #4896
Closed

[Bug] bug in scoop reset #4833

hgkamath opened this issue Mar 22, 2022 · 1 comment · Fixed by #4896
Labels

Comments

@hgkamath
Copy link

hgkamath commented Mar 22, 2022

In libexec/scoop_reset.ps1 , line 42
I think you need to pass the 2nd argument to the function installed in the if condition as (installed $app $global) in order to correctly reset globally installed package. otherwise it just exits with error saying "ERROR pkg isn't installed"

    $app, $bucket, $version = parse_app $app

    if(($global -eq $null) -and (installed $app $true)) {
        # set global flag when running reset command on specific app
        $global = $true
    }

    if($app -eq 'scoop') {
        # skip scoop
        return
    }

    if(!(installed $app)) { // BUG HERE
        error "'$app' isn't installed"
        return
    }

    if ($null -eq $version) {
        $version = Select-CurrentVersion -AppName $app -Global:$global
    }

Bug Report

Current Behavior

Expected Behavior

Additional context/output

Possible Solution

System details

Windows version: [e.g. 7, 8, 10]

OS architecture: [e.g. 32bit, 64bit]

PowerShell version: [output of "$($PSVersionTable.PSVersion)"]

Additional software: [(optional) e.g. ConEmu, Git]

Scoop Configuration

//# Your configuration here
@hgkamath hgkamath added the bug label Mar 22, 2022
@hgkamath
Copy link
Author

hgkamath commented Mar 22, 2022

perhaps dup of #4798 and #4819

Posting further comments in 4798

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

Successfully merging a pull request may close this issue.

1 participant