[oe] [oe-commits] Martin Jansa : midori: fix build with dirty vala

Martin Jansa martin.jansa at gmail.com
Wed Nov 2 23:32:54 UTC 2011


On Wed, Nov 02, 2011 at 10:52:15PM +0100, Andreas Müller wrote:
> On Tuesday, November 01, 2011 09:59:01 AM git at git.openembedded.org wrote:
> > Module: meta-openembedded.git
> > Branch: master
> > Commit: bcfc89236b7a96eb00999492351fb9cafc0cbf84
> > URL:   
> > http://git.openembedded.org/?p=meta-openembedded.git&a=commit;h=bcfc89236b
> > 7a96eb00999492351fb9cafc0cbf84
> > 
> > Author: Martin Jansa <Martin.Jansa at gmail.com>
> > Date:   Wed Oct 19 13:47:03 2011 +0200
> > 
> > midori: fix build with dirty vala
> > 
> > * there is only one line added to default waf file
> >   in unpack_wafdir() just before last chdir and return
> >   because those files are downloaded and cannot be patched in normal way
> >   os.system("sed -i
> > \"s/version=output.split/version=output.replace('-dirty','').split/g\"
> > wafadmin/Tools/vala.py")
> > 
> > * It fails to parse version like this
> >   /OE/shr-core/tmp/sysroots/x86_64-linux/usr/bin/valac --version
> >   Vala 0.12.1-dirty
> >   reporting 0.0.0 instead which breaks configure
> > 
> > Signed-off-by: Martin Jansa <Martin.Jansa at gmail.com>
> > Signed-off-by: Koen Kooi <koen at dominion.thruhere.net>
> > 
> > ---
> > 
> >  .../recipes-connectivity/midori/midori-0.4.1/waf   |  Bin 0 -> 92569 bytes
> >  .../recipes-connectivity/midori/midori_0.4.1.bb    |    8 ++++++++
> >  2 files changed, 8 insertions(+), 0 deletions(-)
> > 
> > diff --git a/meta-oe/recipes-connectivity/midori/midori-0.4.1/waf
> > b/meta-oe/recipes-connectivity/midori/midori-0.4.1/waf new file mode
> > 100755
> > index 0000000..b74a2d8
> > Binary files /dev/null and
> > b/meta-oe/recipes-connectivity/midori/midori-0.4.1/waf differ diff --git
> > a/meta-oe/recipes-connectivity/midori/midori_0.4.1.bb
> > b/meta-oe/recipes-connectivity/midori/midori_0.4.1.bb index
> > 2805b92..860ed21 100644
> > --- a/meta-oe/recipes-connectivity/midori/midori_0.4.1.bb
> > +++ b/meta-oe/recipes-connectivity/midori/midori_0.4.1.bb
> > @@ -1,7 +1,15 @@
> >  require midori.inc
> > 
> > +PR = "r1"
> > +
> >  SRC_URI =
> > "http://archive.xfce.org/src/apps/midori/0.4/midori-${PV}.tar.bz2;name=mid
> > ori \ +           file://waf \
> >            "
> > +
> > +do_configure_prepend() {
> > +  cp -f ${WORKDIR}/waf ${S}/
> > +}
> > +
> >  SRC_URI[midori.md5sum] = "33dde203cd71ae2b1d2adcc7f5739f65"
> >  SRC_URI[midori.sha256sum] =
> > "b3c77ddb6f562a0ea3d2e6fbb756ca0dc158ad8410f4b60f8e5998c2f7305f07"
> > 
> > 
> Hm - I am afraid since this update midori never finishes configuring here. I 
> waited for approximately 2 hours and stopped the process. The log file is 
> attached ( a fresh retry creates similar log ). I commented out the whole 

Have you tried to revert it to see that it's not caused ie by glib
changes or something? Because my vala detection change is already finished 
before it writes this stuff and the tar in waf is compared with
hardcoded checksum, so it also could not change just by this monster
blob.

Could you also check if there is something interesting in ps aux or
strace when it's "hanging"? On all 3 boxes I have tried this it always
worked as expected.

> | do_configure_prepend() 
> 
> function, ran
> 
> | bitbake -ccleanall midori
> 
> and
> 
> | bitbake midori
> 
> without further issues in few minutes. 
> 
> Maybe we should rethink this monster blob commit intended to fix a corner case...

Yes pity that this corner case is our only vala-native and I was hopping
that newer midori with newer waf-1.6 will be released soon and this
won't be needed anymore..

Cheers,

> Andreas

> + cd /home/Superandy/tmp/oe-core-eglibc/work/armv7a-angstrom-linux-gnueabi/midori-0.4.1-r1/midori-0.4.1
> + do_configure
> + cp -f /home/Superandy/tmp/oe-core-eglibc/work/armv7a-angstrom-linux-gnueabi/midori-0.4.1-r1/waf /home/Superandy/tmp/oe-core-eglibc/work/armv7a-angstrom-linux-gnueabi/midori-0.4.1-r1/midori-0.4.1/
> + sed -i -e 's:, shell=False::g' -e s:/usr/X11R6/include::g -e s:/usr/X11R6/lib::g wscript
> + ./configure --prefix=/usr --bindir=/usr/bin --sbindir=/usr/sbin --libexecdir=/usr/libexec --datadir=/usr/share --sysconfdir=/etc --sharedstatedir=/com --localstatedir=/var --libdir=/usr/lib --includedir=/usr/include --infodir=/usr/share/info --mandir=/usr/share/man --disable-hildon
> Checking for Python			:  /home/Superandy/tmp/oe-core-eglibc/sysroots/x86_64-linux/usr/bin/python2
> Checking for WAF			:  /home/Superandy/tmp/oe-core-eglibc/work/armv7a-angstrom-linux-gnueabi/midori-0.4.1-r1/midori-0.4.1/waf
> calling waf configure with parameters
> Checking for program gcc or cc           : ccache arm-angstrom-linux-gnueabi-gcc  -march=armv7-a -fno-tree-vectorize      -mthumb-interwork -mfloat-abi=softfp -mfpu=neon -mtune=cortex-a8 -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed -ljavascriptcoregtk-1.0  -mthumb-interwork -mno-thumb --sysroot=/home/Superandy/tmp/oe-core-eglibc/sysroots/overo 
> Checking for program ar                  : arm-angstrom-linux-gnueabi-ar 
> Checking for program ranlib              : arm-angstrom-linux-gnueabi-ranlib 
> Checking for gcc                         : ok  
> Checking for program valac               : /home/Superandy/tmp/oe-core-eglibc/sysroots/x86_64-linux/usr/bin/valac 
> Checking for gobject-2.0                 : yes 
> Checking for gthread-2.0                 : yes 
> Checking for program version valac >= 0.1.6 : ok 0.12.1 
> Checking for program glib-genmarshal        : /home/Superandy/tmp/oe-core-eglibc/sysroots/x86_64-linux/usr/bin/glib-genmarshal 
> Checking for program glib-mkenums           : /home/Superandy/tmp/oe-core-eglibc/sysroots/x86_64-linux/usr/bin/glib-mkenums 

> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel


-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20111103/b8ae2ee8/attachment-0002.sig>


More information about the Openembedded-devel mailing list