After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 738581 - Build openh264 and librstp plugins in gst-plugins-bad-1.0{,-static}.recipe
Build openh264 and librstp plugins in gst-plugins-bad-1.0{,-static}.recipe
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: cerbero
git master
Other Linux
: Normal normal
: 1.5.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 738582 (view as bug list)
Depends on: 738579
Blocks:
 
 
Reported: 2014-10-15 14:29 UTC by Nirbheek Chauhan
Modified: 2014-10-21 10:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
New openssl.recipe (2.87 KB, patch)
2014-10-15 14:38 UTC, Nirbheek Chauhan
committed Details | Review
New libsrtp.recipe (1.93 KB, patch)
2014-10-15 14:38 UTC, Nirbheek Chauhan
committed Details | Review
glib.recipe: Include glib-gettext.m4 in devel files (771 bytes, patch)
2014-10-15 14:39 UTC, Nirbheek Chauhan
committed Details | Review
New recipe: vala-m4.recipe (1.86 KB, patch)
2014-10-15 14:40 UTC, Nirbheek Chauhan
committed Details | Review
New recipe: gobject-introspection-m4.recipe (1.68 KB, patch)
2014-10-15 14:41 UTC, Nirbheek Chauhan
committed Details | Review
librsvg: Fix automagic gtk+ support, and add missing gdk-pixbuf dep (4.27 KB, patch)
2014-10-15 14:41 UTC, Nirbheek Chauhan
none Details | Review
New recipe: openh264.recipe (2.34 KB, patch)
2014-10-15 14:42 UTC, Nirbheek Chauhan
committed Details | Review
gst-plugins-bad-1.0: Add srtp and openh264 plugins (9.06 KB, patch)
2014-10-15 14:42 UTC, Nirbheek Chauhan
committed Details | Review
librsvg: Fix automagic gtk+ support, and add missing gdk-pixbuf dep (4.34 KB, patch)
2014-10-16 09:20 UTC, Nirbheek Chauhan
committed Details | Review

Description Nirbheek Chauhan 2014-10-15 14:29:03 UTC
The attached patches add new recipes and fix librsvg's build. List of new recipes:

openssl, libsrtp, openh264, vala-m4, gobject-introspection-m4 (similar to gettext-m4, and for the same reason -- needed to autoreconf librsvg).
Comment 1 Nirbheek Chauhan 2014-10-15 14:31:35 UTC
*** Bug 738582 has been marked as a duplicate of this bug. ***
Comment 2 Nirbheek Chauhan 2014-10-15 14:34:00 UTC
*** Bug 738582 has been marked as a duplicate of this bug. ***
Comment 3 Nirbheek Chauhan 2014-10-15 14:34:00 UTC
*** Bug 738582 has been marked as a duplicate of this bug. ***
Comment 4 Nirbheek Chauhan 2014-10-15 14:38:31 UTC
Created attachment 288595 [details] [review]
New openssl.recipe
Comment 5 Nirbheek Chauhan 2014-10-15 14:38:47 UTC
Created attachment 288596 [details] [review]
New libsrtp.recipe
Comment 6 Nirbheek Chauhan 2014-10-15 14:39:17 UTC
Created attachment 288597 [details] [review]
glib.recipe: Include glib-gettext.m4 in devel files
Comment 7 Nirbheek Chauhan 2014-10-15 14:40:47 UTC
Created attachment 288598 [details] [review]
New recipe: vala-m4.recipe
Comment 8 Nirbheek Chauhan 2014-10-15 14:41:24 UTC
Created attachment 288599 [details] [review]
New recipe: gobject-introspection-m4.recipe
Comment 9 Nirbheek Chauhan 2014-10-15 14:41:45 UTC
Created attachment 288600 [details] [review]
librsvg: Fix automagic gtk+ support, and add missing gdk-pixbuf dep
Comment 10 Nirbheek Chauhan 2014-10-15 14:42:28 UTC
Created attachment 288602 [details] [review]
New recipe: openh264.recipe
Comment 11 Nirbheek Chauhan 2014-10-15 14:42:49 UTC
Created attachment 288603 [details] [review]
gst-plugins-bad-1.0: Add srtp and openh264 plugins
Comment 12 Olivier Crête 2014-10-15 15:03:15 UTC
Just one question.. how is the OpenSSL threading initialized here? Is it the app's responsability ?
Comment 13 Nirbheek Chauhan 2014-10-15 21:13:08 UTC
(In reply to comment #12)
> Just one question.. how is the OpenSSL threading initialized here? Is it the
> app's responsability ?

The build leaves the threading support in openssl to default, which is that it's enabled. What that actually implies for an application using it, in this case libsrtp, is upto the srtp plugin. :)
Comment 14 Thibault Saunier 2014-10-15 22:30:11 UTC
Review of attachment 288600 [details] [review]:

::: recipes/librsvg.recipe
@@ +18,3 @@
                    'lib/librsvg-2.a', 'lib/librsvg-2.la']
     files_typelibs = ['Rsvg-2.0']
+    configure_options = ' --disable-pixbuf-loader --without-gtk3 '

I think you should check if the gtk3 variant is enabled first no?
Comment 15 Nirbheek Chauhan 2014-10-16 09:20:12 UTC
Created attachment 288656 [details] [review]
librsvg: Fix automagic gtk+ support, and add missing gdk-pixbuf dep

(In reply to comment #14)
> Review of attachment 288600 [details] [review]:
> 
> ::: recipes/librsvg.recipe
> @@ +18,3 @@
>                     'lib/librsvg-2.a', 'lib/librsvg-2.la']
>      files_typelibs = ['Rsvg-2.0']
> +    configure_options = ' --disable-pixbuf-loader --without-gtk3 '
> 
> I think you should check if the gtk3 variant is enabled first no?

--with-gtk3 just enables building of the `rsvg-view-3` binary, which is mostly useless for cerbero, so I disabled it to avoid a gtk3 dependency while building gst-plugins-bad. 

The attached patch adds a comment to that effect.