Re: [oe-commits] Christopher Friedt : emotion_svn.bb: Make it look for edje_cc in STAGING_DIR.

Re: [oe-commits] Christopher Friedt : emotion_svn.bb: Make it look for edje_cc in STAGING_DIR.

[Stefan Schmidt]

Hello.

On Sat, 2009-08-08 at 07:46, git version control wrote:
> 
> diff --git a/recipes/efl1/emotion_svn.bb b/recipes/efl1/emotion_svn.bb
> index 0f31e66..79b35d9 100644
> --- a/recipes/efl1/emotion_svn.bb
> +++ b/recipes/efl1/emotion_svn.bb
> @@ -3,7 +3,7 @@ LICENSE = "MIT BSD"
>  # we no longer build the libxine backend, since the gstreamer backend seems more promising
>  DEPENDS = "eet evas ecore edje gstreamer gst-plugins-base"
>  PV = "0.1.0.042+svnr${SRCREV}"
> -PR = "r2"
> +PR = "r3"
>  
>  inherit efl
>  
> @@ -12,3 +12,11 @@ EXTRA_OECONF = "--disable-xine --enable-gstreamer --with-edje-cc=${STAGING_BINDI
>  PACKAGES =+ "emotion-backend-gstreamer"
>  FILES_emotion-backend-gstreamer = "${libdir}/emotion/*.so"
>  RRECOMMENDS_${PN} = "emotion-backend-gstreamer"
> +
> +
> +do_configure_append() {
> +	for i in $(find ${WORKDIR}/emotion -name 'Makefile'); do 
> +		sed -i -e s:/usr/bin/edje_cc:${STAGING_DIR}/${BUILD_SYS}/usr/bin/edje_cc: \
> +			${i}
> +	done
> +}

Christopher, can you verify that the configure_append is really needed? I added
--with-edje-cc in EXTRA_OECONF, you can see a part of it above, to fix the
edje_cc native detection.

Starting a build from scratch today emotion fails for me as the sed entry messes
around with the path as follows:

| make[2]:
/home/sschmidt/oe/build/dev/tmp/staging/x86_64-linux/home/sschmidt/oe/build/dev/tmp/staging/x86_64-linux/usr/bin/edje_cc:
Command not found

Removing the do_configure_append again makes it work again over here.

regards
Stefan Schmidt

Re: [oe-commits] Christopher Friedt : emotion_svn.bb: Make it look for edje_cc in STAGING_DIR.

[Stefan Schmidt]

Hello.

[Adding Christoper explicit in CC]

If nobody gives me a good reson against it I will remove the configure hunk at
the weekend as it brekas my build and should not be needed.

regards
Stefan Schmidt

On Tue, 2009-09-08 at 20:20, Stefan Schmidt wrote:
> 
> On Sat, 2009-08-08 at 07:46, git version control wrote:
> > 
> > diff --git a/recipes/efl1/emotion_svn.bb b/recipes/efl1/emotion_svn.bb
> > index 0f31e66..79b35d9 100644
> > --- a/recipes/efl1/emotion_svn.bb
> > +++ b/recipes/efl1/emotion_svn.bb
> > @@ -3,7 +3,7 @@ LICENSE = "MIT BSD"
> >  # we no longer build the libxine backend, since the gstreamer backend seems more promising
> >  DEPENDS = "eet evas ecore edje gstreamer gst-plugins-base"
> >  PV = "0.1.0.042+svnr${SRCREV}"
> > -PR = "r2"
> > +PR = "r3"
> >  
> >  inherit efl
> >  
> > @@ -12,3 +12,11 @@ EXTRA_OECONF = "--disable-xine --enable-gstreamer --with-edje-cc=${STAGING_BINDI
> >  PACKAGES =+ "emotion-backend-gstreamer"
> >  FILES_emotion-backend-gstreamer = "${libdir}/emotion/*.so"
> >  RRECOMMENDS_${PN} = "emotion-backend-gstreamer"
> > +
> > +
> > +do_configure_append() {
> > +	for i in $(find ${WORKDIR}/emotion -name 'Makefile'); do 
> > +		sed -i -e s:/usr/bin/edje_cc:${STAGING_DIR}/${BUILD_SYS}/usr/bin/edje_cc: \
> > +			${i}
> > +	done
> > +}
> 
> Christopher, can you verify that the configure_append is really needed? I added
> --with-edje-cc in EXTRA_OECONF, you can see a part of it above, to fix the
> edje_cc native detection.
> 
> Starting a build from scratch today emotion fails for me as the sed entry messes
> around with the path as follows:
> 
> | make[2]:
> /home/sschmidt/oe/build/dev/tmp/staging/x86_64-linux/home/sschmidt/oe/build/dev/tmp/staging/x86_64-linux/usr/bin/edje_cc:
> Command not found
> 
> Removing the do_configure_append again makes it work again over here.
> 
> regards
> Stefan Schmidt
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel