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

vapoursynth@>=R66: Fix autoupdate url #5864

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BinToss
Copy link

@BinToss BinToss commented May 23, 2024

Changes expected file extensions from .7z (used until VapourSynth R65) to .zip (R66-RC1 and later)

This does not update the package itself. I'll let the bot do that.

Closes #5863


 ScoopInstaller.Main  .\bin\checkver.ps1 vapoursynth
vapoursynth: 68 (scoop version is 65) autoupdate available
 ScoopInstaller.Main  .\bin\checkver.ps1 vapoursynth -u
vapoursynth: 68 (scoop version is 65) autoupdate available
Autoupdating vapoursynth
Downloading VapourSynth64-Portable-R68.zip to compute hashes!
VapourSynth64-Portable-R68.zip (8.5 MB) [=====================================================================================================] 100%
Computed hash: 911d22e6c52886b043194bc17f232f4e054f9e76259c72b66dfc679e7409304b
Downloading VapourSynth64-Portable-R68.zip to compute hashes!
Loading VapourSynth64-Portable-R68.zip from cache
Computed hash: 911d22e6c52886b043194bc17f232f4e054f9e76259c72b66dfc679e7409304b
Writing updated vapoursynth manifest

vapoursynth.json

{
+    "version": "68",
    "description": "A video processing framework with simplicity in mind",
    "homepage": "https://www.vapoursynth.com",
    "suggest": {
        "vcredist": "extras/vcredist2022"
    },
    "license": "LGPL-2.1-or-later",
    "architecture": {
        "64bit": {
+           "url": "https://github.com/vapoursynth/vapoursynth/releases/download/R68/VapourSynth64-Portable-R68.zip",
+           "hash": "911d22e6c52886b043194bc17f232f4e054f9e76259c72b66dfc679e7409304b"
        },
        "32bit": {
+           "url": "https://github.com/vapoursynth/vapoursynth/releases/download/R68/VapourSynth64-Portable-R68.zip",
+           "hash": "911d22e6c52886b043194bc17f232f4e054f9e76259c72b66dfc679e7409304b"
        }
    },
    "pre_install": [
        "$arch = $architecture.SubString(0,2)",
        "if(Test-Path \"$persist_dir\\vapoursynth$arch\") {Copy-Item \"$persist_dir\\*\" \"$dir\\\" -Force -Recurse}"
    ],
    "uninstaller": {
        "script": [
            "$arch = $architecture.SubString(0,2)",
            "ensure \"$persist_dir\" | Out-Null",
            "Copy-Item \"$dir\\vapoursynth$arch\" \"$persist_dir\\\" -Force -Recurse"
        ]
    },
    "bin": [
        "AVFS.exe",
        "VSPipe.exe",
        "pfm-192-vapoursynth-win.exe",
        "vsrepo.py"
    ],
    "checkver": {
        "github": "https://github.com/vapoursynth/vapoursynth",
        "regex": "tag/R(\\d+)"
    },
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://github.com/vapoursynth/vapoursynth/releases/download/R$version/VapourSynth64-Portable-R$version.zip"
            },
            "32bit": {
                "url": "https://github.com/vapoursynth/vapoursynth/releases/download/R$version/VapourSynth64-Portable-R$version.zip"
            }
        }
    }
}

Changes expected file extensions from .7z (used until VapourSynth R65) to .zip (R66-RC1 and later)
@BinToss
Copy link
Author

BinToss commented May 23, 2024

/verify Is everything good to merge?

Copy link
Contributor

Your changes do not pass checks.

vapoursynth

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

1 similar comment
Copy link
Contributor

Your changes do not pass checks.

vapoursynth

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@BinToss
Copy link
Author

BinToss commented May 23, 2024

/verify

I did not know I also needed to commit the checkver-updated manifest.

Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

vapoursynth

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

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

Successfully merging this pull request may close these issues.

[Bug]: Vapoursynth autoupdate broke; releases switched from .7z to .zip
1 participant