-
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
chromium: fix #3476
chromium: fix #3476
Conversation
"version": "79.0.3945.117-1", | ||
"description": "Modifications to Google Chromium for removing Google integration and enhancing privacy, control, and transparency.", | ||
"version": "79.0.3945.117", | ||
"description": "Modifications to Google Chromium for removing Google integration and enhancing privacy, control, and transparency (macchrome's build)", | ||
"homepage": "https://github.com/Eloston/ungoogled-chromium", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we also change the homepage to "https://github.com/macchrome/winchrome"?
"url": "https://github.com/macchrome/winchrome/releases/download/v$version-r$matchRev-Win64/ungoogled-chromium-$version-1_windows.7z", | ||
"extract_dir": "ungoogled-chromium-$version-1_windows" | ||
}, | ||
"32bit": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems the newest version of ungoogled-chromium has merged 32-bit version and 64-bit version.
Take this for example: https://github.com/macchrome/winchrome/releases/tag/v80.0.3987.132-r722274-Win64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess the dev stop built 32 bit version for some reason.
}, | ||
"hash": { | ||
"url": "https://github.com/macchrome/winchrome/releases/tag/v$version-r$matchRev-Win64", | ||
"regex": "(?s)$basename.*?$sha1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because the merge of 32-bit version and 64-bit version, the hash autoupdate will fail for 32-bit version.
close #3473
close #3415
#2308