On Tuesday 16 August 2011 14:57:48 Anders Darander wrote:
> When reducing build time by adding QT_DISTRO_FLAGS from the set '-no-gui
> -nomake tools -nomake examples -nomake demos -nomake docs', build failure
> can occur, due to not building all tools.
> 
> * Make rm not error out when removing one of the, possibly, non-existing
> tools. * Only try to rename qtdemo, if qtdemo exists.
> * Guard do_split_packages() with an try-except-clause, otherwise we get an
> error stating that .../phrasebooks/... do not exist.
> 
> Signed-off-by: Anders Darander <and...@chargestorm.se>
> ---
>  meta/recipes-qt/qt4/qt4.inc |   13 ++++++++-----
>  1 files changed, 8 insertions(+), 5 deletions(-)
> 
> diff --git a/meta/recipes-qt/qt4/qt4.inc b/meta/recipes-qt/qt4/qt4.inc
> index 0410a92..994ec12 100644
> --- a/meta/recipes-qt/qt4/qt4.inc
> +++ b/meta/recipes-qt/qt4/qt4.inc
> @@ -227,7 +227,10 @@ python populate_packages_prepend() {
> 
>          phrasebook_dir =
> bb.data.expand('${datadir}/${QT_DIR_NAME}/phrasebooks/', d)
> phrasebook_name = bb.data.expand('${QT_BASE_NAME}-phrasebook-%s', d) -    
>    do_split_packages(d, phrasebook_dir, '^(.*)\.qph$', phrasebook_name,
> '${PN} phrasebook for %s', extra_depends='' ) +        try:
> +                do_split_packages(d, phrasebook_dir, '^(.*)\.qph$',
> phrasebook_name, '${PN} phrasebook for %s', extra_depends='' ) +       
> except:
> +                pass
> 
>          # Package all the plugins and their -dbg version and create a meta
> package import os
> @@ -278,11 +281,11 @@ do_install() {
> 
>       # These are host binaries, we should only use them in staging
>       rm ${D}/${bindir}/qmake
> -     rm ${D}/${bindir}/uic
> -     rm ${D}/${bindir}/uic3
> +     rm -f ${D}/${bindir}/uic
> +     rm -f ${D}/${bindir}/uic3
>       rm ${D}/${bindir}/moc
>       rm ${D}/${bindir}/rcc
> -     rm ${D}/${bindir}/lrelease
> +     rm -f ${D}/${bindir}/lrelease
> 
>       # fix pkgconfig, libtool and prl files
>       sed -i -e s#-L${S}/lib##g \
> @@ -319,6 +322,6 @@ do_install() {
> 
>       #Append an E to the qtdemo file
>       if [ -n "${QT_LIBINFIX}" ] ; then
> -             mv ${D}${bindir}/qtdemo ${D}${bindir}/qtdemo${QT_LIBINFIX}
> +             [ -f ${D}${bindir}/qtdemo ] && mv ${D}${bindir}/qtdemo
> ${D}${bindir}/qtdemo${QT_LIBINFIX} fi
>  }

Acked-by: Paul Eggleton <paul.eggle...@linux.intel.com>

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to