Summary: | [Flatpak] Second run update-webkitwpe-flatpak fails | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adrian Perez <aperez> | ||||
Component: | Tools / Tests | Assignee: | Philippe Normand <pnormand> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cgarcia, commit-queue, lforschler, mcatanzaro, pnormand, tsaunier, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | Other | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 191196 | ||||||
Attachments: |
|
Description
Adrian Perez
2018-12-14 09:14:32 PST
The error is different now, the issue is that we try to update the runtime and sdk to a commit that no longer exists in the flathub repo. Another issue, the install/update logic is flawed, the sdk/runtime are installed, the latest version published on flathub are installed, but i think we should instead install the versions specified by hashes from the manifest. Currently the hashes are used only for the update workflow, which is prone to errors because the flathub admins seem to periodically remove old commits from their OStrees. Today's hashes: diff --git a/Tools/flatpak/org.webkit.WebKit.yaml b/Tools/flatpak/org.webkit.WebKit.yaml index fe5b008f4be..167e43bc670 100644 --- a/Tools/flatpak/org.webkit.WebKit.yaml +++ b/Tools/flatpak/org.webkit.WebKit.yaml @@ -2,8 +2,8 @@ app-id: %(PORTNAME)s runtime: org.gnome.Platform runtime-version: "3.28" # Control the exact version of the Sdk/Runtime that is being used. -sdk-hash: ba3017972f1cc6c7c8b0ed7f80d3297ad4c6f4076bd799d0bab0e2cbaa3e3c0a -runtime-hash: 9d3fe926d425ad837e40a0945020a00f09c1f531335987a753e07f276d76ee36 +sdk-hash: 31af0c10f8e42151220a728b52df0a5c37ea8e4b033825f40c30e44d11cb31d1 +runtime-hash: 6d1d0ebbd72404c61d109307eb2240542b7ad82608bc6428bba6f3eebcfc8bf3 sdk: org.gnome.Sdk command: %(COMMAND)s finish-args: I would remove the hashes, to be honest. Unless we can have guarantees the server admins that old commits are kept. Any thoughts? (In reply to Philippe Normand from comment #2) > Another issue, the install/update logic is flawed, the sdk/runtime are > installed, the latest version published on flathub are installed, but i > think we should instead install the versions specified by hashes from the > manifest. Currently the hashes are used only for the update workflow Ow, that's bad. :/ (In reply to Philippe Normand from comment #4) > I would remove the hashes, to be honest. Unless we can have guarantees the > server admins that old commits are kept. > > Any thoughts? We do have such guarantees. Alex has promised to keep old commits for some period of time (at least two months; I think I even heard six months? I don't remember the exact time guarantee). But if we leave old hashes there indefinitely, like we've been doing, then yes they will eventually break. Like anything, they have to be maintained. If you remove the hashes, then we'll all have different development environments and won't be able to have reproducible tests, so we'll never be able to get rid of the jhbuild. (In reply to Michael Catanzaro from comment #5) > If you remove the hashes, then we'll all have different development > environments and won't be able to have reproducible tests, so we'll never be > able to get rid of the jhbuild. The install/update logic needs to be reversed then, as explained in comment 2 Created attachment 366924 [details]
Patch
Comment on attachment 366924 [details] Patch Clearing flags on attachment: 366924 Committed r243999: <https://trac.webkit.org/changeset/243999> All reviewed patches have been landed. Closing bug. |