Skip to contentSkip to conversation

qt3: fix bug 1348

PATCH In review Xiaofeng Yan 2 participants 3 messages Aug 12, 2011 – Aug 15, 2011

lore
Share & tools
lore
#0 lore raw

cover qt3: fix bug 1348

From: Xiaofeng Yan <[email protected]>

Hi Richard,
I modified this bug according to your suggestion. If you have any suggestion, I will modify it.

Pull URL: git://git.pokylinux.org/meta-qt3.git
Branch: xiaofeng/meta-qt3
Browse: http://git.yoctoproject.org/cgit/cgit.cgi/meta-qt3/commit/?h=xiaofeng/meta-qt3

Thanks,
Xiaofeng Yan <[email protected]>
---

Xiaofeng Yan (1):
qt3: fix bug 1348

 recipes-qt3/qt3/qt-x11-free-common.inc |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)
#1 lore raw

PATCH 1/1 qt3: fix bug 1348

From: Xiaofeng Yan <[email protected]>

[YOCTO #1348]

The variable is different when building qt-x11-free-native and qt-x11-free. The different cause this bug.

$ bitbake qt-x11-free-native -e | grep ^TARGET_OS
TARGET_OS="linux"
$ bitbake qt-x11-free -e | grep ^TARGET_OS
TARGET_OS="linux-gnueabi"

So I add a task function before do_configure for linking ${TARGET_OS} to linux-g++.

---
 recipes-qt3/qt3/qt-x11-free-common.inc |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)
recipes-qt3/qt3/qt-x11-free-common.inc +7 -1
index e486c52..9e6cc81 100644
@@ -3,7 +3,7 @@ SECTION = "x11/libs"
 PRIORITY = "optional"
 LICENSE = "GPL | QPL"
 HOMEPAGE = "http://www.trolltech.com"
-PR = "r0"
+PR = "r1"
 
 S = "${WORKDIR}/qt-x11-free-${PV}"
 
@@ -24,6 +24,12 @@ EXTRA_ENV = 'QMAKE="${STAGING_BINDIR_NATIVE}/qmake -after INCPATH+=${STAGING_INC
              AR="${TARGET_PREFIX}ar cqs" \
              MOC="${STAGING_BINDIR_NATIVE}/moc3" UIC="${STAGING_BINDIR_NATIVE}/uic3" MAKE="make -e"'
 
+do_configure_prepend() {
+    if [ ! -L ${QMAKE_MKSPEC_PATH}/${TARGET_OS}-oe-g++ ]; then
+        ln -sf ${QMAKE_MKSPEC_PATH}/linux-g++ ${QMAKE_MKSPEC_PATH}/${TARGET_OS}-oe-g++
+    fi
+}
+
 do_configure() {
 	echo "yes" | ./configure -prefix ${prefix} ${QT_CONFIG_FLAGS} -no-fast \
 		-L${STAGING_LIBDIR} -I${STAGING_INCDIR} -I${STAGING_INCDIR}/freetype2 -I${STAGING_INCDIR}/mysql
Signature · 1 line
1.7.0.4
#2 lore raw
Quoted 34 lines · Xiaofeng Yan

From: Xiaofeng Yan <[email protected]>

[YOCTO #1348]

The variable is different when building qt-x11-free-native and qt-x11-free. The different cause this bug.

$ bitbake qt-x11-free-native -e | grep ^TARGET_OS
TARGET_OS="linux"
$ bitbake qt-x11-free -e | grep ^TARGET_OS
TARGET_OS="linux-gnueabi"

So I add a task function before do_configure for linking ${TARGET_OS} to linux-g++.

---
 recipes-qt3/qt3/qt-x11-free-common.inc |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)
recipes-qt3/qt3/qt-x11-free-common.inc +1 -1
index e486c52..9e6cc81 100644
@@ -3,7 +3,7 @@ SECTION = "x11/libs"
 PRIORITY = "optional"
 LICENSE = "GPL | QPL"
 HOMEPAGE = "http://www.trolltech.com"
-PR = "r0"
+PR = "r1"
 
 S = "${WORKDIR}/qt-x11-free-${PV}"
 
@@ -24,6 +24,12 @@ EXTRA_ENV = 'QMAKE="${STAGING_BINDIR_NATIVE}/qmake -after INCPATH+=${STAGING_INC
              AR="${TARGET_PREFIX}ar cqs" \
              MOC="${STAGING_BINDIR_NATIVE}/moc3" UIC="${STAGING_BINDIR_NATIVE}/uic3" MAKE="make -e"'

recipes-qt3/qt3/qt-x11-free-common.inc EXTRA_ENV = 'QMAKE="${STAGING_BINDIR_NATIVE}/qmake -after INCPATH+=${STAGING_INC · line 35

Quoted 10 lines
recipes-qt3/qt3/qt-x11-free-common.inc +6 -0
 
+do_configure_prepend() {
+    if [ ! -L ${QMAKE_MKSPEC_PATH}/${TARGET_OS}-oe-g++ ]; then
+        ln -sf ${QMAKE_MKSPEC_PATH}/linux-g++ ${QMAKE_MKSPEC_PATH}/${TARGET_OS}-oe-g++
+    fi
+}
+
 do_configure() {
 	echo "yes" | ./configure -prefix ${prefix} ${QT_CONFIG_FLAGS} -no-fast \
 		-L${STAGING_LIBDIR} -I${STAGING_INCDIR} -I${STAGING_INCDIR}/freetype2 -I${STAGING_INCDIR}/mysql

Thanks, this looks like a better fix to me. I've merged this with an updated commit message. The commit message summary needs to state what the problem is, not just a bug number.

Cheers,

Richard