Patchwork [3/3,v2] libomxil-bellagio_git: add Git version

login
register
mail settings
Submitter Paul Menzel
Date Jan. 17, 2011, 11:45 a.m.
Message ID <1295264704.4844.54.camel@mattotaupa>
Download mbox | patch
Permalink /patch/299/
State Accepted
Headers show

Comments

Paul Menzel - Jan. 17, 2011, 11:45 a.m.
Date: Mon, 17 Jan 2011 00:15:51 +0100

The last release 0.9.2.1 is over a year old (2009-12-23).

Build tested with `angstrom-2008.1` for `MACHINE = "beagleboard"`. With `angstrom-2010.x` it fails with the following error.

        arm-angstrom-linux-gnueabi-libtool: compile:  arm-angstrom-linux-gnueabi-gcc -march=armv7-a -mtune=cortex-a8 -mfpu=neon -mfloat-abi=softfp -mthumb-interwork -mno-thumb -DHAVE_CONFIG_H -I. -I../.. -isystem/oe/build-angstrom-next/angstrom-dev/sysroots/armv7a-angstrom-linux-gnueabi/usr/include -I../../include -I../../src -isystem/oe/build-angstrom-next/angstrom-dev/sysroots/armv7a-angstrom-linux-gnueabi/usr/include -fexpensive-optimizations -fomit-frame-pointer -frename-registers -O2 -ggdb2 -Wall -Werror -DCONFIG_DEBUG_LEVEL=0 -MT libomxbase_la-omx_base_component.lo -MD -MP -MF .deps/libomxbase_la-omx_base_component.Tpo -c omx_base_component.c  -fPIC -DPIC -o .libs/libomxbase_la-omx_base_component.o
        cc1: warnings being treated as errors
        omx_base_component.c: In function 'omx_base_component_GetParameter':
        omx_base_component.c:991:3: error: case value '2130706435' not in enumerated type 'OMX_INDEXTYPE'
        omx_base_component.c:918:3: error: case value '2130706436' not in enumerated type 'OMX_INDEXTYPE'
        make[3]: *** [libomxbase_la-omx_base_component.lo] Error 1
        make[3]: Leaving directory `/oe/build-angstrom-next/angstrom-dev/work/armv7a-angstrom-linux-gnueabi/libomxil-bellagio-0.9.2.1+gitr0+772f0eca88b242ffb5469e8e381c63ce1f7cd3e1-r7.0/git/src/base'

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
---
Am Montag, den 17.01.2011, 09:13 +0100 schrieb Koen Kooi:
> On 17-01-11 00:43, Paul Menzel wrote:

[…]

> > 1. I hope the package numbering is correctly. I used the Freesmartphone packages as an example.
> > 2. Is a `DP = "-1"` needed? Users of this recipe speak up please!
> 
> > +SRCREV = "772f0eca88b242ffb5469e8e381c63ce1f7cd3e1"
> > +SRC_URI = "git://omxil.git.sourceforge.net/gitroot/omxil/omxil;protocol=git"
> > +PV = "0.9.2.1+gitr${SRCPV}"
> > +PR = "${INC_PR}.0"
> 
> For get recipes we generally do:
> 
> PR_append = "+gitr${SRCPV}" to keep the ugly hash out of PV, but keep it
> available to the users

Should the manual be a updated then [1]?


Thanks,

Paul


[1] http://docs.openembedded.org/usermanual/usermanual.html#id536032
---
 recipes/openmax/libomxil-bellagio_git.bb |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)
 create mode 100644 recipes/openmax/libomxil-bellagio_git.bb

Patch

diff --git a/recipes/openmax/libomxil-bellagio_git.bb b/recipes/openmax/libomxil-bellagio_git.bb
new file mode 100644
index 0000000..4e28251
--- /dev/null
+++ b/recipes/openmax/libomxil-bellagio_git.bb
@@ -0,0 +1,11 @@ 
+require ${PN}.inc
+
+PR = "${INC_PR}.0"
+
+SRCREV = "772f0eca88b242ffb5469e8e381c63ce1f7cd3e1"
+SRC_URI = "git://omxil.git.sourceforge.net/gitroot/omxil/omxil;protocol=git"
+PV = "0.9.2.1"
+PR = "${INC_PR}.0"
+PR_append = "+gitr${SRCPV}"
+
+S = "${WORKDIR}/git"