On Fri, Nov 15, 2013 at 09:25:10AM +0530, Siddharth Heroor wrote:
> On 11/14/2013 7:44 PM, Maupin, Chase wrote:
> >>-----Original Message-----
> >>From: Heroor, Siddharth
> >>Sent: Thursday, November 14, 2013 12:07 AM
> >>To: Maupin, Chase; Hingolikar, Mrinmayee; Dmytriyenko, Denys
> >>Cc: [email protected]; Cooper Jr., Franklin
> >>Subject: Re: [meta-arago] [PATCH 0/5] GStreamer multimedia
> >>framework and plugins recipes
> >>
> >>On 11/11/2013 8:05 PM, Maupin, Chase wrote:
> >>>Mrinmayee,
> >>>
> >>>Some things that would help would be if you could give us an
> >>idea about the following:
> >>>1. What are these patches adding?  Are they adding new
> >>components?
> >>As I mentioned these patches are a result of the GLP work on
> >>OMAP4.
> >>>2. If new components are being added why not create a separate
> >>project for them so that it is easier to work on moving them along
> >>with different releases of gstreamer?  Basically split out the
> >>"base" gstreamer and the new elements.  This way if another group
> >>wants to look at supporting a different gstreamer version your
> >>changes are isolated from the gstreamer base.
> >>
> >>>3. For patches against gstreamer elements what is the
> >>plan/schedule for pushing those patches up to the gstreamer
> >>mainline to minimize what is being carried?
> >>>This basically boils down to what is the roadmap tomorrow to
> >>being ready to adapt to new gstreamer releases and reduce this
> >>patchset?  What is the plan for handling migrations and is a
> >>consolidated code base the easiest way to do this?
> >>Interestingly, as an example, the original work on gst-wayland was
> >>part
> >>of a separate plugin - gst-ducati. As part of the GLP work, this
> >>was
> >>moved to gst-plugins-ugly. Upstream GStreamer have a proposal for
> >>a
> >>Wayland Sink that we will need to move to in the future. So, the
> >>way I
> >>see it, these are specific to current work and not a roadmap.
> >>
> >>The choices are either we carry the patches on glsdk staging tree
> >>or we
> >>host them on meta-ti as a separate recipe because I see that we
> >>will
> >>need to move to GStreamer 1.x in the future regardless. Either
> >>way, we
> >>are carrying patches that we inherited and now are getting flack
> >>for
> >>someone else not upstreaming them 2 years ago when we weren't even
> >>involved. :-(

> >Not trying to give you flack.  Just trying to understand what was going on 
> >and if there was a better way to separate this code to make it more 
> >portable.  Knowing there is a plan to get to 1.x is a good thing and helps 
> >us in making a recommendation on how to proceed.  I think we can work with 
> >these recipes, but they need to be modified to not apply to all devices but 
> >instead limit them to your device.
> 
> Right. Let's limit the recipes to OMAP5 and J6 but is it possible
> for it to be built and tested in the Core SDK context? Or do you
> recommend that we use -glsdk.bb for the recipe and give a preferred
> provider in the branding file?

There are 2 options:

1. create separate -glsdk versions of the recipe (gstreamer and all plugins) 
and then in the arago.conf point preferred provider to those recipes for omap5 
and j6 platforms (or entire omap-a15 SOC family)

2. re-use the same recipes (or rather bbappends, since we don't carry complete 
recipes in meta-arago) and then machine-override necessary variables, like 
SRC_URI to point to your trees, but only for specific machines. E.g. 
SRC_URI_omap-a15 = "git://omapzoom.org..." or SRC_URI_append_omap-a15 = 
"file://some.patch"

The first one is cleaner and easier to maintain, but also easier to diverge or 
get stale...

-- 
Denys


> >>>>-----Original Message-----
> >>>>From: Hingolikar, Mrinmayee
> >>>>Sent: Monday, November 11, 2013 12:16 AM
> >>>>To: Dmytriyenko, Denys; Maupin, Chase
> >>>>Cc: Cooper Jr., Franklin; [email protected]
> >>>>Subject: RE: [meta-arago] [PATCH 0/5] GStreamer multimedia
> >>>>framework and plugins recipes
> >>>>
> >>>>I was on a long vacation, so I apologize for the delay in
> >>>>response.
> >>>>Here are the no. of patches on top of standard GStreamer.
> >>>>
> >>>>Gstreamer - 15
> >>>>Gstreamer base plugins - 56
> >>>>Gstreamer good plugins - 11
> >>>>Gstreamer bad plugins - 195
> >>>>
> >>>>Based on Franklin's suggestions and discussions with Sid, we
> >>have
> >>>>decided to create new recipes with unique identifiers, for eg.
> >>>>gstreamer-glsdk.
> >>>>Kindly let me know if this approach is fine with you.
> >>>>
> >>>>Regards,
> >>>>Mrinmayee
> >>>>
> >>>>>-----Original Message-----
> >>>>>From: Dmytriyenko, Denys
> >>>>>Sent: Thursday, October 31, 2013 4:27 AM
> >>>>>To: Maupin, Chase
> >>>>>Cc: Cooper Jr., Franklin; Hingolikar, Mrinmayee; meta-
> >>>>arago@arago-
> >>>>>project.org
> >>>>>Subject: Re: [meta-arago] [PATCH 0/5] GStreamer multimedia
> >>>>framework
> >>>>>and plugins recipes
> >>>>>
> >>>>>Mrinmayee,
> >>>>>
> >>>>>Myself, Franklin and Chase have asked you to estimate the
> >>amount
> >>>>and
> >>>>>size of the patches that you have in your gstreamer trees. And
> >>>>you
> >>>>>still have not answered that.
> >>>>>
> >>>>>The reason we are asking this simple question is because the
> >>>>changes
> >>>>>you are making to the recipes will affect others and cannot be
> >>>>accepted
> >>>>>as is. You would need to use machine override VAR_omap-a15
> >>>>instead of
> >>>>>COMPATIBLE_MACHINE and depending on the amount and size of the
> >>>>patches,
> >>>>>you either simply apply them just for your machine, or change
> >>>>the
> >>>>>SRC_URI completely for your machine.
> >>>>>
> >>>>>Either way, I would like to encourage you to participate in
> >>the
> >>>>>discussion and not just disappear. Thank you.
> >>>>>
> >>>>>--
> >>>>>Denys
> >>>>>
> >>>>>
> >>>>>On Fri, Oct 25, 2013 at 12:08:48PM -0400, Maupin, Chase wrote:
> >>>>>>>-----Original Message-----
> >>>>>>>From: [email protected] [mailto:meta-
> >>>>arago-
> >>>>>>>[email protected]] On Behalf Of Cooper Jr., Franklin
> >>>>>>>Sent: Friday, October 25, 2013 10:27 AM
> >>>>>>>To: Dmytriyenko, Denys; Hingolikar, Mrinmayee
> >>>>>>>Cc: [email protected]
> >>>>>>>Subject: Re: [meta-arago] [PATCH 0/5] GStreamer multimedia
> >>>>>>>framework and plugins recipes
> >>>>>>>
> >>>>>>>All of your appends are currently blocking all of the
> >>>>gstreamer
> >>>>>>>plugins for all machines other than omap-a15. There is no
> >>way
> >>>>to
> >>>>>>>only apply a bbappend for a specific machine and have all
> >>>>other
> >>>>>>>machines ignore the bbappend.
> >>>>>>>
> >>>>>>>Depending on your patch set on top of the standard gstreamer
> >>>>>>>sources it probably would be better to break out the patches
> >>>>from
> >>>>>>>your git repo and add it to the SRC_URI for only the omap
> >>>>machine.
> >>>>>>This may be preferred just to help keep from getting locked
> >>to
> >>>>>particular
> >>>>>>gstreamer version and make clear the changeds on top.  But
> >>>>number of
> >>>>>patches
> >>>>>>is something that would weigh in here.
> >>>>>>
> >>>>>>>Another approach would be to create a "new "recipe called
> >>>>omap-
> >>>>>>><base recipe name> that "requires/includes" the base
> >>>>gstreamer
> >>>>>>>recipe and then you can override whatever you want. Take a
> >>>>look at
> >>>>>>>the qt4-embedded-gles_4.8.4.bb recipe.
> >>>>>>>http://arago-project.org/git/?p=meta-
> >>arago.git;a=blob;f=meta-
> >>>>>>>arago-extras/recipes-qt/qt4/qt4-embedded-gles_4.8.4.bb
> >>>>>>>
> >>>>>>>Or
> >>>>>>>Create new recipes with some unique identifier to indicate
> >>>>this
> >>>>>>>isn't the standard gstreamer.
> >>>>>>If not patches one of the above seems right.
> >>>>>>
> >>>>>>>Others may have other approaches that they recommend.
> >>>>>>>
> >>>>>>>>-----Original Message-----
> >>>>>>>>From: [email protected] [mailto:meta-
> >>>>arago-
> >>>>>>>>[email protected]] On Behalf Of Dmytriyenko, Denys
> >>>>>>>>Sent: Friday, October 25, 2013 5:35 AM
> >>>>>>>>To: Hingolikar, Mrinmayee
> >>>>>>>>Cc: [email protected]
> >>>>>>>>Subject: Re: [meta-arago] [PATCH 0/5] GStreamer multimedia
> >>>>>>>framework and
> >>>>>>>>plugins recipes
> >>>>>>>>
> >>>>>>>>On Fri, Oct 25, 2013 at 03:03:35PM +0530, Mrinmayee
> >>>>Hingolikar
> >>>>>>>wrote:
> >>>>>>>>>This patch-set adds GStreamer framework and its plugins:
> >>>>>>>>>base, good, bad and ugly.
> >>>>>>>>>
> >>>>>>>>>The development trees for all of these components are
> >>>>>>>mainitained at
> >>>>>>>>>git.ti.com and they are based on top of the Freedesktop
> >>>>>>>releases.
> >>>>>>>>Why? How large is the patchset on top of upstream?
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>>Mrinmayee Hingolikar (5):
> >>>>>>>>>    multimedia: Add GStreamer multimedia framework.
> >>>>>>>>>    multimedia: GStreamer base plugins
> >>>>>>>>>    multimedia: GStreamer good plugins
> >>>>>>>>>    multimedia: GStreamer bad plugins
> >>>>>>>>>    multimedia: GStreamer ugly plugins.
> >>>>>>>>>
> >>>>>>>>>   .../gstreamer/gst-plugins-bad_0.10.23.bbappend     |
> >>>>25
> >>>>>>>>++++++++++++++++++++
> >>>>>>>>>   .../gstreamer/gst-plugins-base_0.10.36.bbappend    |
> >>>>22
> >>>>>>>>+++++++++++++++++
> >>>>>>>>>   .../gstreamer/gst-plugins-good_0.10.31.bbappend    |
> >>>>20
> >>>>>>>>++++++++++++++++
> >>>>>>>>>   .../gstreamer/gst-plugins-ugly_0.10.18.bb          |
> >>>>23
> >>>>>>>++++++++++++++++++
> >>>>>>>>>   .../gstreamer/gstreamer_0.10.36.bbappend           |
> >>>>16
> >>>>>>>+++++++++++++
> >>>>>>>>>   5 files changed, 106 insertions(+)
> >>>>>>>>>   create mode 100644
> >>>>>>>>>meta-arago-extras/recipes-multimedia/gstreamer/gst-
> >>>>plugins-
> >>>>>>>bad_0.10.23
> >>>>>>>>>.bbappend  create mode 100644
> >>>>>>>>>meta-arago-extras/recipes-multimedia/gstreamer/gst-
> >>>>plugins-
> >>>>>>>base_0.10.3
> >>>>>>>>>6.bbappend  create mode 100644
> >>>>>>>>>meta-arago-extras/recipes-multimedia/gstreamer/gst-
> >>>>plugins-
> >>>>>>>good_0.10.3
> >>>>>>>>>1.bbappend  create mode 100644
> >>>>>>>>>meta-arago-extras/recipes-multimedia/gstreamer/gst-
> >>>>plugins-
> >>>>>>>ugly_0.10.1
> >>>>>>>>>8.bb  create mode 100644
> >>>>>>>>>meta-arago-extras/recipes-
> >>>>>>>multimedia/gstreamer/gstreamer_0.10.36.bbapp
> >>>>>>>>>end
> >>>>>>>>>
> >>>>>>>>>--
> >>>>>>>>>1.7.9.5
> >>>>>>>>>
> >>>>>>>>>_______________________________________________
> >>>>>>>>>meta-arago mailing list
> >>>>>>>>>[email protected]
> >>>>>>>>>http://arago-project.org/cgi-bin/mailman/listinfo/meta-
> >>>>arago
> >>>>>>>>_______________________________________________
> >>>>>>>>meta-arago mailing list
> >>>>>>>>[email protected]
> >>>>>>>>http://arago-project.org/cgi-bin/mailman/listinfo/meta-
> >>>>arago
> >>>>>>>_______________________________________________
> >>>>>>>meta-arago mailing list
> >>>>>>>[email protected]
> >>>>>>>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >>>_______________________________________________
> >>>meta-arago mailing list
> >>>[email protected]
> >>>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> 
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Reply via email to