Bug 168321 - [GTK] Add two new buildbots that build-only test on Debian stable and Ubuntu LTS
Summary: [GTK] Add two new buildbots that build-only test on Debian stable and Ubuntu LTS
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: Carlos Alberto Lopez Perez
URL:
Keywords:
Depends on: 168323
Blocks:
  Show dependency treegraph
 
Reported: 2017-02-14 08:15 PST by Carlos Alberto Lopez Perez
Modified: 2017-02-16 09:28 PST (History)
9 users (show)

See Also:


Attachments
Patch (6.12 KB, patch)
2017-02-15 21:32 PST, Carlos Alberto Lopez Perez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2017-02-14 08:15:51 PST
One of the problems we currently have is that we are using ancient versions of libraries in our JHBuild, and old versions of GCC on the bots.

One of the reasons we do this is to unintentionally avoid breaking the build on systems that require using this old versions of the libraries or the compiler.

However, a majority of our users are using distributions that ship the last version of this libraries or the compiler.

So we end testing WebKit with a different version of the dependencies than what the users end running. Not ideal.


The proposed solution is:

1) Add two new buildbots that buid-test WebKitGTK+ on the last Debian stable and Ubuntu LTS without using the internal JHBuild. This bots will build against the system libraries. This will ensure we don't unintentionally break our policy about dependency versions https://trac.webkit.org/wiki/WebKitGTK/DependenciesPolicy

2) Then we will have complete freedom to raise the dependencies on the JHBuild without fearing unintentionally build breakage on older systems. Ideally we would like to keep the versions of the libraries on our jhbuild synced with the last GNOME modulesets. That way we can test WebKit with the version of libraries that most users will end using.
Comment 1 Carlos Alberto Lopez Perez 2017-02-15 21:32:13 PST
Created attachment 301703 [details]
Patch
Comment 2 Carlos Alberto Lopez Perez 2017-02-16 04:18:11 PST
Comment on attachment 301703 [details]
Patch

Clearing flags on attachment: 301703

Committed r212429: <http://trac.webkit.org/changeset/212429>
Comment 3 Carlos Alberto Lopez Perez 2017-02-16 04:18:20 PST
All reviewed patches have been landed.  Closing bug.
Comment 4 Konstantin Tokarev 2017-02-16 04:39:10 PST
IMHO name of "--default-cmake-features" argument does not imply it is disabling JHBuild.
Comment 5 Carlos Alberto Lopez Perez 2017-02-16 05:41:03 PST
(In reply to comment #4)
> IMHO name of "--default-cmake-features" argument does not imply it is
> disabling JHBuild.

You can build webkit with the jhbuild and --default-cmake-features, or you can build webkit without jhbuild and --default-cmake-features.

All possible combinations are supported.

But we want this bots to run without jhbuild, so we can be sure webkit remains build-able on the base configuration using only the distribution dependencies.

Checking for this argument on the buildbot config *and* for the gtk port on the buildbot config in order to decide if disable the jhbuild step is not ideal.

But i don't think there is ideal solution here.

 
If we add a explicit optional argument to disable jhbuild, then we have to take care of filtering that argument on the build step, as otherwise it would be passed to the build... causing the step to fail.
Comment 6 Carlos Alberto Lopez Perez 2017-02-16 09:28:47 PST
Committed r212437: <http://trac.webkit.org/changeset/212437>