[PATCH v2 0/5] Miscellaneous fixes: mainly qt- and DISTRO

Openembedded Core Discussions
 help / color / mirror / Atom feed

* [PATCH v2 0/5] Miscellaneous fixes: mainly qt- and DISTRO_FEATURES-related
@ 2011-08-17  6:41 Anders Darander
  2011-08-17  6:41 ` [PATCH v2 1/5] qt4: allow a reduction in build time Anders Darander
                   ` (4 more replies)
  0 siblings, 5 replies; 7+ messages in thread
From: Anders Darander @ 2011-08-17  6:41 UTC (permalink / raw)
  To: openembedded-core; +Cc: Anders Darander


A series with some miscellaneous fixes/changes.

Patches:
0001: make it possible to build qt4 while disabling certain parts of it.
0002: A partial sync of qmake_base.bbclass from meta-oe. Hopefully we should
      be able to delete qmake_base.bbclass from meta-oe after this sync.
0003-4: respect wifi and bluetooth in DISTRO_FEATURES while building connman 
	and ofono.
0005: increase the likelihood that a rename and modification is detected, 
      instead of a deleta/add-pair.

Patch 0005 is the patch that is most likely to be controversial. The other 
patches should be of minor impact, and result in a more consistent build 
environment (i.e. better respect for DISTRO_FEATURES).

V2: * remove unneccesary helper variable in the ofono-patch.
    No other changes.

The following changes since commit 13db5f420ca9bff98561f80d78958278734ad1f4:

  distro-tracking: Update info for gpgme, libassuan, apr... after last upgrade. (2011-08-15 15:26:04 +0100)

are available in the git repository at:
  git://github.com/darander/oe-core qt4-fixes
  https://github.com/darander/oe-core/tree/qt4-fixes

Anders Darander (5):
  qt4: allow a reduction in build time
  qmake_base: add uclibceabi
  connman: use DISTRO_FEATURES to enable wifi and bluetooth
  ofono: conditionally enable bluetooth
  create-pull-request: increase likelihood of detecting a rename

 meta/classes/qmake_base.bbclass                   |    2 +-
 meta/recipes-connectivity/connman/connman_0.75.bb |    4 ++--
 meta/recipes-connectivity/ofono/ofono.inc         |    2 +-
 meta/recipes-connectivity/ofono/ofono_0.50.bb     |    1 +
 meta/recipes-connectivity/ofono/ofono_git.bb      |    4 ++++
 meta/recipes-qt/qt4/qt4.inc                       |   13 ++++++++-----
 scripts/create-pull-request                       |    2 +-
 7 files changed, 18 insertions(+), 10 deletions(-)

-- 
1.7.5.4




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

* [PATCH v2 1/5] qt4: allow a reduction in build time
  2011-08-17  6:41 [PATCH v2 0/5] Miscellaneous fixes: mainly qt- and DISTRO_FEATURES-related Anders Darander
@ 2011-08-17  6:41 ` Anders Darander
  2011-08-17  6:41 ` [PATCH v2 2/5] qmake_base: add uclibceabi Anders Darander
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 7+ messages in thread
From: Anders Darander @ 2011-08-17  6:41 UTC (permalink / raw)
  To: openembedded-core

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 <anders@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
 }
-- 
1.7.5.4




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

* [PATCH v2 2/5] qmake_base: add uclibceabi
  2011-08-17  6:41 [PATCH v2 0/5] Miscellaneous fixes: mainly qt- and DISTRO_FEATURES-related Anders Darander
  2011-08-17  6:41 ` [PATCH v2 1/5] qt4: allow a reduction in build time Anders Darander
@ 2011-08-17  6:41 ` Anders Darander
  2011-08-17  6:41 ` [PATCH v2 3/5] connman: use DISTRO_FEATURES to enable wifi and bluetooth Anders Darander
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 7+ messages in thread
From: Anders Darander @ 2011-08-17  6:41 UTC (permalink / raw)
  To: openembedded-core

Partial sync of qmake_base from meta-oe.
* Add a matching rule for *linux-uclibceabi-oe-g++, to make qmake_base.bbclase more
similar to the version in meta-oe.

Signed-off-by: Anders Darander <anders@chargestorm.se>
---
 meta/classes/qmake_base.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/classes/qmake_base.bbclass b/meta/classes/qmake_base.bbclass
index 165d689..b61b84c 100644
--- a/meta/classes/qmake_base.bbclass
+++ b/meta/classes/qmake_base.bbclass
@@ -47,7 +47,7 @@ addtask generate_qt_config_file after do_patch before do_configure
 
 qmake_base_do_configure() {
 	case ${QMAKESPEC} in
-	*linux-oe-g++|*linux-uclibc-oe-g++|*linux-gnueabi-oe-g++)
+	*linux-oe-g++|*linux-uclibc-oe-g++|*linux-gnueabi-oe-g++|*linux-uclibceabi-oe-g++)
 		;;
 	*-oe-g++)
 		die Unsupported target ${TARGET_OS} for oe-g++ qmake spec
-- 
1.7.5.4




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

* [PATCH v2 3/5] connman: use DISTRO_FEATURES to enable wifi and bluetooth
  2011-08-17  6:41 [PATCH v2 0/5] Miscellaneous fixes: mainly qt- and DISTRO_FEATURES-related Anders Darander
  2011-08-17  6:41 ` [PATCH v2 1/5] qt4: allow a reduction in build time Anders Darander
  2011-08-17  6:41 ` [PATCH v2 2/5] qmake_base: add uclibceabi Anders Darander
@ 2011-08-17  6:41 ` Anders Darander
  2011-08-17  6:41 ` [PATCH v2 4/5] ofono: conditionally enable bluetooth Anders Darander
  2011-08-17  6:41 ` [PATCH v2 5/5] create-pull-request: increase likelihood of detecting a rename Anders Darander
  4 siblings, 0 replies; 7+ messages in thread
From: Anders Darander @ 2011-08-17  6:41 UTC (permalink / raw)
  To: openembedded-core

As wifi and bluetooth is set in DISTRO_FEATURES, connman should respect this during configuring.

Signed-off-by: Anders Darander <anders@chargestorm.se>
---
 meta/recipes-connectivity/connman/connman_0.75.bb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-connectivity/connman/connman_0.75.bb b/meta/recipes-connectivity/connman/connman_0.75.bb
index aeff439..5a7b284 100644
--- a/meta/recipes-connectivity/connman/connman_0.75.bb
+++ b/meta/recipes-connectivity/connman/connman_0.75.bb
@@ -8,8 +8,8 @@ EXTRA_OECONF += "\
   --enable-threads \
   --enable-loopback \
   --enable-ethernet \
-  --enable-wifi \
-  --enable-bluetooth \
+  ${@base_contains('DISTRO_FEATURES', 'wifi','--enable-wifi', '--disable-wifi', d)} \
+  ${@base_contains('DISTRO_FEATURES', 'bluetooth','--enable-bluetooth', '--disable-bluetooth', d)} \
   --enable-dnsproxy \
   --disable-dhclient \
   --enable-test \
-- 
1.7.5.4




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

* [PATCH v2 4/5] ofono: conditionally enable bluetooth
  2011-08-17  6:41 [PATCH v2 0/5] Miscellaneous fixes: mainly qt- and DISTRO_FEATURES-related Anders Darander
                   ` (2 preceding siblings ...)
  2011-08-17  6:41 ` [PATCH v2 3/5] connman: use DISTRO_FEATURES to enable wifi and bluetooth Anders Darander
@ 2011-08-17  6:41 ` Anders Darander
  2011-08-17 14:19   ` Richard Purdie
  2011-08-17  6:41 ` [PATCH v2 5/5] create-pull-request: increase likelihood of detecting a rename Anders Darander
  4 siblings, 1 reply; 7+ messages in thread
From: Anders Darander @ 2011-08-17  6:41 UTC (permalink / raw)
  To: openembedded-core

Use DISTRO_FEATURE to conditionally depend on bluez4, as well as enabling bluetooth.

Signed-off-by: Anders Darander <anders@chargestorm.se>
---
 meta/recipes-connectivity/ofono/ofono.inc     |    2 +-
 meta/recipes-connectivity/ofono/ofono_0.50.bb |    1 +
 meta/recipes-connectivity/ofono/ofono_git.bb  |    4 ++++
 3 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-connectivity/ofono/ofono.inc b/meta/recipes-connectivity/ofono/ofono.inc
index a14b35a..0b1a6e1 100644
--- a/meta/recipes-connectivity/ofono/ofono.inc
+++ b/meta/recipes-connectivity/ofono/ofono.inc
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=eb723b61539feef013de476e68b5c50a \
 
 inherit autotools pkgconfig update-rc.d
 
-DEPENDS  = "dbus glib-2.0 udev bluez4"
+DEPENDS  = "dbus glib-2.0 udev ${@base_contains('DISTRO_FEATURES', 'bluetooth','bluez4', '', d)}"
 
 INITSCRIPT_NAME = "ofono"
 INITSCRIPT_PARAMS = "defaults 22"
diff --git a/meta/recipes-connectivity/ofono/ofono_0.50.bb b/meta/recipes-connectivity/ofono/ofono_0.50.bb
index ef4fbf0..b679200 100644
--- a/meta/recipes-connectivity/ofono/ofono_0.50.bb
+++ b/meta/recipes-connectivity/ofono/ofono_0.50.bb
@@ -7,6 +7,7 @@ SRC_URI  = "${KERNELORG_MIRROR}/linux/network/ofono/${BPN}-${PV}.tar.bz2 \
 
 EXTRA_OECONF += "\
     --enable-test \
+    ${@base_contains('DISTRO_FEATURES', 'bluetooth','--enable-bluetooth', '--disable-bluetooth', d)} \
 "
 
 SRC_URI[md5sum] = "b2656fd0bbf33f926fc86c1e8915d697"
diff --git a/meta/recipes-connectivity/ofono/ofono_git.bb b/meta/recipes-connectivity/ofono/ofono_git.bb
index dbad5a1..f1770c2 100644
--- a/meta/recipes-connectivity/ofono/ofono_git.bb
+++ b/meta/recipes-connectivity/ofono/ofono_git.bb
@@ -10,6 +10,10 @@ PR = "r1"
 SRC_URI  = "git://git.kernel.org/pub/scm/network/ofono/ofono.git;protocol=git \
 	    file://ofono"
 
+EXTRA_OECONF += "\
+    ${@base_contains('DISTRO_FEATURES', 'bluetooth','--enable-bluetooth', '--disable-bluetooth', d)} \
+"
+
 do_configure_prepend () {
   ${S}/bootstrap
 }
-- 
1.7.5.4




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

* Re: [PATCH v2 4/5] ofono: conditionally enable bluetooth
  2011-08-17  6:41 ` [PATCH v2 4/5] ofono: conditionally enable bluetooth Anders Darander
@ 2011-08-17 14:19   ` Richard Purdie
  0 siblings, 0 replies; 7+ messages in thread
From: Richard Purdie @ 2011-08-17 14:19 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Wed, 2011-08-17 at 08:41 +0200, Anders Darander wrote:
> Use DISTRO_FEATURE to conditionally depend on bluez4, as well as enabling bluetooth.
> 
> Signed-off-by: Anders Darander <anders@chargestorm.se>
> ---
>  meta/recipes-connectivity/ofono/ofono.inc     |    2 +-
>  meta/recipes-connectivity/ofono/ofono_0.50.bb |    1 +
>  meta/recipes-connectivity/ofono/ofono_git.bb  |    4 ++++
>  3 files changed, 6 insertions(+), 1 deletions(-)

Merged to master, thanks.

Richard




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

* [PATCH v2 5/5] create-pull-request: increase likelihood of detecting a rename
  2011-08-17  6:41 [PATCH v2 0/5] Miscellaneous fixes: mainly qt- and DISTRO_FEATURES-related Anders Darander
                   ` (3 preceding siblings ...)
  2011-08-17  6:41 ` [PATCH v2 4/5] ofono: conditionally enable bluetooth Anders Darander
@ 2011-08-17  6:41 ` Anders Darander
  4 siblings, 0 replies; 7+ messages in thread
From: Anders Darander @ 2011-08-17  6:41 UTC (permalink / raw)
  To: openembedded-core

Decrease the similarity percentage needed to recognize a delete/add-pair followed by an edit,
as a rename.
This make reviewing patches easier.

Signed-off-by: Anders Darander <anders@chargestorm.se>
---
 scripts/create-pull-request |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/scripts/create-pull-request b/scripts/create-pull-request
index 7d021d3..600fa7a 100755
--- a/scripts/create-pull-request
+++ b/scripts/create-pull-request
@@ -166,7 +166,7 @@ mkdir $ODIR
 
 
 # Generate the patches and cover letter
-git format-patch -M --subject-prefix="$PREFIX" -n -o $ODIR --thread=shallow --cover-letter $RELATIVE_TO..$COMMIT_ID > /dev/null
+git format-patch -M40 --subject-prefix="$PREFIX" -n -o $ODIR --thread=shallow --cover-letter $RELATIVE_TO..$COMMIT_ID > /dev/null
 
 
 # Customize the cover letter
-- 
1.7.5.4




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

end of thread, other threads:[~2011-08-17 14:24 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-17  6:41 [PATCH v2 0/5] Miscellaneous fixes: mainly qt- and DISTRO_FEATURES-related Anders Darander
2011-08-17  6:41 ` [PATCH v2 1/5] qt4: allow a reduction in build time Anders Darander
2011-08-17  6:41 ` [PATCH v2 2/5] qmake_base: add uclibceabi Anders Darander
2011-08-17  6:41 ` [PATCH v2 3/5] connman: use DISTRO_FEATURES to enable wifi and bluetooth Anders Darander
2011-08-17  6:41 ` [PATCH v2 4/5] ofono: conditionally enable bluetooth Anders Darander
2011-08-17 14:19   ` Richard Purdie
2011-08-17  6:41 ` [PATCH v2 5/5] create-pull-request: increase likelihood of detecting a rename Anders Darander

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