* [PATCH] connman: update to 0.78
@ 2011-12-09 10:51 Otavio Salvador
2011-12-09 14:18 ` Richard Purdie
0 siblings, 1 reply; 4+ messages in thread
From: Otavio Salvador @ 2011-12-09 10:51 UTC (permalink / raw)
To: openembedded-core
Updates the package to 0.78 bringing changes done on meta-oe to
OE-Core. This also removes unrequired build-depends and makes the
package to respect wifi and bluetooth distro features.
Following patches were removed:
* xtables_version.patch: merged upstream
* dbusperms.patch: dropped in favour of using sed to make the change
in do_compile_append
Another important change is the addition of ppp in depends that is
used by some plugins.
This uses PR as "r1" since it was previously available on meta-oe.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
Acked-by: Joshua Lock <josh@linux.intel.com>
---
.../connman-0.75/add_xuser_dbus_permission.patch | 23 -----
.../connman/connman-0.75/connman | 62 --------------
.../connman/connman-0.75/dbusperms.patch | 14 ---
.../connman/connman-0.75/xtables_version.patch | 64 --------------
meta/recipes-connectivity/connman/connman.inc | 90 ++++++++++++++------
.../connman/add_xuser_dbus_permission.patch | 23 +++++
meta/recipes-connectivity/connman/connman/connman | 62 ++++++++++++++
meta/recipes-connectivity/connman/connman_0.75.bb | 29 ------
meta/recipes-connectivity/connman/connman_0.78.bb | 10 ++
meta/recipes-connectivity/connman/connman_git.bb | 11 ---
10 files changed, 157 insertions(+), 231 deletions(-)
delete mode 100644 meta/recipes-connectivity/connman/connman-0.75/add_xuser_dbus_permission.patch
delete mode 100644 meta/recipes-connectivity/connman/connman-0.75/connman
delete mode 100644 meta/recipes-connectivity/connman/connman-0.75/dbusperms.patch
delete mode 100644 meta/recipes-connectivity/connman/connman-0.75/xtables_version.patch
create mode 100644 meta/recipes-connectivity/connman/connman/add_xuser_dbus_permission.patch
create mode 100644 meta/recipes-connectivity/connman/connman/connman
delete mode 100644 meta/recipes-connectivity/connman/connman_0.75.bb
create mode 100644 meta/recipes-connectivity/connman/connman_0.78.bb
delete mode 100644 meta/recipes-connectivity/connman/connman_git.bb
diff --git a/meta/recipes-connectivity/connman/connman-0.75/add_xuser_dbus_permission.patch b/meta/recipes-connectivity/connman/connman-0.75/add_xuser_dbus_permission.patch
deleted file mode 100644
index 764c689..0000000
--- a/meta/recipes-connectivity/connman/connman-0.75/add_xuser_dbus_permission.patch
+++ /dev/null
@@ -1,23 +0,0 @@
-Some platform (like atom-pc) enables rootless X,
-thus we need to add the xuser in the list.
-
-Upstream-Status: Inappropriate [configuration]
-
-Signed-off-by: Dongxiao Xu <dongxiao.xu@intel.com>
-
-diff -ruN connman-0.65-orig/src/connman-dbus.conf connman-0.65/src/connman-dbus.conf
---- connman-0.65-orig/src/connman-dbus.conf 2011-03-04 09:34:49.000000000 +0800
-+++ connman-0.65/src/connman-dbus.conf 2011-03-04 09:35:21.000000000 +0800
-@@ -7,6 +7,12 @@
- <allow send_interface="net.connman.Agent"/>
- <allow send_interface="net.connman.Counter"/>
- </policy>
-+ <policy user="xuser">
-+ <allow own="net.connman"/>
-+ <allow send_destination="net.connman"/>
-+ <allow send_interface="net.connman.Agent"/>
-+ <allow send_interface="net.connman.Counter"/>
-+ </policy>
- <policy at_console="true">
- <allow send_destination="net.connman"/>
- </policy>
diff --git a/meta/recipes-connectivity/connman/connman-0.75/connman b/meta/recipes-connectivity/connman/connman-0.75/connman
deleted file mode 100644
index f01bf37..0000000
--- a/meta/recipes-connectivity/connman/connman-0.75/connman
+++ /dev/null
@@ -1,62 +0,0 @@
-#!/bin/sh
-
-DAEMON=/usr/sbin/connmand
-PIDFILE=/var/run/connmand.pid
-DESC="Connection Manager"
-
-if [ -f /etc/default/connman ] ; then
- . /etc/default/connman
-fi
-
-set -e
-
-nfsroot=0
-
-exec 9<&0 < /proc/mounts
-while read dev mtpt fstype rest; do
- if test $mtpt = "/" ; then
- case $fstype in
- nfs | nfs4)
- nfsroot=1
- break
- ;;
- *)
- ;;
- esac
- fi
-done
-
-do_start() {
- EXTRA_PARAM=""
- if test $nfsroot -eq 1 ; then
- EXTRA_PARAM="-P ethernet"
- fi
- $DAEMON $EXTRA_PARAM
-}
-
-do_stop() {
- start-stop-daemon --stop --name connmand --quiet
-}
-
-case "$1" in
- start)
- echo "Starting $DESC"
- do_start
- ;;
- stop)
- echo "Stopping $DESC"
- do_stop
- ;;
- restart|force-reload)
- echo "Restarting $DESC"
- do_stop
- sleep 1
- do_start
- ;;
- *)
- echo "Usage: $0 {start|stop|restart|force-reload}" >&2
- exit 1
- ;;
-esac
-
-exit 0
diff --git a/meta/recipes-connectivity/connman/connman-0.75/dbusperms.patch b/meta/recipes-connectivity/connman/connman-0.75/dbusperms.patch
deleted file mode 100644
index c331654..0000000
--- a/meta/recipes-connectivity/connman/connman-0.75/dbusperms.patch
+++ /dev/null
@@ -1,14 +0,0 @@
-Upstream-Status: Inappropriate [configuration]
-
-Index: git/src/connman-dbus.conf
-===================================================================
---- git.orig/src/connman-dbus.conf 2009-05-26 00:34:35.000000000 +0100
-+++ git/src/connman-dbus.conf 2009-05-26 00:34:48.000000000 +0100
-@@ -10,6 +10,6 @@
- <allow send_destination="org.moblin.connman"/>
- </policy>
- <policy context="default">
-- <deny send_destination="org.moblin.connman"/>
-+ <allow send_destination="org.moblin.connman"/>
- </policy>
- </busconfig>
diff --git a/meta/recipes-connectivity/connman/connman-0.75/xtables_version.patch b/meta/recipes-connectivity/connman/connman-0.75/xtables_version.patch
deleted file mode 100644
index 13e2c36..0000000
--- a/meta/recipes-connectivity/connman/connman-0.75/xtables_version.patch
+++ /dev/null
@@ -1,64 +0,0 @@
-xtables_merge_options API change, fix its calling in connman.
-
-Signed-off-by: Dongxiao Xu <dongxiao.xu@intel.com>
-
-Upstream-Status: Backport
-
-diff -ruN connman-0.75-orig//src/iptables.c connman-0.75/src/iptables.c
---- connman-0.75-orig//src/iptables.c 2011-02-22 12:16:25.000000000 +0800
-+++ connman-0.75/src/iptables.c 2011-10-26 09:21:33.941164000 +0800
-@@ -1089,7 +1089,11 @@
- if (xt_t->init != NULL)
- xt_t->init(xt_t->t);
- iptables_globals.opts =
-- xtables_merge_options(iptables_globals.opts,
-+ xtables_merge_options(
-+#if XTABLES_VERSION_CODE > 5
-+ iptables_globals.orig_opts,
-+#endif
-+ iptables_globals.opts,
- xt_t->extra_opts,
- &xt_t->option_offset);
- if (iptables_globals.opts == NULL)
-@@ -1113,7 +1117,11 @@
- xt_m->init(xt_m->m);
- if (xt_m != xt_m->next) {
- iptables_globals.opts =
-- xtables_merge_options(iptables_globals.opts,
-+ xtables_merge_options(
-+#if XTABLES_VERSION_CODE > 5
-+ iptables_globals.orig_opts,
-+#endif
-+ iptables_globals.opts,
- xt_m->extra_opts,
- &xt_m->option_offset);
- if (iptables_globals.opts == NULL)
-diff -ruN connman-0.75-orig//tools/iptables-test.c connman-0.75/tools/iptables-test.c
---- connman-0.75-orig//tools/iptables-test.c 2011-02-22 12:16:25.000000000 +0800
-+++ connman-0.75/tools/iptables-test.c 2011-10-26 09:23:25.701164000 +0800
-@@ -1076,7 +1076,11 @@
- if (xt_t->init != NULL)
- xt_t->init(xt_t->t);
- connman_iptables_globals.opts =
-- xtables_merge_options(connman_iptables_globals.opts,
-+ xtables_merge_options(
-+#if XTABLES_VERSION_CODE > 5
-+ connman_iptables_globals.orig_opts,
-+#endif
-+ connman_iptables_globals.opts,
- xt_t->extra_opts,
- &xt_t->option_offset);
- if (connman_iptables_globals.opts == NULL)
-@@ -1102,7 +1106,11 @@
- xt_m->init(xt_m->m);
- if (xt_m != xt_m->next) {
- connman_iptables_globals.opts =
-- xtables_merge_options(connman_iptables_globals.opts,
-+ xtables_merge_options(
-+#if XTABLES_VERSION_CODE > 5
-+ connman_iptables_globals.orig_opts,
-+#endif
-+ connman_iptables_globals.opts,
- xt_m->extra_opts,
- &xt_m->option_offset);
- if (connman_iptables_globals.opts == NULL)
diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
index 7ee460e..938d074 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -13,7 +13,30 @@ LICENSE = "GPLv2"
LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e \
file://src/main.c;beginline=1;endline=20;md5=4b55b550fa6b33cc2055ef30dd262b3e"
-DEPENDS = "libgdbus dbus glib-2.0 iptables"
+# we need to define the depends here, the dynamic stuff is too late
+DEPENDS = "dbus glib-2.0 ppp iptables gnutls ntp \
+ ${@base_contains('DISTRO_FEATURES', 'bluetooth','bluez4', '', d)} \
+ ${@base_contains('DISTRO_FEATURES', 'wifi','wpa-supplicant', '', d)} \
+ "
+
+EXTRA_OECONF += "\
+ ac_cv_path_WPASUPPLICANT=${sbindir}/wpa_supplicant \
+ ac_cv_path_PPPD=${sbindir}/pppd \
+ --disable-gtk-doc \
+ --enable-debug \
+ --enable-threads \
+ --enable-loopback \
+ --enable-ethernet \
+ ${@base_contains('DISTRO_FEATURES', 'wifi','--enable-wifi', '--disable-wifi', d)} \
+ ${@base_contains('DISTRO_FEATURES', 'bluetooth','--enable-bluetooth', '--disable-bluetooth', d)} \
+ --enable-ofono \
+ --enable-tools \
+ --disable-polkit \
+ --enable-client \
+ --enable-fake \
+ --enable-ntpd \
+ --with-ntpd=${bindir}/ntpd \
+"
INITSCRIPT_NAME = "connman"
INITSCRIPT_PARAMS = "start 05 5 2 3 . stop 22 0 1 6 ."
@@ -25,44 +48,55 @@ USERADD_PARAM_${PN} = "--system --no-create-home \
inherit autotools pkgconfig update-rc.d useradd
-do_install_append() {
- install -d ${D}${sysconfdir}/init.d/
- install -m 0755 ${WORKDIR}/connman ${D}${sysconfdir}/init.d/connman
+do_compile_append() {
+ sed -i -e s:deny:allow:g src/connman-dbus.conf
}
-PACKAGES += "${PN}-scripts"
-RDEPENDS_${PN}-scripts += "python"
-
-PACKAGES_DYNAMIC = "${PN}-plugin-*"
-FILES_${PN} = "${bindir}/* ${sbindir}/* ${libexecdir}/* ${libdir}/lib*.so.* \
- ${sysconfdir} ${sharedstatedir} ${localstatedir} \
- ${base_bindir}/* ${base_sbindir}/* ${base_libdir}/*.so* ${datadir}/${PN} \
- ${datadir}/pixmaps ${datadir}/applications \
- ${datadir}/idl ${datadir}/omf ${datadir}/sounds \
- ${libdir}/bonobo/servers \
- ${datadir}/dbus-1/system-services/* \
- ${libdir}/connman/scripts/dhclient*"
-FILES_${PN}-dbg += "${libdir}/connman/plugins/.debug \
- ${libdir}/connman/scripts/.debug"
-
-FILES_${PN}-dev += "${libdir}/connman/plugins/*.la"
+do_install_append() {
+ install -d ${D}${sysconfdir}/init.d
+ install -m 0755 ${WORKDIR}/connman ${D}${sysconfdir}/init.d/connman
-FILES_${PN}-scripts += "${libdir}/${PN}/test/*"
+ install -d ${D}${bindir}
+ install -m 0755 ${S}/tools/*-test ${D}${bindir}
+ install -m 0755 ${S}/tools/wispr ${D}${bindir}
+}
python populate_packages_prepend() {
- prefix = (d.getVar("MLPREFIX", True) or "")
- depmap = dict( wifi=prefix+"wpa-supplicant", bluetooth=prefix+"bluez4", ofono=prefix+"ofono" )
+ depmap = dict( pppd="ppp",
+ udhcp="busybox connman-scripts",
+ dhclient="dhcp-client",
+ wifi="wpa-supplicant",
+ resolvconf="resolvconf",
+ bluetooth="bluez4",
+ ntpd="ntp",
+ ofono="ofono" )
packages = []
hook = lambda file,pkg,b,c,d:packages.append((file,pkg))
-
- plugin_dir = bb.data.expand('${libdir}/connman/plugins/', d)
- plugin_name = bb.data.expand('${PN}-plugin-%s', d)
-
+ plugin_dir = d.expand('${libdir}/connman/plugins/')
+ plugin_name = d.expand('${PN}-plugin-%s')
do_split_packages(d, plugin_dir, '^(.*).so$', plugin_name, '${PN} plugin for %s', extra_depends='', hook=hook )
-
for (file, package) in packages:
plugintype = package.split( '-' )[-1]
if plugintype in depmap:
+ rdepends = d.getVar( "RDEPENDS_%s" % package)
bb.note( "Adding rdependency on %s to package %s" % ( depmap[plugintype], package ) )
d.setVar("RDEPENDS_%s" % package, depmap[plugintype])
}
+
+PACKAGES_DYNAMIC = "${PN}-plugin-*"
+
+PACKAGES =+ "${PN}-scripts ${PN}-tools ${PN}-tests"
+
+FILES_${PN}-tools = "${bindir}/wispr"
+FILES_${PN}-tests = "${bindir}/*-test"
+FILES_${PN} = "${bindir}/* ${sbindir}/* ${libexecdir}/* ${libdir}/lib*.so.* \
+ ${sysconfdir} ${sharedstatedir} ${localstatedir} \
+ ${base_bindir}/* ${base_sbindir}/* ${base_libdir}/*.so* ${datadir}/${PN} \
+ ${datadir}/pixmaps ${datadir}/applications \
+ ${datadir}/idl ${datadir}/omf ${datadir}/sounds \
+ ${libdir}/bonobo/servers \
+ ${datadir}/dbus-1/system-services/*"
+
+FILES_${PN}-scripts += "${libdir}/connman/scripts"
+FILES_${PN}-dbg += "${libdir}/connman/*/.debug"
+FILES_${PN}-dev += "${libdir}/connman/*/*.la"
diff --git a/meta/recipes-connectivity/connman/connman/add_xuser_dbus_permission.patch b/meta/recipes-connectivity/connman/connman/add_xuser_dbus_permission.patch
new file mode 100644
index 0000000..764c689
--- /dev/null
+++ b/meta/recipes-connectivity/connman/connman/add_xuser_dbus_permission.patch
@@ -0,0 +1,23 @@
+Some platform (like atom-pc) enables rootless X,
+thus we need to add the xuser in the list.
+
+Upstream-Status: Inappropriate [configuration]
+
+Signed-off-by: Dongxiao Xu <dongxiao.xu@intel.com>
+
+diff -ruN connman-0.65-orig/src/connman-dbus.conf connman-0.65/src/connman-dbus.conf
+--- connman-0.65-orig/src/connman-dbus.conf 2011-03-04 09:34:49.000000000 +0800
++++ connman-0.65/src/connman-dbus.conf 2011-03-04 09:35:21.000000000 +0800
+@@ -7,6 +7,12 @@
+ <allow send_interface="net.connman.Agent"/>
+ <allow send_interface="net.connman.Counter"/>
+ </policy>
++ <policy user="xuser">
++ <allow own="net.connman"/>
++ <allow send_destination="net.connman"/>
++ <allow send_interface="net.connman.Agent"/>
++ <allow send_interface="net.connman.Counter"/>
++ </policy>
+ <policy at_console="true">
+ <allow send_destination="net.connman"/>
+ </policy>
diff --git a/meta/recipes-connectivity/connman/connman/connman b/meta/recipes-connectivity/connman/connman/connman
new file mode 100644
index 0000000..f01bf37
--- /dev/null
+++ b/meta/recipes-connectivity/connman/connman/connman
@@ -0,0 +1,62 @@
+#!/bin/sh
+
+DAEMON=/usr/sbin/connmand
+PIDFILE=/var/run/connmand.pid
+DESC="Connection Manager"
+
+if [ -f /etc/default/connman ] ; then
+ . /etc/default/connman
+fi
+
+set -e
+
+nfsroot=0
+
+exec 9<&0 < /proc/mounts
+while read dev mtpt fstype rest; do
+ if test $mtpt = "/" ; then
+ case $fstype in
+ nfs | nfs4)
+ nfsroot=1
+ break
+ ;;
+ *)
+ ;;
+ esac
+ fi
+done
+
+do_start() {
+ EXTRA_PARAM=""
+ if test $nfsroot -eq 1 ; then
+ EXTRA_PARAM="-P ethernet"
+ fi
+ $DAEMON $EXTRA_PARAM
+}
+
+do_stop() {
+ start-stop-daemon --stop --name connmand --quiet
+}
+
+case "$1" in
+ start)
+ echo "Starting $DESC"
+ do_start
+ ;;
+ stop)
+ echo "Stopping $DESC"
+ do_stop
+ ;;
+ restart|force-reload)
+ echo "Restarting $DESC"
+ do_stop
+ sleep 1
+ do_start
+ ;;
+ *)
+ echo "Usage: $0 {start|stop|restart|force-reload}" >&2
+ exit 1
+ ;;
+esac
+
+exit 0
diff --git a/meta/recipes-connectivity/connman/connman_0.75.bb b/meta/recipes-connectivity/connman/connman_0.75.bb
deleted file mode 100644
index c6e911b..0000000
--- a/meta/recipes-connectivity/connman/connman_0.75.bb
+++ /dev/null
@@ -1,29 +0,0 @@
-require connman.inc
-PR = "r4"
-
-EXTRA_OECONF += "\
- ac_cv_path_WPASUPPLICANT=/usr/sbin/wpa_supplicant \
- --disable-gtk-doc \
- --enable-debug \
- --enable-threads \
- --enable-loopback \
- --enable-ethernet \
- ${@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 \
- --disable-polkit \
- --enable-client \
- --enable-ofono \
- --prefix=/usr --sysconfdir=/etc --localstatedir=/var"
-
-SRC_URI = "\
- ${KERNELORG_MIRROR}/linux/network/connman/connman-${PV}.tar.gz \
- file://add_xuser_dbus_permission.patch \
- file://xtables_version.patch \
- file://connman \
-"
-
-SRC_URI[md5sum] = "9973cb89a11fff6b51fc85b51c13b711"
-SRC_URI[sha256sum] = "b15361237f7ec8092fb0e55d4585550ab35491485edaf10ddd032d6e36299db7"
diff --git a/meta/recipes-connectivity/connman/connman_0.78.bb b/meta/recipes-connectivity/connman/connman_0.78.bb
new file mode 100644
index 0000000..cc23c28
--- /dev/null
+++ b/meta/recipes-connectivity/connman/connman_0.78.bb
@@ -0,0 +1,10 @@
+require connman.inc
+
+PR = "r1"
+
+# 0.78 tag
+SRCREV = "02f5d5fe2d7c71514a6387ba2b772b42d8e8d297"
+SRC_URI = "git://git.kernel.org/pub/scm/network/connman/connman.git \
+ file://add_xuser_dbus_permission.patch \
+ file://connman"
+S = "${WORKDIR}/git"
diff --git a/meta/recipes-connectivity/connman/connman_git.bb b/meta/recipes-connectivity/connman/connman_git.bb
deleted file mode 100644
index 54ac4a5..0000000
--- a/meta/recipes-connectivity/connman/connman_git.bb
+++ /dev/null
@@ -1,11 +0,0 @@
-require connman.inc
-
-SRCREV = "36e877dece72577a3ae6197eafd505d0e6e0c1d2"
-PV = "0.47+git${SRCPV}"
-PR = "r17"
-S = "${WORKDIR}/git"
-
-SRC_URI = "git://git.kernel.org/pub/scm/network/connman/connman.git;protocol=git \
- file://dbusperms.patch \
- file://connman "
-
--
1.7.2.5
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] connman: update to 0.78
2011-12-09 10:51 [PATCH] connman: update to 0.78 Otavio Salvador
@ 2011-12-09 14:18 ` Richard Purdie
2011-12-09 16:38 ` Otavio Salvador
0 siblings, 1 reply; 4+ messages in thread
From: Richard Purdie @ 2011-12-09 14:18 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On Fri, 2011-12-09 at 10:51 +0000, Otavio Salvador wrote:
> Updates the package to 0.78 bringing changes done on meta-oe to
> OE-Core. This also removes unrequired build-depends and makes the
> package to respect wifi and bluetooth distro features.
>
> Following patches were removed:
>
> * xtables_version.patch: merged upstream
>
> * dbusperms.patch: dropped in favour of using sed to make the change
> in do_compile_append
>
> Another important change is the addition of ppp in depends that is
> used by some plugins.
>
> This uses PR as "r1" since it was previously available on meta-oe.
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> Acked-by: Joshua Lock <josh@linux.intel.com>
This patch appears to:
a) Remove multilib support from the recipe
b) Drop the git version of the recipe (no mention of this above?)
c) Add a pointless rdepends = d.getVar( "RDEPENDS_%s" % package)
d) Remove the dnsproxy, dhclient and test options
e) Add the tools, fake and ntpd options
f) Reorder variables in the file so as to make the diff unreadable and
hard to review
g) Change the contents of the ${PN}-scripts package
h) Add tools and tests packages
i) Add a dependeny on ntp
j) Uses bitbake master funtionality (d.expand()) which will break on
older versions of bitbake (we need to increase the bitbake minimum
version)
k) Drops the python scripts dependency
So all things considered, this kind of change is a nightmare to review
in this form and the commit message only covers half the changes it
makes. We need to do better than this...
Cheers,
Richard
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] connman: update to 0.78
2011-12-09 14:18 ` Richard Purdie
@ 2011-12-09 16:38 ` Otavio Salvador
2011-12-09 16:55 ` Richard Purdie
0 siblings, 1 reply; 4+ messages in thread
From: Otavio Salvador @ 2011-12-09 16:38 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 1666 bytes --]
On Fri, Dec 9, 2011 at 12:18, Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:
> This patch appears to:
>
> a) Remove multilib support from the recipe
>
Fixed.
> b) Drop the git version of the recipe (no mention of this above?)
>
I moved it to another commit.
> c) Add a pointless rdepends = d.getVar( "RDEPENDS_%s" % package)
>
Fixed.
> d) Remove the dnsproxy, dhclient and test options
>
Fixed.
> e) Add the tools, fake and ntpd options
>
Fixed.
> f) Reorder variables in the file so as to make the diff unreadable and
> hard to review
>
This was indeed done and I am sorry but it does seems more easy to follow
now.
> g) Change the contents of the ${PN}-scripts package
>
> h) Add tools and tests packages
>
> i) Add a dependeny on ntp
>
Fixed.
> j) Uses bitbake master funtionality (d.expand()) which will break on
> older versions of bitbake (we need to increase the bitbake minimum
> version)
>
Fixed adding a commit that bumps it.
> k) Drops the python scripts dependency
>
Fixed.
> So all things considered, this kind of change is a nightmare to review
> in this form and the commit message only covers half the changes it
> makes. We need to do better than this...
Not easy to accomplish as the difference between meta-oe and oe-core on
this was huge. It could have been done better, sure ... and a v2 patch
will come shortly.
--
Otavio Salvador O.S. Systems
E-mail: otavio@ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
[-- Attachment #2: Type: text/html, Size: 3832 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] connman: update to 0.78
2011-12-09 16:38 ` Otavio Salvador
@ 2011-12-09 16:55 ` Richard Purdie
0 siblings, 0 replies; 4+ messages in thread
From: Richard Purdie @ 2011-12-09 16:55 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On Fri, 2011-12-09 at 14:38 -0200, Otavio Salvador wrote:
> On Fri, Dec 9, 2011 at 12:18, Richard Purdie:
>
> f) Reorder variables in the file so as to make the diff
> unreadable and
> hard to review
>
> This was indeed done and I am sorry but it does seems more easy to
> follow now.
In future please do this as a separate commit which just changes the
order. I will start just outright rejecting patches which combine code
changes and unrelated ordering changes since it makes the patches
unnecessarily hard to review.
> So all things considered, this kind of change is a nightmare
> to review
> in this form and the commit message only covers half the
> changes it
> makes. We need to do better than this...
>
>
> Not easy to accomplish as the difference between meta-oe and oe-core
> on this was huge. It could have been done better, sure ... and a v2
> patch will come shortly.
Thanks. I think the best way to improve this in future would be to keep
code changes and ordering (or whitespace) changes in separate commits
(and also enable rename detection on the change you send).
Cheers,
Richard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-12-09 17:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-09 10:51 [PATCH] connman: update to 0.78 Otavio Salvador
2011-12-09 14:18 ` Richard Purdie
2011-12-09 16:38 ` Otavio Salvador
2011-12-09 16:55 ` Richard Purdie
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.