Bug 174540

Summary: [GTK] Sort inspector GResource manifest to ensure reproducible builds
Product: WebKit Reporter: Bernhard M. Wiedemann <webkitbmw>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aperez, bugs-noreply, clopez, commit-queue, mcatanzaro, zan
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
fixing patch
none
patch with changelog entry
none
patch with updated changelog entry none

Description Bernhard M. Wiedemann 2017-07-15 01:13:24 PDT
When building the webkitgtk3 package for openSUSE,
it produces random variations in the resulting libwebkitgtk-3.0.so.0.22.17
because GResourceBundle.xml is generated via python's glob.glob function
that returns entries in undeterministic filesystem order.
Comment 1 Bernhard M. Wiedemann 2017-07-15 01:15:32 PDT
Created attachment 315536 [details]
fixing patch
Comment 2 Adrian Perez 2017-07-15 03:35:37 PDT
Comment on attachment 315536 [details]
fixing patch

Informal review: I would r+ this without doubt.

Also: Having reproducible builds is IMHO a noble goal, so we should
help from our side merging patches upstream whenever possible :-)
Comment 3 Adrian Perez 2017-07-15 03:39:15 PDT
Bernhard: I have CC'd a few people to try and get the patch landed. If
you used “webkit-patch upload”, a one nice tip for next time is passing
“--suggest-reviewers --request-commit” to it, which will get reviewers
CC'd automatically, and ask the automated commit-queue to land your patch.

And many thanks for posting the patch!
Comment 4 Michael Catanzaro 2017-07-15 08:07:18 PDT
Thanks. It also needs a ChangeLog entry ('Tools/Scripts/prepare-ChangeLog -b 174540')
Comment 5 Michael Catanzaro 2017-07-15 08:07:59 PDT
Also note that Fedora and Debian have both removed their old/insecure webkitgtk3 packages. Now would be a good time for openSUSE to follow suit.
Comment 6 Bernhard M. Wiedemann 2017-07-15 13:15:10 PDT
Created attachment 315565 [details]
patch with changelog entry
Comment 7 Michael Catanzaro 2017-07-15 15:33:49 PDT
Sorry, I forgot to change the title... please change the title in the ChangeLog, then we can land this.
Comment 8 Bernhard M. Wiedemann 2017-07-15 22:24:29 PDT
Created attachment 315594 [details]
patch with updated changelog entry
Comment 9 WebKit Commit Bot 2017-07-16 05:07:31 PDT
The commit-queue encountered the following flaky tests while processing attachment 315594 [details]:

storage/indexeddb/modern/new-database-after-user-delete.html bug 174354 (author: beidson@apple.com)
The commit-queue is continuing to process your patch.
Comment 10 WebKit Commit Bot 2017-07-16 05:07:32 PDT
The commit-queue encountered the following flaky tests while processing attachment 315594 [details]:

The commit-queue is continuing to process your patch.
Comment 11 WebKit Commit Bot 2017-07-16 05:51:30 PDT
Comment on attachment 315594 [details]
patch with updated changelog entry

Clearing flags on attachment: 315594

Committed r219546: <http://trac.webkit.org/changeset/219546>
Comment 12 WebKit Commit Bot 2017-07-16 05:51:31 PDT
All reviewed patches have been landed.  Closing bug.