[beagleboard-gsoc] bitbake reciped diff

Google Summer of Code
1

Pramod,

Thanks for the patch, see my comments below.

--- a/recipes/opencv/opencv-dsp-acceleration_svn.bb
+++ b/recipes/opencv/opencv-dsp-acceleration_svn.bb

do_configure() {
+ #DSPLIB uses '\' instead of '/' as directory seperator. Correct it
+ sed -i -e 's:\\:\/:g' ${DSPLIB_PATH}/dsplib64plus.h

2

Hi Koen,

Thanks for the comments.

If the directory seperator was aleady fixed in dsplib recipe, it should not be needed. I had trouble bitbaking until I fixed it. I will see that again.

I will send you the new patch with those comments addressed.

Regards,
Pramod

3

Hi Koen,

I made the necessary changes in the Makefile as suggested. Please find the patch. I have bumped the revsion number in this patch.

Regards,
Pramod Poudel

0001-opencv-dsp-acceleration-Revision-changed-to-128.patch (1.38 KB)

4

Hi Koen,

I made the necessary changes in the Makefile as suggested. Please find the patch. I have bumped the revsion number in this patch.

Close, but not quite there :slight_smile:

Signed-off-by: Pramod Poudel <ppoudel@jars-desktop.(none)>

Those aren't valid address, you probably need to do:

git config --global user.name "Your Name"
git config --global user.email "your@email.com"

${OE_BASE}/build/tmp-angstrom_2008_1/sysroots/beagleboard-angstrom-linux-gnueabi is a bit to specific, please change that to:
${STAGING_DIR}/${MULTIMACH_TARGET_SYS}${installdir}/imglib_v201 that way it will work for other machines and other build layouts.

regards,

Koen

5

Hi Koen,

Please find the attached file. Hope I have’t done something wrong with it :slight_smile: .

Regards,
Pramod Poudel

0001-opencv-dsp-acceleration-Revision-changed-to-128.patch (1.44 KB)

6

Looks good, it's in: http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=9353e56c0ea4f31592749fc21d937561b9109aff

congrats on your first successfull patch to OE :slight_smile: