[oe] [meta-networking][PATCH] autofs: correct QA issues with orphaned systemd files

[Joe MacDonald]

Okay, that'd be 'no'.  Wrong version of the patch, v2 coming soon.

-J.

[[oe] [meta-networking][PATCH] autofs: correct QA issues with orphaned systemd files] On 13.11.01 (Fri 14:14) Joe MacDonald wrote:

> If we build a project without systemd support on a system that has systemd
> enabled, the following QA items get flagged:
> 
>   /lib
>   /lib/systemd
>   /lib/systemd/system
>   /lib/systemd/system/autofs.service
> 
> Trying to keep the solution to the problem as contained as possible in a
> way that may be accepted up-stream by adding an optional parameter to the
> --with-systemd flag.
> 
> Signed-off-by: Joe MacDonald <joe at deserted.net>
> ---
>  ...emd-allow-with-systemd-to-take-a-path-arg.patch | 45 ++++++++++++++++++++++
>  .../recipes-daemons/autofs/autofs_5.0.7.bb         |  4 +-
>  2 files changed, 47 insertions(+), 2 deletions(-)
>  create mode 100644 meta-networking/recipes-daemons/autofs/autofs-5.0.7/0001-systemd-allow-with-systemd-to-take-a-path-arg.patch
> 
> diff --git a/meta-networking/recipes-daemons/autofs/autofs-5.0.7/0001-systemd-allow-with-systemd-to-take-a-path-arg.patch b/meta-networking/recipes-daemons/autofs/autofs-5.0.7/0001-systemd-allow-with-systemd-to-take-a-path-arg.patch
> new file mode 100644
> index 0000000..0eeceb0
> --- /dev/null
> +++ b/meta-networking/recipes-daemons/autofs/autofs-5.0.7/0001-systemd-allow-with-systemd-to-take-a-path-arg.patch
> @@ -0,0 +1,45 @@
> +From 40971911d653bf53de295d7462c643e4073916b9 Mon Sep 17 00:00:00 2001
> +From: Joe MacDonald <joe at deserted.net>
> +Date: Fri, 1 Nov 2013 12:47:18 -0400
> +Subject: [PATCH] systemd: allow --with-systemd to take a path arg
> +
> +If building for a cross-compile environment with systemd it is convenient
> +to be able to specify a systemd path for the target that may not be the
> +same as that on the host.
> +
> +Signed-off-by: Joe MacDonald <joe at deserted.net>
> +---
> + aclocal.m4 |   10 ++++++++--
> + 1 file changed, 8 insertions(+), 2 deletions(-)
> +
> +diff --git a/aclocal.m4 b/aclocal.m4
> +index 3e6f223..105e3e9 100644
> +--- a/aclocal.m4
> ++++ b/aclocal.m4
> +@@ -229,8 +229,10 @@ dnl Check the location of the systemd unit files directory
> + dnl --------------------------------------------------------------------------
> + AC_DEFUN([AF_WITH_SYSTEMD],
> + [AC_ARG_WITH(systemd,
> +-[  --with-systemd         install systemd unit file if systemd unit directory
> +-			  is found on system],
> ++[  --with-systemd@<:@=systemddir@:>@  install systemd unit file.  If 'yes'
> ++			  probe the system for unit directory.
> ++			  If a path is specified, assume that
> ++			  is a valid install path.],
> + [if test "$withval" = yes; then
> +   if test -z "$systemddir"; then
> +     AC_MSG_CHECKING([location of the systemd unit files directory])
> +@@ -247,6 +249,10 @@ AC_DEFUN([AF_WITH_SYSTEMD],
> +   else
> +     AC_MSG_RESULT(not found)
> +   fi
> ++else
> ++ if test "$withval" != no; then
> ++  systemddir=$withval
> ++ fi
> + fi])
> + ])
> + 
> +-- 
> +1.7.10.4
> +
> diff --git a/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb b/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
> index c4eea31..54e89f5 100644
> --- a/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
> +++ b/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
> @@ -60,6 +60,7 @@ SRC_URI = "${KERNELORG_MIRROR}/linux/daemons/autofs/v5/autofs-${PV}.tar.bz2 \
>             file://autofs-5.0.7-do-not-check-for-modprobe.patch \
>             file://fix_disable_ldap.patch \
>             file://autofs-5.0.7-fix-lib-deps.patch \
> +           file://0001-systemd-allow-with-systemd-to-take-a-path-arg.patch \
>  "
>  
>  SRC_URI[md5sum] = "bc46838dece83c02d800ff144ed9f431"
> @@ -73,7 +74,7 @@ INITSCRIPT_PARAMS = "defaults"
>  # FIXME: modules/Makefile has crappy rules that don't obey LDFLAGS
>  CFLAGS += "${LDFLAGS}"
>  
> -PACKAGECONFIG[systemd] = "--with-systemd,--without-systemd,systemd"
> +PACKAGECONFIG[systemd] = "--with-systemd=${systemd_unitdir}/system,--without-systemd,systemd"
>  
>  PACKAGECONFIG ?= "${@base_contains('DISTRO_FEATURES', 'systemd', 'systemd', '', d)}"
>  
> @@ -85,7 +86,6 @@ EXTRA_OECONF += "--disable-mount-locking \
>  "
>  CACHED_CONFIGUREVARS = "ac_cv_path_RANLIB=${RANLIB} \
>                          ac_cv_path_RPCGEN=rpcgen \
> -                        systemddir=${systemd_unitdir}/system \
>  "
>  
>  do_configure_prepend () {
-- 
-Joe MacDonald.
:wq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20131101/2bc4f4fb/attachment-0002.sig>