Bug 106178 - [EFL] Add gstreamer 1.0.5 to jhbuild
Summary: [EFL] Add gstreamer 1.0.5 to jhbuild
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
: 103723 (view as bug list)
Depends on: 106552
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-05 07:33 PST by Chris Dumez
Modified: 2013-01-11 08:39 PST (History)
14 users (show)

See Also:


Attachments
Patch (12.48 KB, patch)
2013-01-08 10:25 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (15.28 KB, patch)
2013-01-09 04:09 PST, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2013-01-05 07:33:37 PST
<http://trac.webkit.org/changeset/138786> ported the webaudio backend to gstreamer 1.0.4. Ubuntu 12.04 is shipping gstreamer 0.10.x so we need to add gstreamer 1.0.4 to jhbuild.
Comment 1 Chris Dumez 2013-01-05 10:58:12 PST
Apparently, this is not required so let's fix the regression with gstreamer 0.10 instead.
Comment 2 Chris Dumez 2013-01-08 02:21:49 PST
Reopening since it seems we may need it soon after all, based on the mailing list discussion.
Comment 3 Chris Dumez 2013-01-08 10:25:42 PST
Created attachment 181705 [details]
Patch
Comment 4 Chris Dumez 2013-01-08 10:27:24 PST
We should make sure the following packages are installed on the bot due to avoid Bug 104792:
  libtheora-dev
  libvorbis-dev
  libfaad-dev
Comment 5 Philippe Normand 2013-01-08 10:31:54 PST
Have you checked the media tests? I haven't had time to check the 1.0.5 release yet
Comment 6 Chris Dumez 2013-01-08 10:53:04 PST
Comment on attachment 181705 [details]
Patch

You are right, there are a few regressions in media tests (webaudio tests seem fine though). I'll give gstreamer 1.0.4 a try.
Comment 7 Chris Dumez 2013-01-09 04:09:10 PST
Created attachment 181893 [details]
Patch

Improved patch:
- Skips a few media tests that started crashing with gstreamer 1.0
- Force several gstreamer plugins to be built in jhbuild since they are required by LayoutTests
Comment 8 Laszlo Gombos 2013-01-09 14:31:19 PST
Comment on attachment 181893 [details]
Patch

Looks good to me. Thanks !
Comment 9 Chris Dumez 2013-01-10 00:38:53 PST
Comment on attachment 181893 [details]
Patch

I installed the right packages on the build bots, we should be good to go. I'll carefully watch the bots when it lands though.
Comment 10 WebKit Review Bot 2013-01-10 00:44:05 PST
Comment on attachment 181893 [details]
Patch

Clearing flags on attachment: 181893

Committed r139291: <http://trac.webkit.org/changeset/139291>
Comment 11 WebKit Review Bot 2013-01-10 00:44:10 PST
All reviewed patches have been landed.  Closing bug.
Comment 12 Chris Dumez 2013-01-10 02:55:35 PST
I updated the build dependencies in https://trac.webkit.org/wiki/EFLWebKit

We no longer need "libgstreamer0.10-dev libgstreamer-plugins-base0.10-dev" since it is part of jhbuild but we now need the following packages to build gstreamer 1.0: "libtheora-dev libvorbis-dev libfaad-dev libpulse-dev".
Comment 13 Chris Dumez 2013-01-10 03:29:16 PST
Hmm. Everything went fine on the debug build bot and the release one is getting timeouts for all media tests. I checked the output and it appears that the gstreamer pulse plugin was not built. I'm not sure why yet as I installed libpulse-dev package already.
Comment 14 Chris Dumez 2013-01-10 04:35:40 PST
(In reply to comment #13)
> Hmm. Everything went fine on the debug build bot and the release one is getting timeouts for all media tests. I checked the output and it appears that the gstreamer pulse plugin was not built. I'm not sure why yet as I installed libpulse-dev package already.

- The issue on the WK2 Release build bot was related to pulseaudio being in an error state and not running. We solved that.

- The issue on the WK1 Release build bot is due to missing gstreamer dependencies on the build bot. I asked gyuyoung to take care of it.
Comment 15 Halton Huo 2013-01-11 08:39:12 PST
*** Bug 103723 has been marked as a duplicate of this bug. ***