Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH 1/5] cmake: Add ugly hack from meta-qt5 to prevent cmake trying to detect qt5
@ 2014-04-02 12:25 Martin Jansa
  2014-04-02 12:25 ` [PATCH 2/5] ed: remove infodir Martin Jansa
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Martin Jansa @ 2014-04-02 12:25 UTC (permalink / raw)
  To: openembedded-core

* cmake doesn't have dependency on qt4/qt5, so these tests usually fail
  but still can cause undeterministic results or build failures (when
  OE_QMAKE_PATH_EXTERNAL_HOST_BINS is undefined or native qmake removed
  while running the test in cmake)

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-devtools/cmake/cmake.inc | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/meta/recipes-devtools/cmake/cmake.inc b/meta/recipes-devtools/cmake/cmake.inc
index 1d5303f..a0fcf49 100644
--- a/meta/recipes-devtools/cmake/cmake.inc
+++ b/meta/recipes-devtools/cmake/cmake.inc
@@ -21,6 +21,28 @@ SRC_URI = "http://www.cmake.org/files/v${CMAKE_MAJOR_VERSION}/cmake-${PV}.tar.gz
 
 inherit autotools-brokensep
 
+# Ugly hack to work around undefined OE_QMAKE_PATH_EXTERNAL_HOST_BINS variable
+# and possibly missing qmake binary (qtbase-native can be removed from sysroot
+# e.g. in order to upgrade it, even when there is target qtbase)
+
+# Fixes errors like this in cmake(-native).do_configure:
+#| -- Performing Test run_pic_test - Success
+#| CMake Error at tmp-eglibc/sysroots/qemuarm/usr/lib/cmake/Qt5Core/Qt5CoreConfig.cmake:27 (message):
+#|   The imported target "Qt5::Core" references the file
+#|
+#|      "/qmake"
+#|
+#|   but this file does not exist.  Possible reasons include:
+
+do_configure_prepend() {
+	sed -i 's/^find_package(Qt5Core QUIET)$/#find_package(Qt5Core QUIET)/g' ${S}/Tests/RunCMake/CMakeLists.txt
+	sed -i 's/^find_package(Qt5Core REQUIRED)/#find_package(Qt5Core REQUIRED)/g' ${S}/Tests/RunCMake/IncompatibleQt/IncompatibleQt.cmake
+	sed -i 's/^  find_package(Qt5Widgets REQUIRED)/#  find_package(Qt5Widgets REQUIRED)/g' ${S}/Tests/QtAutomoc/CMakeLists.txt
+	sed -i 's/^find_package(Qt5Core REQUIRED)/#find_package(Qt5Core REQUIRED)/g' ${S}/Tests/Qt4And5Automoc/CMakeLists.txt
+	sed -i 's/^  find_package(Qt5Widgets QUIET NO_MODULE)/#  find_package(Qt5Widgets QUIET NO_MODULE)/g' ${S}/Tests/CMakeLists.txt
+	sed -i 's/^find_package(Qt5Widgets QUIET)/#find_package(Qt5Widgets QUIET)/g' ${S}/Source/QtDialog/CMakeLists.txt
+}
+
 do_configure () {
 	./configure --prefix=${prefix}
 }
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/5] ed: remove infodir
  2014-04-02 12:25 [PATCH 1/5] cmake: Add ugly hack from meta-qt5 to prevent cmake trying to detect qt5 Martin Jansa
@ 2014-04-02 12:25 ` Martin Jansa
  2014-04-02 12:25 ` [PATCH 3/5] gcc-target: " Martin Jansa
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 7+ messages in thread
From: Martin Jansa @ 2014-04-02 12:25 UTC (permalink / raw)
  To: openembedded-core

* unlike 0.5 version, this doesn't use autotools.bbclass which removes infodir automatically
* fixes QA warning:
  ed-1.9: The /usr/share/info/dir file is not meant to be shipped in a particular package.

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-extended/ed/ed_1.9.bb | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-extended/ed/ed_1.9.bb b/meta/recipes-extended/ed/ed_1.9.bb
index 8ae99cc..d162154 100644
--- a/meta/recipes-extended/ed/ed_1.9.bb
+++ b/meta/recipes-extended/ed/ed_1.9.bb
@@ -22,4 +22,8 @@ do_configure() {
 
 do_install() {
 	oe_runmake 'DESTDIR=${D}' install
+	# Info dir listing isn't interesting at this point so remove it if it exists.
+	if [ -e "${D}${infodir}/dir" ]; then
+		rm -f ${D}${infodir}/dir
+	fi
 }
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 3/5] gcc-target: remove infodir
  2014-04-02 12:25 [PATCH 1/5] cmake: Add ugly hack from meta-qt5 to prevent cmake trying to detect qt5 Martin Jansa
  2014-04-02 12:25 ` [PATCH 2/5] ed: remove infodir Martin Jansa
@ 2014-04-02 12:25 ` Martin Jansa
  2014-04-02 17:06   ` Khem Raj
  2014-04-02 12:25 ` [PATCH 4/5] Revert "qt-mobility: remove /usr/lib from ld rpath-link option" Martin Jansa
  2014-04-02 12:25 ` [PATCH 5/5] send-error-report: show response Martin Jansa
  3 siblings, 1 reply; 7+ messages in thread
From: Martin Jansa @ 2014-04-02 12:25 UTC (permalink / raw)
  To: openembedded-core

* it uses autotools but doesn't call autotools_do_install
* fixes QA warning:
  gcc-4.8.2: The /usr/share/info/dir file is not meant to be shipped in a particular package.

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-devtools/gcc/gcc-target.inc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/meta/recipes-devtools/gcc/gcc-target.inc b/meta/recipes-devtools/gcc/gcc-target.inc
index e1179ac..3412aa8 100644
--- a/meta/recipes-devtools/gcc/gcc-target.inc
+++ b/meta/recipes-devtools/gcc/gcc-target.inc
@@ -104,6 +104,10 @@ FILES_${PN}-doc = "\
 
 do_install () {
 	oe_runmake 'DESTDIR=${D}' install-host
+	# Info dir listing isn't interesting at this point so remove it if it exists.
+	if [ -e "${D}${infodir}/dir" ]; then
+		rm -f ${D}${infodir}/dir
+	fi
 
 	# Cleanup some of the ${libdir}{,exec}/gcc stuff ...
 	rm -r ${D}${libdir}/gcc/${TARGET_SYS}/${BINV}/install-tools
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 4/5] Revert "qt-mobility: remove /usr/lib from ld rpath-link option"
  2014-04-02 12:25 [PATCH 1/5] cmake: Add ugly hack from meta-qt5 to prevent cmake trying to detect qt5 Martin Jansa
  2014-04-02 12:25 ` [PATCH 2/5] ed: remove infodir Martin Jansa
  2014-04-02 12:25 ` [PATCH 3/5] gcc-target: " Martin Jansa
@ 2014-04-02 12:25 ` Martin Jansa
  2014-04-02 12:25 ` [PATCH 5/5] send-error-report: show response Martin Jansa
  3 siblings, 0 replies; 7+ messages in thread
From: Martin Jansa @ 2014-04-02 12:25 UTC (permalink / raw)
  To: openembedded-core

* Basically part of the headers/libs are installed in
  ${D}(${libdir}/${includedir}) instead of
  ${D}(${libdir}/${includedir})/qt4

* http://lists.openembedded.org/pipermail/openembedded-core/2013-October/085815.html
  is related, but doesn't fix the issue completely, so better
  revert the commit which introduced this issue

[YOCTO #5414]

This reverts commit f7409a9fe83ba2535a43f39ed57cd78242a88557.

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-qt/qt4/qt-mobility_1.2.0.inc | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc b/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
index 72d361e..ff63d2b 100644
--- a/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
+++ b/meta/recipes-qt/qt4/qt-mobility_1.2.0.inc
@@ -91,12 +91,6 @@ EOF
 	echo CONFIG += ${qtm_embedded} >>./config.pri
 }
 
-do_compile_prepend() {
-	for NAME in `find ./ -name Makefile` ; do
-	    sed -e 's:-Wl,-rpath-link,${libdir}::g' -i $NAME 
-	done
-}
-
 do_install() {
 	for NAME in */*.prf
 	do
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 5/5] send-error-report: show response
  2014-04-02 12:25 [PATCH 1/5] cmake: Add ugly hack from meta-qt5 to prevent cmake trying to detect qt5 Martin Jansa
                   ` (2 preceding siblings ...)
  2014-04-02 12:25 ` [PATCH 4/5] Revert "qt-mobility: remove /usr/lib from ld rpath-link option" Martin Jansa
@ 2014-04-02 12:25 ` Martin Jansa
  3 siblings, 0 replies; 7+ messages in thread
From: Martin Jansa @ 2014-04-02 12:25 UTC (permalink / raw)
  To: openembedded-core

* useful when debuging why it was refused by server

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 scripts/send-error-report | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/scripts/send-error-report b/scripts/send-error-report
index 0d85776..8eea3cf 100755
--- a/scripts/send-error-report
+++ b/scripts/send-error-report
@@ -53,9 +53,11 @@ def sendData(json_file, server):
             print response.status, response.reason
             res = response.read()
             if response.status == 200:
-                print res
+                print(res)
             else:
-                print("There was a problem submiting your data")
+                print("There was a problem submiting your data, response written in %s.response.html" % json_file)
+                with open("%s.response.html" % json_file, "w") as f:
+                    f.write(res)
             conn.close()
         except:
                 print("Server connection failed")
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 3/5] gcc-target: remove infodir
  2014-04-02 12:25 ` [PATCH 3/5] gcc-target: " Martin Jansa
@ 2014-04-02 17:06   ` Khem Raj
  2014-04-02 17:20     ` Martin Jansa
  0 siblings, 1 reply; 7+ messages in thread
From: Khem Raj @ 2014-04-02 17:06 UTC (permalink / raw)
  To: Martin Jansa; +Cc: Patches and discussions about the oe-core layer

On Wed, Apr 2, 2014 at 5:25 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> +       # Info dir listing isn't interesting at this point so remove it if it exists.
> +       if [ -e "${D}${infodir}/dir" ]; then
> +               rm -f ${D}${infodir}/dir
> +       fi

why check if it exists when you are already using rm -f ?


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 3/5] gcc-target: remove infodir
  2014-04-02 17:06   ` Khem Raj
@ 2014-04-02 17:20     ` Martin Jansa
  0 siblings, 0 replies; 7+ messages in thread
From: Martin Jansa @ 2014-04-02 17:20 UTC (permalink / raw)
  To: Khem Raj; +Cc: Patches and discussions about the oe-core layer

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

On Wed, Apr 02, 2014 at 10:06:16AM -0700, Khem Raj wrote:
> On Wed, Apr 2, 2014 at 5:25 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> > +       # Info dir listing isn't interesting at this point so remove it if it exists.
> > +       if [ -e "${D}${infodir}/dir" ]; then
> > +               rm -f ${D}${infodir}/dir
> > +       fi
> 
> why check if it exists when you are already using rm -f ?

Just because I was copying snippet from autotools.bbclass

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2014-04-02 17:20 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-02 12:25 [PATCH 1/5] cmake: Add ugly hack from meta-qt5 to prevent cmake trying to detect qt5 Martin Jansa
2014-04-02 12:25 ` [PATCH 2/5] ed: remove infodir Martin Jansa
2014-04-02 12:25 ` [PATCH 3/5] gcc-target: " Martin Jansa
2014-04-02 17:06   ` Khem Raj
2014-04-02 17:20     ` Martin Jansa
2014-04-02 12:25 ` [PATCH 4/5] Revert "qt-mobility: remove /usr/lib from ld rpath-link option" Martin Jansa
2014-04-02 12:25 ` [PATCH 5/5] send-error-report: show response Martin Jansa

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox