On Fri, Aug 29, 2014 at 10:51 AM, Lauren Post <lauren.p...@freescale.com> wrote:
> We kept gst 0.1 and 1.0 in separate patches since gst 1.0 is a new component.

Oh, I see.

So please, in this patch, change the commit log when

>>> Gstreamer Freescale 1.0 specific features moved into the 1.0 recipe.

Use something like:

Gstreamer Freescale 1.0 specific features will be moved into the 1.0 recipe.


Strange we had gstreamer freescale 1.0 specific features needing to me
moved out of .inc file in master....

Well, Ok, other than commit log change I´m ok with that.


Daiane
>
> -----Original Message-----
> From: angol...@gmail.com [mailto:angol...@gmail.com] On Behalf Of Daiane 
> Angolini
> Sent: Friday, August 29, 2014 8:23 AM
> To: Post Lauren-RAA013
> Cc: Wang Zidan-B50113; meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 1/5] gst-fsl-plugin: Make 
> a common include file for 0.1 and 1.0 gstreamer plugins
>
> On Fri, Aug 29, 2014 at 10:13 AM, Lauren Post <lauren.p...@freescale.com> 
> wrote:
>> For us to use a common include that works for both gstreamer 1.0 and 
>> gstreamer 0.1 we kept only common settings in the gst-fsl-plugin.inc and put 
>> 0.1 specific changes into 0.1 package recipe and 1.0 changes into 1.0 
>> package recipe.   WE preferred that over creating a 1.0 and 0.1 include file.
>>
>
> I meant I would expect that both gst1.0.bb and gst0.1.bb were changed by the 
> patch. But only gst0.1 was changed in this patch.
>
>
> Daiane
>
>> Lauren
>>
>> -----Original Message-----
>> From: meta-freescale-boun...@yoctoproject.org
>> [mailto:meta-freescale-boun...@yoctoproject.org] On Behalf Of Daiane
>> Angolini
>> Sent: Friday, August 29, 2014 7:56 AM
>> To: Wang Zidan-B50113
>> Cc: meta-freescale@yoctoproject.org
>> Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 1/5]
>> gst-fsl-plugin: Make a common include file for 0.1 and 1.0 gstreamer
>> plugins
>>
>> On Fri, Aug 29, 2014 at 2:42 AM, Zidan Wang <b50...@freescale.com> wrote:
>>> From: Wang Zidan <b50...@freescale.com>
>>>
>>> To avoid duplication, use one include file that supports both 0.1 and 1.0 
>>> gstreamer plugins common features.
>>> Gstreamer Freescale  0.1 specific features moved into the 0.1 recipe.
>>> Gstreamer Freescale 1.0 specific features moved into the 1.0 recipe.
>>
>> I´m confused, Zidan.
>>
>> You say specific features was moved into the 0.1 and 1.0 recipes...
>>
>>>
>>> Signed-off-by: Wang Zidan <b50...@freescale.com>
>>> ---
>>>  recipes-multimedia/gstreamer/gst-fsl-plugin.inc    |   18 
>>> +-----------------
>>>  .../gstreamer/gst-fsl-plugin_3.0.11.bb             |   20 
>>> ++++++++++++++++++++
>>
>> however, you change one recipe only here.
>>
>> Is that right?
>>
>> Have you built this patchset for mx5/mx28/mx6 or only for mx6?
>>
>> Daiane
>>
>>>  2 files changed, 21 insertions(+), 17 deletions(-)
>>>
>>> diff --git a/recipes-multimedia/gstreamer/gst-fsl-plugin.inc
>>> b/recipes-multimedia/gstreamer/gst-fsl-plugin.inc
>>> index f8685bc..d925de6 100644
>>> --- a/recipes-multimedia/gstreamer/gst-fsl-plugin.inc
>>> +++ b/recipes-multimedia/gstreamer/gst-fsl-plugin.inc
>>> @@ -6,7 +6,7 @@ DESCRIPTION = "Gstreamer freescale plugins"
>>>  LICENSE = "GPLv2 & LGPLv2 & LGPLv2.1"
>>>  SECTION = "multimedia"
>>>
>>> -DEPENDS = "gstreamer gst-plugins-base libfslcodec libfslparser 
>>> virtual/kernel"
>>> +DEPENDS = "libfslcodec libfslparser virtual/kernel"
>>>  DEPENDS_append_mx5 = " imx-lib imx-vpu"
>>>  DEPENDS_append_mx6q = " imx-lib imx-vpu libfslvpuwrap"
>>>  DEPENDS_append_mx6dl = " imx-lib imx-vpu libfslvpuwrap"
>>> @@ -16,12 +16,6 @@ LIC_FILES_CHKSUM = 
>>> "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552 \
>>>                      
>>> file://COPYING-LGPL-2;md5=5f30f0716dfdd0d91eb439ebec522ec2 \
>>>                      
>>> file://COPYING-LGPL-2.1;md5=fbc093901857fcd118f065f900982c24"
>>>
>>> -S = "${WORKDIR}/gst-fsl-plugins-${PV}"
>>> -
>>> -SRC_URI = "${FSL_MIRROR}/gst-fsl-plugins-${PV}.tar.gz \
>>> -           file://build-Fix-out-of-tree-build.patch \
>>> -           file://Remove-use-of-obsolete-VIDIOC_DBG_G_CHIP_IDENT.patch"
>>> -
>>>  inherit autotools pkgconfig
>>>
>>>  PLATFORM_mx5 = "MX51"
>>> @@ -45,22 +39,12 @@ PACKAGECONFIG[aacp] +=
>>> "--enable-aacpdec,--disable-aacpdec,libfslaacpcodec,libfs
>>>  MSDEPENDS = "libfslmsparser libfslmscodec"
>>>  PACKAGECONFIG[wma10dec] 
>>> +="--enable-wma10dec,--disable-wma10dec,${MSDEPENDS},${MSDEPENDS}"
>>>  PACKAGECONFIG[wma8enc] 
>>> +="--enable-wma8enc,--disable-wma8enc,${MSDEPENDS},${MSDEPENDS}"
>>> -PACKAGECONFIG[wmv9mpdec] 
>>> +="--enable-wmv9mpdec,--disable-wmv9mpdec,${MSDEPENDS},${MSDEPENDS}"
>>> -PACKAGECONFIG[wmv78dec] 
>>> +="--enable-wmv78dec,--disable-wmv78dec,${MSDEPENDS},${MSDEPENDS}"
>>>
>>> -FILES_${PN} = "${libdir}/gstreamer-0.10/*.so ${datadir}"
>>>  # Add codec list that the beep plugin run-time depended
>>> BEEP_RDEPENDS = "libfslcodec-aac libfslcodec-mp3 libfslcodec-oggvorbis"
>>>  RDEPENDS_${PN} += "gst-plugins-good-id3demux libfslparser ${BEEP_RDEPENDS}"
>>>
>>> -FILES_${PN}-dbg += "${libdir}/gstreamer-0.10/.debug"
>>> -FILES_${PN}-dev += "${libdir}/gstreamer-0.10/*.la ${libdir}/pkgconfig/*.pc"
>>> -FILES_${PN}-staticdev += "${libdir}/gstreamer-0.10/*.a"
>>> -
>>> -FILES_${PN}-gplay = "${bindir}/gplay"
>>>  FILES_${PN}-libmfwsconf = "${libdir}/libmfwsconf${SOLIBS}"
>>> -FILES_${PN}-libgplaycore = "${libdir}/libgplaycore${SOLIBS}"
>>> -FILES_${PN}-libgstfsl = "${libdir}/libgstfsl-0.10${SOLIBS}"
>>>  FILES_${PN}-libme = "${libdir}/libme${SOLIBS}"
>>>  FILES_${PN}-libgstbufmeta = "${libdir}/libgstbufmeta${SOLIBS}"
>>>  FILES_${PN}-libmfwba =  "${libdir}/libmfwba${SOLIBS}"
>>> diff --git a/recipes-multimedia/gstreamer/gst-fsl-plugin_3.0.11.bb
>>> b/recipes-multimedia/gstreamer/gst-fsl-plugin_3.0.11.bb
>>> index d0b0c9e..e8fe34e 100644
>>> --- a/recipes-multimedia/gstreamer/gst-fsl-plugin_3.0.11.bb
>>> +++ b/recipes-multimedia/gstreamer/gst-fsl-plugin_3.0.11.bb
>>> @@ -5,7 +5,27 @@ require gst-fsl-plugin.inc
>>>
>>>  EXTRA_OECONF += " CROSS_ROOT=${PKG_CONFIG_SYSROOT_DIR}"
>>>
>>> +SRC_URI = "${FSL_MIRROR}/gst-fsl-plugins-${PV}.tar.gz \
>>> +           file://build-Fix-out-of-tree-build.patch \
>>> +
>>> +file://Remove-use-of-obsolete-VIDIOC_DBG_G_CHIP_IDENT.patch \ "
>>> +
>>> +S = "${WORKDIR}/gst-fsl-plugins-${PV}"
>>> +
>>>  SRC_URI[md5sum] = "0ed858681a74857034c006036023e6ce"
>>>  SRC_URI[sha256sum] = 
>>> "fbc6000b401ac2c8d67d1414372f4a929cf0a5808f6ed1640f1d2bfcce2f2a4f"
>>>
>>> +DEPENDS_append = " gstreamer gst-plugins-base"
>>> +
>>> +PACKAGECONFIG[wmv9mpdec] 
>>> +="--enable-wmv9mpdec,--disable-wmv9mpdec,${MSDEPENDS},${MSDEPENDS}"
>>> +PACKAGECONFIG[wmv78dec] 
>>> +="--enable-wmv78dec,--disable-wmv78dec,${MSDEPENDS},${MSDEPENDS}"
>>> +
>>> +FILES_${PN} = "${libdir}/gstreamer-0.10/*.so ${datadir}"
>>> +FILES_${PN}-dbg += "${libdir}/gstreamer-0.10/.debug"
>>> +FILES_${PN}-dev += "${libdir}/gstreamer-0.10/*.la ${libdir}/pkgconfig/*.pc"
>>> +FILES_${PN}-staticdev += "${libdir}/gstreamer-0.10/*.a"
>>> +FILES_${PN}-gplay = "${bindir}/gplay"
>>> +FILES_${PN}-libgplaycore = "${libdir}/libgplaycore${SOLIBS}"
>>> +FILES_${PN}-libgstfsl = "${libdir}/libgstfsl-0.10${SOLIBS}"
>>> +
>>>  COMPATIBLE_MACHINE = "(mx28|mx5|mx6)"
>>> --
>>> 1.7.9.5
>>>
>>> --
>>> _______________________________________________
>>> meta-freescale mailing list
>>> meta-freescale@yoctoproject.org
>>> https://lists.yoctoproject.org/listinfo/meta-freescale
>> --
>> _______________________________________________
>> meta-freescale mailing list
>> meta-freescale@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-freescale
-- 
_______________________________________________
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale

Reply via email to