From: Joshua Lock <joshua.g.lock@intel.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH v5 0/2] Clean up connman-conf on systemd systems
Date: Wed, 27 Jan 2016 11:23:58 +0000 [thread overview]
Message-ID: <cover.1453893763.git.joshua.g.lock@intel.com> (raw)
As reported in YP#8399 we currently hack the connman service to try and
execute a script which is only installed on qemu targets, its absence on
non-qemu machines results in an error in the journal:
connman.service: Failed at step EXEC spawning /usr/lib/connman/wired-setup: No such file or directory
This pair of patches changes the connman-conf recipe to install a oneshot
systemd unit that is called before ConnMan and removes the ExecStartPre.
SysV init based images are unaffected by this change, the existing
mechanisms to check for the presence of the wired-setup script from the
init script remain in place.
Since v4 I've rebased on master and updated the unit file to have the path
to the script sed'ed in, as the script location has changed since the
original submission.
Tested by 1) booting a qemux86 and validating the network was brought up
per the wired.config settings 2) booting a genericx86 image and checking
that the wired-connection.service isn't installed.
Regards,
Joshua
The following changes since commit 2a6e061712cfe9cb4738806a0c351a64e0d30144:
cmake: update to 3.4.2 (2016-01-26 22:48:57 +0000)
are available in the git repository at:
git://git.openembedded.org/openembedded-core-contrib joshuagl/connman-conf
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=joshuagl/connman-conf
Joshua Lock (2):
connman-conf: convert to systemd oneshot
connman: tidy up connman-conf usage
meta/recipes-connectivity/connman/connman-conf.bb | 12 +++++++++++-
.../connman/connman-conf/qemuall/wired-connection.service | 10 ++++++++++
meta/recipes-connectivity/connman/connman.inc | 5 -----
3 files changed, 21 insertions(+), 6 deletions(-)
create mode 100644 meta/recipes-connectivity/connman/connman-conf/qemuall/wired-connection.service
--
2.5.0
next reply other threads:[~2016-01-27 11:24 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-27 11:23 Joshua Lock [this message]
2016-01-27 11:23 ` [PATCH v5 1/2] connman-conf: convert to systemd oneshot Joshua Lock
2016-01-27 11:24 ` [PATCH v5 2/2] connman: tidy up connman-conf usage Joshua Lock
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1453893763.git.joshua.g.lock@intel.com \
--to=joshua.g.lock@intel.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox