-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Bucket tests will fail unless you specify helper path #3468
Comments
Like so? if ($env:CI_WINDOWS -eq $true) {
$helpersdir = $env:SCOOP_HELPERS, "C:\projects\helpers" | Where-Object { -not [String]::IsNullOrEmpty($_) } | Select-Object -First 1
if(!(Test-Path $helpersdir)) {
New-Item -ItemType Directory -Path $helpersdir
}
if(!(Test-Path "$helpersdir\lessmsi\lessmsi.exe")) {
Start-FileDownload 'https://github.com/activescott/lessmsi/releases/download/v1.6.3/lessmsi-v1.6.3.zip' -FileName "$helpersdir\lessmsi.zip"
& 7z.exe x "$helpersdir\lessmsi.zip" -o"$helpersdir\lessmsi" -y
}
if(!(Test-Path "$helpersdir\innounp\innounp.exe")) {
Start-FileDownload 'https://raw.githubusercontent.com/ScoopInstaller/Binary/master/innounp/innounp048.rar' -FileName "$helpersdir\innounp.rar"
& 7z.exe x "$helpersdir\innounp.rar" -o"$helpersdir\innounp" -y
}
} |
I guess this should work. I am just tired of adding these lines into all buckets Ash258/Shovel-Ash258@788260f I am too lazy to add it into sysinternals bucket too :D |
MCOfficer
added a commit
to ScoopInstaller/Nirsoft
that referenced
this issue
May 20, 2019
Fixed in f52b339 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For bucket tests helpers are not needed at all.
Since f63219f all external buckets test will fail unless you add environment variable for scoop_helpers.
https://ci.appveyor.com/project/Ash258/scoop-ash258/builds/24502846/job/66ps0e8csga27tj5
There should be check if that environment variable exists and if no, just use hardcoded path instead of forcing users to duplicate redundant three lines into own pipeline configuration (As they really do not need them).
The text was updated successfully, but these errors were encountered: