[meta-ti] [PATCH 4/6] libdce: add version 1.6.5.3

[Enrico]

On Thu, Oct 11, 2012 at 10:17 AM, Nicolas Dechesne <ndec13 at gmail.com> wrote:
> On Thu, Oct 11, 2012 at 1:40 AM, Enrico Butera <ebutera at users.berlios.de> wrote:
>> Signed-off-by: Enrico Butera <ebutera at users.berlios.de>
>> +DESCRIPTION = "TI Ducati IVA-HD hardware codec acceleration API"
>> +HOMEPAGE = "http://gitorious.org/ubuntu-omap/dce"
>
> our real 'upstream' tree for libdce is here:
> http://gitorious.org/gstreamer-omap/libdce/
>
> the one you are using is our 'ubuntu package' tree where the
> 'upstream' is merged along with debian/ubuntu meta data (./debian
> folder).
>
> sometimes we maintain some patches in ubuntu tree in debian/patches,
> but for libdce since we are also the real upstream, we tend to push
> all patches in upstream.
>
> so you should use the upstream tree instead, branch master. in fact
> the 'upstream' branch in the tree you use is the same as the 'master'
> branch of the upstream tree ;-)

Yes i had some confusion at first between the two repositories, i
think i started using the ubuntu one because it has tags that can help
in releases maintenance, but in the end i didn't use them so yes, i
will switch to the upstream tree.


> libdce tree contains both the linux side lib, and the ducati side DCE
> code for now (which can't be rebuild completely for now, as we aren't
> releasing all the bits/pieces for ducati side).
>
> we have been discussing about spliting this tree into 2 for better
> clarity, so that might happen at some point..

I learned it the hard way some time ago, at my first attempt to add gst-ducati!


>> +
>> +require ../includes/ti-paths.inc
>> +require ../includes/ti-staging.inc
>
> what is this for?

some copy/paste leftovers :D
will delete them


>> +
>> +inherit autotools
>> +
>> +DEPENDS = "libdrm"
>> +RDEPENDS = "libdrm ti-firmware-ipu-dce"
>
> i would hope that libdrm is not needed in RDEPENDS, as it should be
> handled automagically by shlibdeps.

I really don't know, if you are sure about that i will just remove it.

Enrico