WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
105539
[GTK] Development builds should be possible without build-webkit
https://bugs.webkit.org/show_bug.cgi?id=105539
Summary
[GTK] Development builds should be possible without build-webkit
Zan Dobersek
Reported
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).
Attachments
Patch
(4.57 KB, patch)
2012-12-21 06:32 PST
,
Zan Dobersek
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Zan Dobersek
Comment 1
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.
Zan Dobersek
Comment 2
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.
Zan Dobersek
Comment 3
2012-12-21 06:32:32 PST
Created
attachment 180513
[details]
Patch
Carlos Garcia Campos
Comment 4
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?
Zan Dobersek
Comment 5
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.
Zan Dobersek
Comment 6
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.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug