(jubler) Switch to GitHub release assets and drop 32-bit#2727
Draft
AdmiringWorm wants to merge 2 commits into
Draft
(jubler) Switch to GitHub release assets and drop 32-bit#2727AdmiringWorm wants to merge 2 commits into
AdmiringWorm wants to merge 2 commits into
Conversation
JPRuskin
previously approved these changes
Sep 18, 2025
Member
Author
|
Seems like perhaps the installation of the package is broken. Need to have a look at that at a later date. Avoid merging this PR it has been fixed (anyone seeing this message are free to create a PR despite this one existing). |
Member
|
@AdmiringWorm have you looked at the failures? |
Member
Author
Not had the chance yet, hoping to get to it today. |
Update Jubler metadata and updater script use HTTPS URLs instead of HTTP. Change the release page in automatic/jubler/update.ps1 to https://jubler.org/download.html and update the projectUrl and changelog links in automatic/jubler/jubler.nuspec to secure https://www.jubler.org/... and https://www.jubler.org/changelog.html. This improves security and avoids mixed-content or redirect issues when retrieving release information and package metadata.
Update Jubler autoupdate and installer scripts to use GitHub release assets instead of scraping a downloads page, and remove handling for a missing 32-bit build. - chocolateyinstall.ps1: set 32-bit file to empty and keep x64 file path. This reflects that no 32-bit installer is available. - update.ps1: refactor au_GetLatest to call Get-GitHubRelease and read asset fields (browser_download_url and digest) for version, URL, and checksum. Remove URL32/FileName32 and 32-bit checksum fields and update replacements to use 64-bit checksum/type and file names. - update.ps1: simplify update invocation to call update -ChecksumFor none and remove brittle web scraping and error-catching logic. This makes the update flow more robust, uses authoritative GitHub metadata for checksums and download URLs, and prevents broken updates for a non-existent 32-bit installer.
AdmiringWorm
force-pushed
the
jubler-use-https-urls
branch
from
cdedd30 to
7a79afc
Compare
AdmiringWorm
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Motivation and Context
How Has this Been Tested?
.\update_all.ps1 jublerto ensure package is updated correctly.Screenshot (if appropriate, usually isn't needed):
Types of changes
Checklist: