Bug 128163

Summary: [GTK] [Stable] Build unittests only when requested
Product: WebKit Reporter: Pacho Ramos <pachoramos1>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: berto, bugs-noreply, cgarcia, gnome, gustavo, zan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
webkit-gtk-2.2.4-unittests-build.patch none

Description Pacho Ramos 2014-02-03 23:08:22 PST
Created attachment 223074 [details]
webkit-gtk-2.2.4-unittests-build.patch

We are applying this patch on Gentoo to build unittests only when going to run tests

Thanks for applying it
Comment 1 Alberto Garcia 2014-02-05 05:41:13 PST
Hey, thanks for the patch.

The problem is that we're switching to cmake soon so this will be
obsolete in a few weeks.

We run the tests all the time and changing the workflow of the
developers (even if it's such a small change) shortly before the
switch is probably not a good idea.

This would also need changes in the build-webkit script else the build
bots will probably break.
Comment 2 Carlos Garcia Campos 2014-02-05 05:49:43 PST
We are not going to switch to cmake in the stable branches.
Comment 3 Pacho Ramos 2014-02-05 13:19:02 PST
Maybe would be interesting to apply it to 2.2.x branch only then (if there will be more releases)
Comment 4 Alberto Garcia 2014-02-06 01:25:34 PST
I wouldn't change that at this point for the reasons I already said, but I don't have strong arguments against fixing it in the stable branches either.
Comment 5 Gustavo Noronha (kov) 2014-02-11 03:42:49 PST
Not opposed, but this is only useful for the 2.2.x stable branch right? In 2.4 we have developer mode.