Closed Bug 1426785 Opened 7 years ago Closed 7 years ago

Stop using the gtk+3 tooltool package

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox59 fixed)

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(5 files)

No description provided.
I should mention that I did a diff between builds with and without this patch queue, and the differences are trivial: - SourceStamp (obviously). - Linked library order in .dynamic sections. - buildconfig.html changes from the mozconfig changes.
Attachment #8938567 - Flags: review?(core-build-config-reviews) → review?(gps)
Attachment #8938568 - Flags: review?(core-build-config-reviews) → review?(gps)
Attachment #8938569 - Flags: review?(core-build-config-reviews) → review?(gps)
Attachment #8938570 - Flags: review?(core-build-config-reviews) → review?(gps)
Attachment #8938571 - Flags: review?(core-build-config-reviews) → review?(gps)
Comment on attachment 8938567 [details] Bug 1426785 - Install gtk+3 in the Centos images used for desktop builds. https://reviewboard.mozilla.org/r/209208/#review215058
Attachment #8938567 - Flags: review?(gps) → review+
Comment on attachment 8938568 [details] Bug 1426785 - Allow more `mach artifact toolchain` calls to not use a tooltool manifest. https://reviewboard.mozilla.org/r/209210/#review215060
Attachment #8938568 - Flags: review?(gps) → review+
Attachment #8938569 - Flags: review?(gps) → review+
Comment on attachment 8938570 [details] Bug 1426785 - Remove uses of the gtk+3 tooltool package. https://reviewboard.mozilla.org/r/209214/#review215064 Niiiiice.
Attachment #8938570 - Flags: review?(gps) → review+
Comment on attachment 8938571 [details] Bug 1426785 - Remove mozconfig.gtk. https://reviewboard.mozilla.org/r/209216/#review215066 Great series! Since this is only touching Mozilla's abilities to use gtk2 in official CI, I'm fine landing this before we officially drop gtk2 support in general. FWIW, my previous attempts at merging the desktop-build Docker image were somewhat foiled by pkg-config and gtk. I'm glad to see this situation greatly simplified. And I'm sure there's more awesomeness to come.
Attachment #8938571 - Flags: review?(gps) → review+
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/2a52bf9e0898 Install gtk+3 in the Centos images used for desktop builds. r=gps https://hg.mozilla.org/integration/autoland/rev/386cd0532519 Allow more `mach artifact toolchain` calls to not use a tooltool manifest. r=gps https://hg.mozilla.org/integration/autoland/rev/851ce8944b41 Use gtk+3 from /usr/local on automation. r=gps https://hg.mozilla.org/integration/autoland/rev/61453b6473f1 Remove uses of the gtk+3 tooltool package. r=gps https://hg.mozilla.org/integration/autoland/rev/08b5850633de Remove mozconfig.gtk. r=gps
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/9452138c4e83 Keep build files in sync (Port bug 1426785: Remove mozconfig.gtk). rs=bustage-fix
Backout by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/2a682edf04ac Backed out changeset 9452138c4e83 since ported bug 1426785 was backed out. rs=bustage-fix
Need some more cleanup around the removed tooltool manifests. I'm going to land directly with those fixups, without re-review because: - This is (mostly) only about removing tooltool_manifest_src entries for the browser/config/tooltool-manifests/linux*/releng.manifest that is removed. - The remainder is a change to _run_tooltool in testing/mozharness/scripts/repackage.py that a) mimicks the code in testing/mozharness/mozharness/mozilla/building/buildbase.py b) don't throw an error if there's nothing to download from tooltool because repacks don't actually necessarily need anything from tooltool (case in point: linux builds now don't ; and in fact they didn't really require that manifest in the first place, cargo curl ftw)
Flags: needinfo?(mh+mozilla)
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/83a7d1921b05 Install gtk+3 in the Centos images used for desktop builds. r=gps https://hg.mozilla.org/integration/autoland/rev/101f3a29dc4f Allow more `mach artifact toolchain` calls to not use a tooltool manifest. r=gps https://hg.mozilla.org/integration/autoland/rev/e9eacb78877a Use gtk+3 from /usr/local on automation. r=gps https://hg.mozilla.org/integration/autoland/rev/83fabeaf6582 Remove uses of the gtk+3 tooltool package. r=gps https://hg.mozilla.org/integration/autoland/rev/68d6c3cb89eb Remove mozconfig.gtk. r=gps
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/13c3ef7d9883 Keep build files in sync (Port bug 1426785: Remove mozconfig.gtk). rs=bustage-fix
Pushed by mozilla@hocat.ca: https://hg.mozilla.org/comm-central/rev/2cf483e70c05 Re-add config to use tooltool gtk for Thunderbird buildbots; r=me
Pushed by mozilla@hocat.ca: https://hg.mozilla.org/comm-central/rev/be410fe9478c Drop tooltool manifests for linux taskcluster builds; r=me
Depends on: 1427232
Product: Core → Firefox Build System
Blocks: 1686177
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: