[PATCH 1/2] boost : Add icu to DEPENDS to fix a build failure.

[PATCH 1/2] boost : Add icu to DEPENDS to fix a build failure.

[Philip Balister]

Part of boost needs libraries from icu. Without this, either the build or
image construction fails. (I forget exactly which)

Signed-off-by: Philip Balister <philip@balister.org>
---
 meta/recipes-support/boost/boost.inc |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-support/boost/boost.inc b/meta/recipes-support/boost/boost.inc
index fecdb97..a1499f6 100644
--- a/meta/recipes-support/boost/boost.inc
+++ b/meta/recipes-support/boost/boost.inc
@@ -6,7 +6,7 @@
 DESCRIPTION = "Free peer-reviewed portable C++ source libraries"
 HOMEPAGE = "http://www.boost.org/"
 SECTION = "libs"
-DEPENDS = "boost-native zlib"
+DEPENDS = "boost-native zlib icu"
 DEPENDS_virtclass-native = ""
 LICENSE = "Boost"
 
-- 
1.7.3.4

[PATCH 2/2] qt4.inc : Add uic, moc, rcc, and lrelease to tools package.

[Philip Balister]

These binaries are needed in the target tools package so we can build
qt apps on the target. You also need to install qmake from meta-oe.

Test by building gnuradio on the USRP e100 with qt support.

Signed-off-by: Philip Balister <philip@balister.org>
---
 meta/recipes-qt/qt4/qt4.inc |    9 ++-------
 1 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/meta/recipes-qt/qt4/qt4.inc b/meta/recipes-qt/qt4/qt4.inc
index bec81b4..2638249 100644
--- a/meta/recipes-qt/qt4/qt4.inc
+++ b/meta/recipes-qt/qt4/qt4.inc
@@ -138,8 +138,8 @@ RRECOMMENDS_${QT_BASE_NAME}-qmlviewer += " \
 RRECOMMENDS_${QT_BASE_NAME}-doc += " \
             ${QT_BASE_NAME}-demos-doc"
 
-FILES_${QT_BASE_NAME}-tools                = "${bindir}/qttracereplay ${bindir}/qdoc*"
-FILES_${QT_BASE_NAME}-tools-dbg            = "${bindir}/.debug/qttracereplay ${bindir}/.debug/qdoc*"
+FILES_${QT_BASE_NAME}-tools                = "${bindir}/qttracereplay ${bindir}/qdoc* ${bindir}/moc ${bindir}/uic* ${bindir}/rcc"
+FILES_${QT_BASE_NAME}-tools-dbg            = "${bindir}/.debug/qttracereplay ${bindir}/.debug/qdoc* ${bindir}/.debug/uic* ${bindir}/.debug/moc ${bindir}/.debug/rcc"
 FILES_${QT_BASE_NAME}-assistant            = "${bindir}/*assistant* ${bindir}/qcollectiongenerator ${bindir}/qhelpconverter ${bindir}/qhelpgenerator"
 FILES_${QT_BASE_NAME}-assistant-dbg        = "${bindir}/.debug/*assistant* ${bindir}/.debug/qcollectiongenerator ${bindir}/.debug/qhelpconverter ${bindir}/.debug/qhelpgenerator"
 FILES_${QT_BASE_NAME}-common               = "${bindir}/qtconfig"
@@ -287,11 +287,6 @@ do_install() {
 
 	# These are host binaries, we should only use them in staging
 	rm ${D}/${bindir}/qmake
-	rm -f ${D}/${bindir}/uic
-	rm -f ${D}/${bindir}/uic3
-	rm ${D}/${bindir}/moc
-	rm ${D}/${bindir}/rcc
-	rm -f ${D}/${bindir}/lrelease
 
 	# fix pkgconfig, libtool and prl files
 	sed -i -e 's#-L${S}/lib/\?##g' \
-- 
1.7.3.4

Re: [PATCH 2/2] qt4.inc : Add uic, moc, rcc, and lrelease to tools package.

[Koen Kooi]

[-- Attachment #1: Type: text/plain, Size: 2080 bytes --]


Op 7 dec. 2011, om 16:59 heeft Philip Balister het volgende geschreven:

> These binaries are needed in the target tools package so we can build
> qt apps on the target. You also need to install qmake from meta-oe.
> 
> Test by building gnuradio on the USRP e100 with qt support.
> 
> Signed-off-by: Philip Balister <philip@balister.org>
> ---
> meta/recipes-qt/qt4/qt4.inc |    9 ++-------
> 1 files changed, 2 insertions(+), 7 deletions(-)
> 
> diff --git a/meta/recipes-qt/qt4/qt4.inc b/meta/recipes-qt/qt4/qt4.inc
> index bec81b4..2638249 100644
> --- a/meta/recipes-qt/qt4/qt4.inc
> +++ b/meta/recipes-qt/qt4/qt4.inc
> @@ -138,8 +138,8 @@ RRECOMMENDS_${QT_BASE_NAME}-qmlviewer += " \
> RRECOMMENDS_${QT_BASE_NAME}-doc += " \
>             ${QT_BASE_NAME}-demos-doc"
> 
> -FILES_${QT_BASE_NAME}-tools                = "${bindir}/qttracereplay ${bindir}/qdoc*"
> -FILES_${QT_BASE_NAME}-tools-dbg            = "${bindir}/.debug/qttracereplay ${bindir}/.debug/qdoc*"
> +FILES_${QT_BASE_NAME}-tools                = "${bindir}/qttracereplay ${bindir}/qdoc* ${bindir}/moc ${bindir}/uic* ${bindir}/rcc"
> +FILES_${QT_BASE_NAME}-tools-dbg            = "${bindir}/.debug/qttracereplay ${bindir}/.debug/qdoc* ${bindir}/.debug/uic* ${bindir}/.debug/moc ${bindir}/.debug/rcc"
> FILES_${QT_BASE_NAME}-assistant            = "${bindir}/*assistant* ${bindir}/qcollectiongenerator ${bindir}/qhelpconverter ${bindir}/qhelpgenerator"
> FILES_${QT_BASE_NAME}-assistant-dbg        = "${bindir}/.debug/*assistant* ${bindir}/.debug/qcollectiongenerator ${bindir}/.debug/qhelpconverter ${bindir}/.debug/qhelpgenerator"
> FILES_${QT_BASE_NAME}-common               = "${bindir}/qtconfig"
> @@ -287,11 +287,6 @@ do_install() {
> 
> 	# These are host binaries, we should only use them in staging
> 	rm ${D}/${bindir}/qmake
> -	rm -f ${D}/${bindir}/uic
> -	rm -f ${D}/${bindir}/uic3
> -	rm ${D}/${bindir}/moc
> -	rm ${D}/${bindir}/rcc
> -	rm -f ${D}/${bindir}/lrelease

Can you verify that those are actually target binaries and not x86 ones?

regards,

Koen 

[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 169 bytes --]

Re: [PATCH 2/2] qt4.inc : Add uic, moc, rcc, and lrelease to tools package.

[Philip Balister]

On 12/07/2011 11:23 AM, Koen Kooi wrote:
> 
> Op 7 dec. 2011, om 16:59 heeft Philip Balister het volgende geschreven:
> 
>> These binaries are needed in the target tools package so we can build
>> qt apps on the target. You also need to install qmake from meta-oe.
>>
>> Test by building gnuradio on the USRP e100 with qt support.
>>
>> Signed-off-by: Philip Balister <philip@balister.org>
>> ---
>> meta/recipes-qt/qt4/qt4.inc |    9 ++-------
>> 1 files changed, 2 insertions(+), 7 deletions(-)
>>
>> diff --git a/meta/recipes-qt/qt4/qt4.inc b/meta/recipes-qt/qt4/qt4.inc
>> index bec81b4..2638249 100644
>> --- a/meta/recipes-qt/qt4/qt4.inc
>> +++ b/meta/recipes-qt/qt4/qt4.inc
>> @@ -138,8 +138,8 @@ RRECOMMENDS_${QT_BASE_NAME}-qmlviewer += " \
>> RRECOMMENDS_${QT_BASE_NAME}-doc += " \
>>             ${QT_BASE_NAME}-demos-doc"
>>
>> -FILES_${QT_BASE_NAME}-tools                = "${bindir}/qttracereplay ${bindir}/qdoc*"
>> -FILES_${QT_BASE_NAME}-tools-dbg            = "${bindir}/.debug/qttracereplay ${bindir}/.debug/qdoc*"
>> +FILES_${QT_BASE_NAME}-tools                = "${bindir}/qttracereplay ${bindir}/qdoc* ${bindir}/moc ${bindir}/uic* ${bindir}/rcc"
>> +FILES_${QT_BASE_NAME}-tools-dbg            = "${bindir}/.debug/qttracereplay ${bindir}/.debug/qdoc* ${bindir}/.debug/uic* ${bindir}/.debug/moc ${bindir}/.debug/rcc"
>> FILES_${QT_BASE_NAME}-assistant            = "${bindir}/*assistant* ${bindir}/qcollectiongenerator ${bindir}/qhelpconverter ${bindir}/qhelpgenerator"
>> FILES_${QT_BASE_NAME}-assistant-dbg        = "${bindir}/.debug/*assistant* ${bindir}/.debug/qcollectiongenerator ${bindir}/.debug/qhelpconverter ${bindir}/.debug/qhelpgenerator"
>> FILES_${QT_BASE_NAME}-common               = "${bindir}/qtconfig"
>> @@ -287,11 +287,6 @@ do_install() {
>>
>> 	# These are host binaries, we should only use them in staging
>> 	rm ${D}/${bindir}/qmake
>> -	rm -f ${D}/${bindir}/uic
>> -	rm -f ${D}/${bindir}/uic3
>> -	rm ${D}/${bindir}/moc
>> -	rm ${D}/${bindir}/rcc
>> -	rm -f ${D}/${bindir}/lrelease
> 
> Can you verify that those are actually target binaries and not x86 ones?

root@usrp-e1xx:~# file /usr/bin/lrelease
/usr/bin/lrelease: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripped

root@usrp-e1xx:~# file /usr/bin/rcc
/usr/bin/rcc: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripp

root@usrp-e1xx:~# file /usr/bin/moc
/usr/bin/moc: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripped

root@usrp-e1xx:~# file /usr/bin/uic3
/usr/bin/uic3: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripp

root@usrp-e1xx:~# file /usr/bin/uic
/usr/bin/uic: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripped

Philip

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

Re: [PATCH 2/2] qt4.inc : Add uic, moc, rcc, and lrelease to tools package.

[Paul Eggleton]

On Wednesday 07 December 2011 11:44:10 Philip Balister wrote:
> On 12/07/2011 11:23 AM, Koen Kooi wrote:
> > Op 7 dec. 2011, om 16:59 heeft Philip Balister het volgende geschreven:
> >> These binaries are needed in the target tools package so we can build
> >> qt apps on the target. You also need to install qmake from meta-oe.
> >> 
> >> Test by building gnuradio on the USRP e100 with qt support.
> >> 
> >> Signed-off-by: Philip Balister <philip@balister.org>
> >> ---
> >> meta/recipes-qt/qt4/qt4.inc |    9 ++-------
> >> 1 files changed, 2 insertions(+), 7 deletions(-)
> >> 
> >> diff --git a/meta/recipes-qt/qt4/qt4.inc b/meta/recipes-qt/qt4/qt4.inc
> >> index bec81b4..2638249 100644
> >> --- a/meta/recipes-qt/qt4/qt4.inc
> >> +++ b/meta/recipes-qt/qt4/qt4.inc
> >> @@ -138,8 +138,8 @@ RRECOMMENDS_${QT_BASE_NAME}-qmlviewer += " \
> >> RRECOMMENDS_${QT_BASE_NAME}-doc += " \
> >> 
> >>             ${QT_BASE_NAME}-demos-doc"
> >> 
> >> -FILES_${QT_BASE_NAME}-tools                = "${bindir}/qttracereplay
> >> ${bindir}/qdoc*" -FILES_${QT_BASE_NAME}-tools-dbg            =
> >> "${bindir}/.debug/qttracereplay ${bindir}/.debug/qdoc*"
> >> +FILES_${QT_BASE_NAME}-tools                =
> >> "${bindir}/qttracereplay ${bindir}/qdoc* ${bindir}/moc ${bindir}/uic*
> >> ${bindir}/rcc" +FILES_${QT_BASE_NAME}-tools-dbg            =
> >> "${bindir}/.debug/qttracereplay ${bindir}/.debug/qdoc*
> >> ${bindir}/.debug/uic* ${bindir}/.debug/moc ${bindir}/.debug/rcc"
> >> FILES_${QT_BASE_NAME}-assistant            = "${bindir}/*assistant*
> >> ${bindir}/qcollectiongenerator ${bindir}/qhelpconverter
> >> ${bindir}/qhelpgenerator" FILES_${QT_BASE_NAME}-assistant-dbg       
> >> = "${bindir}/.debug/*assistant* ${bindir}/.debug/qcollectiongenerator
> >> ${bindir}/.debug/qhelpconverter ${bindir}/.debug/qhelpgenerator"
> >> FILES_${QT_BASE_NAME}-common               = "${bindir}/qtconfig" @@
> >> -287,11 +287,6 @@ do_install() {
> >> 
> >> 	# These are host binaries, we should only use them in staging
> >> 	rm ${D}/${bindir}/qmake
> >> 
> >> -	rm -f ${D}/${bindir}/uic
> >> -	rm -f ${D}/${bindir}/uic3
> >> -	rm ${D}/${bindir}/moc
> >> -	rm ${D}/${bindir}/rcc
> >> -	rm -f ${D}/${bindir}/lrelease
> > 
> > Can you verify that those are actually target binaries and not x86 ones?
> 
> root@usrp-e1xx:~# file /usr/bin/lrelease
> /usr/bin/lrelease: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
> dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripped
> 
> root@usrp-e1xx:~# file /usr/bin/rcc
> /usr/bin/rcc: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
> dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripp
> 
> root@usrp-e1xx:~# file /usr/bin/moc
> /usr/bin/moc: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
> dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripped
> 
> root@usrp-e1xx:~# file /usr/bin/uic3
> /usr/bin/uic3: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
> dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripp
> 
> root@usrp-e1xx:~# file /usr/bin/uic
> /usr/bin/uic: ELF 32-bit LSB executable, ARM, version 1 (SYSV),
> dynamically linked (uses shared libs), for GNU/Linux 2.6.16, stripped

I thought this sounded strange when we talked about it on IRC, and looking 
back through the history, commit 69eeb3d2276e5b10d084b47d308ecfc8daf8b467 (by 
me as it happens) added these lines because we were getting architecture 
warnings at packaging time for qt4-x11-free for these specific files. If that's 
no longer the case I'd like to understand why.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre

Re: [PATCH 1/2] boost : Add icu to DEPENDS to fix a build failure.

[Richard Purdie]

On Wed, 2011-12-07 at 10:59 -0500, Philip Balister wrote:
> Part of boost needs libraries from icu. Without this, either the build or
> image construction fails. (I forget exactly which)

That is a fairly heavy dependency to add and we're not seeing builds or
images fail due to this being missing as far as I can tell. Do you have
any further information on this?

Cheers,

Richard

> Signed-off-by: Philip Balister <philip@balister.org>
> ---
>  meta/recipes-support/boost/boost.inc |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/recipes-support/boost/boost.inc b/meta/recipes-support/boost/boost.inc
> index fecdb97..a1499f6 100644
> --- a/meta/recipes-support/boost/boost.inc
> +++ b/meta/recipes-support/boost/boost.inc
> @@ -6,7 +6,7 @@
>  DESCRIPTION = "Free peer-reviewed portable C++ source libraries"
>  HOMEPAGE = "http://www.boost.org/"
>  SECTION = "libs"
> -DEPENDS = "boost-native zlib"
> +DEPENDS = "boost-native zlib icu"
>  DEPENDS_virtclass-native = ""
>  LICENSE = "Boost"
>  

Re: [PATCH 1/2] boost : Add icu to DEPENDS to fix a build failure.

[Philip Balister]

On 12/07/2011 11:40 AM, Richard Purdie wrote:
> On Wed, 2011-12-07 at 10:59 -0500, Philip Balister wrote:
>> Part of boost needs libraries from icu. Without this, either the build or
>> image construction fails. (I forget exactly which)
> 
> That is a fairly heavy dependency to add and we're not seeing builds or
> images fail due to this being missing as far as I can tell. Do you have
> any further information on this?

Googling gets me this page:

http://www.boost.org/doc/libs/1_41_0/libs/regex/doc/html/boost_regex/ref/non_std_strings/icu/intro.html

I'm no expert at this unicode stuff, maybe we need to nuild boost
without unicode?

Philip


> 
> Cheers,
> 
> Richard
> 
>> Signed-off-by: Philip Balister <philip@balister.org>
>> ---
>>  meta/recipes-support/boost/boost.inc |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/meta/recipes-support/boost/boost.inc b/meta/recipes-support/boost/boost.inc
>> index fecdb97..a1499f6 100644
>> --- a/meta/recipes-support/boost/boost.inc
>> +++ b/meta/recipes-support/boost/boost.inc
>> @@ -6,7 +6,7 @@
>>  DESCRIPTION = "Free peer-reviewed portable C++ source libraries"
>>  HOMEPAGE = "http://www.boost.org/"
>>  SECTION = "libs"
>> -DEPENDS = "boost-native zlib"
>> +DEPENDS = "boost-native zlib icu"
>>  DEPENDS_virtclass-native = ""
>>  LICENSE = "Boost"
>>  
> 
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
> 

Re: [PATCH 1/2] boost : Add icu to DEPENDS to fix a build failure.

[Richard Purdie]

On Wed, 2011-12-07 at 11:49 -0500, Philip Balister wrote:
> On 12/07/2011 11:40 AM, Richard Purdie wrote:
> > On Wed, 2011-12-07 at 10:59 -0500, Philip Balister wrote:
> >> Part of boost needs libraries from icu. Without this, either the build or
> >> image construction fails. (I forget exactly which)
> > 
> > That is a fairly heavy dependency to add and we're not seeing builds or
> > images fail due to this being missing as far as I can tell. Do you have
> > any further information on this?
> 
> Googling gets me this page:
> 
> http://www.boost.org/doc/libs/1_41_0/libs/regex/doc/html/boost_regex/ref/non_std_strings/icu/intro.html
> 
> I'm no expert at this unicode stuff, maybe we need to nuild boost
> without unicode?

That sounds preferable since I can't see how we can be using it at
present anyway. I'm hoping we can just disable that bit of boost until
somebody needs it...

Cheers,

Richard