New: gst-plugins-good: Allow soup plugin to be configured
https://bugs.busybox.net/show_bug.cgi?id=383
Summary: gst-plugins-good: Allow soup plugin to be configured
Product: buildroot
Version: unspecified
Platform: PC
OS/Version: Linux
Status: NEW
Severity: enhancement
Priority: P5
Component: Other
AssignedTo: unassigned at buildroot.uclibc.org
ReportedBy: will.newton at gmail.com
CC: buildroot at uclibc.org
Estimated Hours: 0.0
Created an attachment (id=383)
--> (https://bugs.busybox.net/attachment.cgi?id=383)
Allow soup plugin to be configured
This patch adds support for configuring the soup plugin as part of gst-plugins-good. The soup plugin uses the libsoup library to access http streams.
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
https://bugs.busybox.net/show_bug.cgi?id=383
--- Comment #1 from Markus Heidelberg <[email protected]> 2009-06-09 18:58:56 UTC --- We use the exact name of the plugin in Config.in, the name which is used on the command line with gst-launch for example. It is written in the configure help of the gst package. So "souphttpsrc" instead of "soup" and BR2_..._SOUPHTTPSRC.
Also, the short description should be a bit more descriptive. Why not just take it from configure: soup http client plugin (2.4)
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
--- Comment #2 from Will Newton <[email protected]> 2009-06-10 09:10:03 UTC ---
I tried to follow the existing conventions in the file. Could you point me to an example usage of the convention you are talking about?
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
--- Comment #3 from Markus Heidelberg <[email protected]> 2009-06-11 17:03:14 UTC --- (In reply to comment #2)
Quoted 2 lines
I tried to follow the existing conventions in the file. Could you point me to an example usage of the convention you are talking about?
All the other config options for plugins?
And without knowing what libsoup is all about, I only think on something to eat. The words "http client" at least gives some information.
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
--- Comment #4 from Will Newton <[email protected]> 2009-06-11 17:29:17 UTC --- (In reply to comment #3)
Quoted 7 lines
(In reply to comment #2)
Quoted 2 lines
I tried to follow the existing conventions in the file. Could you point me to an example usage of the convention you are talking about?
All the other config options for plugins?
And without knowing what libsoup is all about, I only think on something to eat. The words "http client" at least gives some information.
Ok, for example:
config BR2_PACKAGE_GST_PLUGINS_GOOD_PLUGIN_FLAC
bool "flac (libFLAC)"
select BR2_PACKAGE_FLAC
The names of the plugins are "flacenc" and "flacdec", and are not mentioned in the config. In parentheses we have "libFLAC" which appears to be telling us what the dependency is. No other plugins are described, e.g. matroska is a container format but this is not explained in the config.
The name of the config variable seems to me to correspond to the name of the plugin/configure option in gstreamer rather than the name of the element. I could change the patch, it's not a problem, but I do think it would look a little weird the way things are laid out at the moment.
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
--- Comment #5 from Markus Heidelberg <[email protected]> 2009-06-11 18:16:22 UTC --- (In reply to comment #4)
Quoted 9 lines
(In reply to comment #3)
Quoted 7 lines
(In reply to comment #2)
Quoted 2 lines
I tried to follow the existing conventions in the file. Could you point me to an example usage of the convention you are talking about?
All the other config options for plugins?
And without knowing what libsoup is all about, I only think on something to eat. The words "http client" at least gives some information.
Quoted 8 lines
Ok, for example:
config
BR2_PACKAGE_GST_PLUGINS_GOOD_PLUGIN_FLAC
bool "flac (libFLAC)"
selectBR2_PACKAGE_FLACThe names of the plugins are "flacenc" and "flacdec", and are not mentioned in the config.
Yes, I noticed it myself after sending the response. A config option doesn't necessarily correspond to only 1 plugin. But it's still valid what I said earlier ("It is written in the configure help of the gst package."), I think we should stick to this.
Note that I speak about all gst-plugins-{base,good,ugly} packages.
Quoted 2 lines
In parentheses we have "libFLAC" which appears to be telling us what the dependency is.
libFLAC is the only config option where just the dependency is written in the description, I would have preferred "FLAC lossless audio" as in the configure script help from the gst-plugins-good package.
Quoted 2 lines
No other plugins are described, e.g. matroska is a container format but this is not explained in the config.
The plugins with external dependencies all have a little description, if it's not obviously clear from the name. The matroska plugin and all the other plugins without description are dependency-less plugins. When I initially wrote support for the gstreamer/gst-plugins-* packages, I had to be able to disable most of the dependency-less plugins, because I didn't need them (and didn't have much space on the flash) but they were all compiled by default. So I just listed them so that I was able to disable them. You may notice, some of them have a little description anyway - these were the ones I needed and did enable. The other ones I mostly didn't have a clue what they were about, neither did I care about - and the configure help of the particular gst* package doesn't have a description, so I couldn't easily add a little description for them.
So if someone now adds support for a new plugin, he definetely knows what it is about and should include a little description.
Quoted 2 lines
The name of the config variable seems to me to correspond to the name of the plugin/configure option in gstreamer rather than the name of the element.
The name of the element was wrong as said above, sorry. But they actually correspond to the name of the configure help (the word after the colon), which mostly is equivalent to the --disable/enable-option. But for example it is --diable-oss, but ossaudio after the colon.
Quoted 3 lines
I
could change the patch, it's not a problem, but I do think it would look a little weird the way things are laid out at the moment.
I don't think "souphttpsrc" would look weird. And it would be consistent with the current style. Of course just using "soup" also wouldn't be really inconsistent.
I know, my comments can be petty, but I'd like to keep things clean and consistent. This is even more true, since the gstreamer support came from me initially.
I hope you are more happy to receive such comments than none at all.
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
--- Comment #6 from Will Newton <[email protected]> 2009-06-11 19:16:58 UTC --- Created an attachment (id=389)
--> (https://bugs.busybox.net/attachment.cgi?id=389)
Allow souphttpsrc support to be configured
Hopefully this patch takes into account your concerns. Thanks for the feedback.
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
Will Newton <[email protected]> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #383 is|0 |1
obsolete| |
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
--- Comment #7 from Markus Heidelberg <[email protected]> 2009-06-11 19:22:33 UTC --- (In reply to comment #6)
Quoted 1 line
Created an attachment (id=389)
--> (https://bugs.busybox.net/attachment.cgi?id=389) [details]
Quoted 4 lines
Allow souphttpsrc support to be configured
Hopefully this patch takes into account your concerns. Thanks for the feedback.
It's OK. Again, it's not my intention to be annoying. Thanks for the patience :)
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
--- Comment #8 from Sven Neumann <[email protected]> 2009-06-12 15:09:19 UTC --- Patch looks good to me.
Perhaps also bump the version to 0.10.15 while we are on it. I have just tested this update and it seems to be unproblematic.
Signature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
[Buildroot] [Bug 383] gst-plugins-good: Allow soup plugin to be configured
In reply to bugzilla at busybox.nethttps://bugs.busybox.net/show_bug.cgi?id=383
Will Newton <[email protected]> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXEDSignature · 3 lines
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.