Bug 105539 - [GTK] Development builds should be possible without build-webkit
Summary: [GTK] Development builds should be possible without build-webkit
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Zan Dobersek
URL:
Keywords: Gtk
Depends on:
Blocks:
 
Reported: 2012-12-20 08:37 PST by Zan Dobersek
Modified: 2013-01-23 02:06 PST (History)
5 users (show)

See Also:


Attachments
Patch (4.57 KB, patch)
2012-12-21 06:32 PST, Zan Dobersek
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zan Dobersek 2012-12-20 08:37:20 PST
There's still interest in building development builds without using build-webkit. One use case provided by Carlos is using custom jhbuild module.

This was, however, recently made impossible as the feature defines listed in Source/WebCore/GNUmakefile.features.am.in are in process of having their values adjusted to represent values intended for release builds. Development values for feature defines are again tracked by the FeatureList.pm, used solely by build-webkit.

This means that development builds are now only possible by using build-webkit due to the release build values being overridden only when autogen.sh is run by build-webkit (as the latter outputs the development-oriented feature define list that is then used by the Tools/gtk/override-feature-defines, which performs the overriding).
Comment 1 Zan Dobersek 2012-12-20 08:40:29 PST
There are currently at least two manual ways to perform the overriding of desired feature defines' release values:

- creating feature-defines-overriding.txt file (normally created by build-webkit) and listing feature defines that are to be overridden
- modifying the generated (more like copied) GNUmakefile.features.am by adjusting feature define values as desired

But, as said, the listed methods are manual.
Comment 2 Zan Dobersek 2012-12-20 08:48:07 PST
Carlos and I brainstormed about this on IRC, eventually came to an idea of having an extra Perl script that would use the FeatureList.pm to get the development values for feature defines and work with those from there.
Comment 3 Zan Dobersek 2012-12-21 06:32:32 PST
Created attachment 180513 [details]
Patch
Comment 4 Carlos Garcia Campos 2013-01-04 01:26:23 PST
Comment on attachment 180513 [details]
Patch

The problem I see with the patch is that it duplicates the code, ideally we could leave the --enable-unstable-features and call that from build-webkit, but then we could not know whether we need to run autogen again or not. Since I'm probably the only dev affected by this, and I've recently switched to build-webkit, maybe we can just close this bug. What do you think?
Comment 5 Zan Dobersek 2013-01-04 03:16:56 PST
(In reply to comment #4)
> (From update of attachment 180513 [details])
> The problem I see with the patch is that it duplicates the code, ideally we could leave the --enable-unstable-features and call that from build-webkit, but then we could not know whether we need to run autogen again or not. Since I'm probably the only dev affected by this, and I've recently switched to build-webkit, maybe we can just close this bug. What do you think?

I'm OK with closing this bug if it's not a problem for you anymore. If anyone else voices the desire for this functionality we can reopen it and get it polished.
Comment 6 Zan Dobersek 2013-01-23 02:06:25 PST
Closing for now, we can reopen it if anyone wants to be able to use this out-of-the-box.