Skip to content

RFC Clean up connman-conf

Joshua Lock openembedded-core 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest2 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2connman-conf: convert to systemd oneshotmeta/recipes-connectivity/connman/connman-conf.bb meta/recipes-connectivity/connman/connman-conf/qemuall/wired-connection.service+21 -20 replies
2/2connman: tidy up connman-conf usagemeta/recipes-connectivity/connman/connman.inc meta/recipes-connectivity/connman/connman_1.30.bb+1 -60 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [RFC PATCH 0/2] Clean up connman-conf

As reported in YP#8399[1] we currently hack the connman service to try and execute a script which is only installed on qemu targets. This pair of patches changes the connman-conf recipe to install a oneshot systemd unit that is called before ConnMan, removes the ExecStartPre and adds an override to the RRECOMMENDS so that the connman-conf is only installed by default for qemu machines and images for other machines are left with an error in the journal about failing to start a service.

Please review the following changes for suitability for inclusion. If you have any objections or suggestions for improvement, please respond to the patches. If you agree with the changes, please provide your Acked-by.

Regards,

Joshua

1. https://bugzilla.yoctoproject.org/show_bug.cgi?id=8399

The following changes since commit e42d8eff9eed7d1454b4f331d96dcee6dea232df:

bash: Disable custom memory allocator (2015-10-12 14:35:42 +0100)

are available in the git repository at:

git://git.openembedded.org/openembedded-core-contrib joshuagl/connman
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=joshuagl/connman

Joshua Lock (2):
connman-conf: convert to systemd oneshot
connman: tidy up connman-conf usage

 meta/recipes-connectivity/connman/connman-conf.bb           | 13 +++++++++++--
 .../connman/connman-conf/qemuall/wired-connection.service   | 10 ++++++++++
 meta/recipes-connectivity/connman/connman.inc               |  5 -----
 meta/recipes-connectivity/connman/connman_1.30.bb           |  2 +-
 4 files changed, 22 insertions(+), 8 deletions(-)
 create mode 100644 meta/recipes-connectivity/connman/connman-conf/qemuall/wired-connection.service
Signature · 1 line
2.1.4