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

[email protected]: installation reports "$dir\vcredist_2010_x64.exe" does not exist #298

Open
kkimurak opened this issue Apr 7, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@kkimurak
Copy link

kkimurak commented Apr 7, 2024

Bug Report

Package Name: [name of package which has bug(s)]

[email protected]

Current Behaviour

Installing vokoscreenng-np reports following error (but it was installed successfully. App runs normally as well)

Installing 'vokoscreenng-np' (4.1.0) [64bit] from nonportable bucket
vokoscreenNG-4.1.0-win64.exe (113.0 MB) [=====================================================================] 100%
Checking hash of vokoscreenNG-4.1.0-win64.exe ... ok.
Linking C:\scoop\apps\vokoscreenng-np\current => C:\software\scoop\apps\vokoscreenng-np\4.1.0
Running post_install script...
Remove-Item:
Line |
   3 |  Remove-Item "$dir\setup.exe", "$dir\vcredist_2010_x64.exe"
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path 'C:\scoop\apps\vokoscreenng-np\current\vcredist_2010_x64.exe' because it does not exist.
'vokoscreenng-np' (4.1.0) was installed successfully!

(Replaced error message after setting UI language to en-US)

Expected Behaviour

Reports no error

Additional context/output

VokoscreenNG for windows version 4.1.0 no longer contains vcredist_2010 installer. It have been replaced by vcruntime140.dll and vcruntime140_1.dll. See vkohaupt/vokoscreenNG#310
Current post_install script will try to remove vcredist installer and reports such error.

    "post_install": [
        "if (!(is_admin)) { error \"$app requires admin rights to $cmd\"; break }",
        "Start-Process \"$dir\\setup.exe\" -Wait -Verb 'RunAs' -WindowStyle 'Hidden' -ArgumentList @('in', '-c', '--am', '--al', '-t', \"$dir\")",
        "Remove-Item \"$dir\\setup.exe\", \"$dir\\vcredist_2010_x64.exe\""
    ],

I originally posted #297 but it have been closed automatically because I set a wrong title so that the bot recognized that the problematicVersion is 4.1.0 and does not match to the latest version 4.1.0

Possible Solution

Remove vcredist from post_install removal target.
I'm ready to submit PR if it is OK.

    "post_install": [
        "if (!(is_admin)) { error \"$app requires admin rights to $cmd\"; break }",
        "Start-Process \"$dir\\setup.exe\" -Wait -Verb 'RunAs' -WindowStyle 'Hidden' -ArgumentList @('in', '-c', '--am', '--al', '-t', \"$dir\")",
        "Remove-Item \"$dir\\setup.exe\""
    ],

System details

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

11

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

64bit

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

Major  Minor  Build  Revision
-----  -----  -----  --------
5      1      22621  2506

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

Scoop Configuration

{
  "last_update": "2024-04-08T00:11:12.1991676+09:00",
  "scoop_repo": "https://github.com/lukesampson/scoop",
  "scoop_branch": "master"
}
@kkimurak kkimurak added the bug Something isn't working label Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant