RESOLVED FIXED 123144
[GTK] enable media-stream in build-webkit 
https://bugs.webkit.org/show_bug.cgi?id=123144
Summary [GTK] enable media-stream in build-webkit
Philippe Normand
Reported 2013-10-22 02:24:34 PDT
SSIA
Attachments
enable mediastream in autofools (6.50 KB, patch)
2013-10-22 02:30 PDT, Philippe Normand 		mrobinson: review-
DetailsFormatted DiffDiff
enable media-stream in GTK build (1.75 KB, patch)
2013-10-28 02:09 PDT, Philippe Normand 		gtk-ews: commit-queue-
DetailsFormatted DiffDiff
enable media-stream in GTK build (2.56 KB, patch)
2013-10-28 03:45 PDT, Philippe Normand 		gtk-ews: commit-queue-
DetailsFormatted DiffDiff
enable media-stream in GTK build (3.29 KB, patch)
2013-10-28 04:07 PDT, Philippe Normand 		mrobinson: review+
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Philippe Normand
Comment 1 2013-10-22 02:30:22 PDT
Created attachment 214826 [details] enable mediastream in autofools
Martin Robinson
Comment 2 2013-10-23 13:07:18 PDT
Comment on attachment 214826 [details] enable mediastream in autofools View in context: https://bugs.webkit.org/attachment.cgi?id=214826&action=review I'm okay with enabling media stream for build-webkit, but let's not add a configure flag. > Source/autotools/ReadCommandLineArguments.m4:133 > +AC_MSG_CHECKING([whether to enable media stream support]) > +AC_ARG_ENABLE(media_stream, > + AC_HELP_STRING([--enable-media-stream], [enable media stream support (incomplete) [default=no]]), > + [], > + [enable_media_stream="no"]) > +AC_MSG_RESULT([$enable_media_stream]) > + I don't think we actually need a configuration flag for this. It's incomplete and we don't typically add flags for web features. The existing flags are exceptions to this rule.
Philippe Normand
Comment 3 2013-10-25 08:29:25 PDT
So how is it supposed to work without the configure option? All this patch relies on it. I only want MEDIA_STREAM on for developer builds...
Martin Robinson
Comment 4 2013-10-25 09:06:21 PDT
(In reply to comment #3) > So how is it supposed to work without the configure option? All this patch relies on it. > > I only want MEDIA_STREAM on for developer builds... If you don't add the setting to %configurableFeatures, the default will be overridden by WebKitBuild/Release/WebKitFeatureOverrides.txt, which is combined with the default options during configuration with the defaults defined in SetupWebKitFeatures.m4.
Philippe Normand
Comment 5 2013-10-28 02:09:28 PDT
Created attachment 215290 [details] enable media-stream in GTK build
kov's GTK+ EWS bot
Comment 6 2013-10-28 03:33:00 PDT
Comment on attachment 215290 [details] enable media-stream in GTK build Attachment 215290 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/15608065
Philippe Normand
Comment 7 2013-10-28 03:45:54 PDT
Created attachment 215293 [details] enable media-stream in GTK build
kov's GTK+ EWS bot
Comment 8 2013-10-28 04:02:02 PDT
Comment on attachment 215293 [details] enable media-stream in GTK build Attachment 215293 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/15568087
Philippe Normand
Comment 9 2013-10-28 04:07:17 PDT
Created attachment 215294 [details] enable media-stream in GTK build
Philippe Normand
Comment 10 2013-10-29 09:32:12 PDT
Committed r158194: <http://trac.webkit.org/changeset/158194>
Note You need to log in before you can comment on or make changes to this bug.