Bug 190581 - [GTK] Disable MSE build support on Debian and Ubuntu LTS bots
Summary: [GTK] Disable MSE build support on Debian and Ubuntu LTS bots
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Philippe Normand
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-10-15 08:33 PDT by Philippe Normand
Modified: 2020-10-27 10:57 PDT (History)
6 users (show)

See Also:


Attachments
Patch (2.01 KB, patch)
2018-10-15 08:34 PDT, Philippe Normand
mcatanzaro: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Normand 2018-10-15 08:33:38 PDT
Their GStreamer version is too old.
Comment 1 Philippe Normand 2018-10-15 08:34:54 PDT
Created attachment 352324 [details]
Patch
Comment 2 Philippe Normand 2018-10-15 08:36:02 PDT
Lucas, when this lands can you please update build.webkit.org ?
Comment 3 Michael Catanzaro 2018-10-15 08:37:03 PDT
Comment on attachment 352324 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=352324&action=review

> Tools/BuildSlaveSupport/build.webkit.org-config/config.json:332
> +                      "additionalArguments": ["--no-experimental-features --no-media-source"],

Actually pretty sure this should be:

["--no-experimental-features", "--no-media-source"]

right? That matches the rest of this file.
Comment 4 Philippe Normand 2018-10-15 08:41:39 PDT
Yes! Thanks for the review!
Comment 5 Philippe Normand 2018-10-15 08:42:47 PDT
Committed r237113: <https://trac.webkit.org/changeset/237113>
Comment 6 Radar WebKit Bug Importer 2018-10-15 08:43:23 PDT
<rdar://problem/45272072>
Comment 7 Ryan Haddad 2018-10-15 21:05:22 PDT
(In reply to Philippe Normand from comment #2)
> Lucas, when this lands can you please update build.webkit.org ?
I just updated build.webkit.org for another change, so this should be live now.
Comment 8 Aakash Jain 2019-01-22 07:11:16 PST
Because of this change an additional buildbot step 'jhbuild' is being run. e.g.: notice 'jhbuild' step in: https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20Debian%20Stable%20%28Build%29/builds/18912


This is because https://trac.webkit.org/browser/webkit/trunk/Tools/BuildSlaveSupport/build.webkit.org-config/factories.py#L42 checks for "additionalArguments != ["--no-experimental-features"]"

This causes unit-test failures: https://bugs.webkit.org/show_bug.cgi?id=193669

Is this step 'jhbuild' expected?

If this step is expected, we should simply update unit-tests.

If this step is not expected, factories.py L42 should be changed to something like:

    if platform == "gtk" and "--no-experimental-features" not in additionalArguments:
Comment 9 Philippe Normand 2019-01-22 08:14:38 PST
(In reply to Aakash Jain from comment #8)
> Because of this change an additional buildbot step 'jhbuild' is being run.
> e.g.: notice 'jhbuild' step in:
> https://build.webkit.org/builders/GTK%20Linux%2064-
> bit%20Release%20Debian%20Stable%20%28Build%29/builds/18912
> 
> 
> This is because
> https://trac.webkit.org/browser/webkit/trunk/Tools/BuildSlaveSupport/build.
> webkit.org-config/factories.py#L42 checks for "additionalArguments !=
> ["--no-experimental-features"]"
> 
> This causes unit-test failures:
> https://bugs.webkit.org/show_bug.cgi?id=193669
> 
> Is this step 'jhbuild' expected?
> 

Hum, I'm not sure this is expected. IIRC the Debian and Ubuntu LTS bots shouldn't rely in jhbuild.
Comment 10 Michael Catanzaro 2019-01-22 08:18:06 PST
Wow, thanks for noticing Aakash. Indeed, these bots should not be running jhbuild as that defeats the purpose of having the bots.

(In reply to Aakash Jain from comment #8) 
>     if platform == "gtk" and "--no-experimental-features" not in
> additionalArguments:

This sounds like the right solution.