Skip to content
This repository has been archived by the owner on Oct 25, 2022. It is now read-only.

Checksum verification mode doesn't work with updateall #36

Closed
majkinetor opened this issue Oct 11, 2016 · 5 comments
Closed

Checksum verification mode doesn't work with updateall #36

majkinetor opened this issue Oct 11, 2016 · 5 comments
Labels

Comments

@majkinetor
Copy link
Owner

majkinetor commented Oct 11, 2016

It just happens that updateall doesn't finish the package, it isn't shown in log at all or some other package doubles.

When update.ps1 is called directly this doesn't happen.

Test for this doesn't exist.

-------------------------------------------------------------------------------------------------------------------------------------------------------------------------
13  16-10-11 22.47.39  C:\work\_\chocolatey-coreteampackages\automatic
> updateall y*
Updating 1 automatic packages at 2016-10-11 22:47:56
Push is disabled
    has no updates

@majkinetor
Copy link
Owner Author

majkinetor commented Oct 14, 2016

Happend with chromium too on core packages, here is the temp log (the log was the same with youtube-dl)

chromium - checking updates using au version 2016.9.25
nuspec version: 56.0.2890.0
remote version: 56.0.2891.0
New version is available
Automatic checksum started
Downloading chromium 32 bit
  from 'https://storage.googleapis.com/chromium-browser-snapshots/Win/425259/mini_installer.exe'

Download of mini_installer.exe (35.36 MB) completed.

@majkinetor majkinetor added the bug label Oct 19, 2016
@majkinetor
Copy link
Owner Author

majkinetor commented Oct 19, 2016

Turns out that this simple thing doesn't work

Start-Job -Name youtube {
    $global:au_Force   = $true
    cd c:\work\_\chocolatey-coreteampackages\automatic\youtube-dl
    ./update.ps1
}

within the job but outside it works ....

> rcjb -name youtube
youtube-dl - checking updates using au version 2016.10.19.122047
URL check
  https://github.com/rg3/youtube-dl/releases/download/2016.10.19/youtube-dl.exe
nuspec version: 2016.10.19.20161019
remote version: 2016.10.19
No new version found, but update is forced
New version is available
Automatic checksum started
File appears to be downloaded already. Verifying with package checksum to determine if it needs to be redownloaded.

@majkinetor
Copy link
Owner Author

Alternative output

youtube-dl - checking updates using au version 2016.10.19.122047
URL check
  https://github.com/rg3/youtube-dl/releases/download/2016.10.16/youtube-dl.exe
nuspec version: 2016.10.16
remote version: 2016.10.16
No new version found, but update is forced
New version is available
Automatic checksum started
Downloading youtube-dl 32 bit
  from 'https://github.com/rg3/youtube-dl/releases/download/2016.10.16/youtube-dl.exe'

Download of youtube-dl.exe (7,19 MB) completed.

@majkinetor
Copy link
Owner Author

Turns out that it works when I remove:

Get-ChecksumValid -File $fileFullPath -Checksum $checksum -ChecksumType $checksumType -OriginalUrl $url

from Get-ChocolateyWebFile.

@RedBaron2
Copy link
Contributor

Get-ChocolateyWebFile is kind of a necessity of Chocolatey, right? It must be something else I would think.

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

No branches or pull requests

2 participants