[oe] [PATCH] libcap2: fix build with and without libpam

[Khem Raj]

On Thu, Jan 13, 2011 at 2:24 PM, Andreas Oberritter
<obi at opendreambox.org> wrote:
> * Add libpam to depends if DISTRO_FEATURES contains pam.
>
> * Fixes the following compile error, when libpam-dev is installed
>  on the host, but not built for the target:
>
> pam_cap.c:19:34: error: security/pam_modules.h: No such file or
> directory
> pam_cap.c:20:34: error: security/_pam_macros.h: No such file or
> directory
>
> Signed-off-by: Andreas Oberritter <obi at opendreambox.org>
> CC: Frans Meulenbroeks <fransmeulenbroeks at gmail.com>
> CC: Otavio Salvador <otavio at ossystems.com.br>
> CC: Tom Rini <tom_rini at mentor.com>
> ---
> v2: Updated patch to fix both the dependency problem and
>    the compile error without libpam.
>
>  recipes/libcap/libcap2-2.16/make.patch |   15 ++++++++++++---
>  recipes/libcap/libcap2_2.16.bb         |    6 ++++--
>  2 files changed, 16 insertions(+), 5 deletions(-)
>
> diff --git a/recipes/libcap/libcap2-2.16/make.patch b/recipes/libcap/libcap2-2.16/make.patch
> index 3672f77..2d88ad9 100644
> --- a/recipes/libcap/libcap2-2.16/make.patch
> +++ b/recipes/libcap/libcap2-2.16/make.patch
> @@ -1,6 +1,8 @@
> ---- libcap-2.16/Make.Rules.orig        2009-03-07 18:50:23.000000000 +0100
> -+++ libcap-2.16/Make.Rules     2009-03-07 18:50:44.000000000 +0100
> -@@ -45,19 +45,12 @@
> +Index: libcap-2.16/Make.Rules
> +===================================================================
> +--- libcap-2.16.orig/Make.Rules        2008-12-04 08:03:30.000000000 +0100
> ++++ libcap-2.16/Make.Rules     2011-01-13 22:51:04.568844000 +0100
> +@@ -45,25 +45,18 @@
>  KERNEL_HEADERS := $(topdir)/libcap/include
>  IPATH += -I$(topdir)/libcap/include -I$(KERNEL_HEADERS)
>
> @@ -21,3 +23,10 @@
>
>  SYSTEM_HEADERS = /usr/include
>  INCS=$(topdir)/libcap/include/sys/capability.h
> + LDFLAGS += -L$(topdir)/libcap
> + CFLAGS += -Dlinux $(WARNINGS) $(DEBUG) $(IPATH)
> +-PAM_CAP := $(shell if [ -f /usr/include/security/pam_modules.h ]; then echo yes ; else echo no ; fi)
> ++PAM_CAP := $(shell if [ -f $(STAGING_INCDIR)/security/pam_modules.h ]; then echo yes ; else echo no ; fi)

can you use ${includedir} or something else ? STAGING_INCDIR is an OE
variable we should avoid perpetuating it into sourcecode and limit it
to metadata if possible

> + INDENT := $(shell if [ -n "$(which indent 2>/dev/null)" ]; then echo "| indent -kr" ; fi)
> + DYNAMIC := $(shell if [ ! -d "$(topdir)/.git" ]; then echo yes; fi)
> + LIBATTR := yes
> diff --git a/recipes/libcap/libcap2_2.16.bb b/recipes/libcap/libcap2_2.16.bb
> index 97b03d5..33594de 100644
> --- a/recipes/libcap/libcap2_2.16.bb
> +++ b/recipes/libcap/libcap2_2.16.bb
> @@ -2,8 +2,8 @@ DESCRIPTION = "Libcap is a library for getting and setting POSIX.1e (formerly PO
>  PRIORITY = "optional"
>  SECTION = "libs"
>  LICENSE = "GPL"
> -DEPENDS = "bison-native flex-native attr"
> -PR = "r2"
> +DEPENDS = "bison-native flex-native attr ${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}"
> +PR = "r3"
>
>  BUILD_CFLAGS += "-I${S}/libcap/include"
>  CFLAGS += "-I${S}/libcap/include"
> @@ -15,6 +15,8 @@ SRC_URI = "${KERNELORG_MIRROR}/pub/linux/libs/security/linux-privs/libcap2/libca
>
>  S = "${WORKDIR}/libcap-${PV}"
>
> +EXTRA_OEMAKE = "STAGING_INCDIR=${STAGING_INCDIR}"
> +
>  do_install() {
>        install -d ${D}${includedir}/sys
>        install -m 0644 libcap/include/sys/capability.h ${D}${includedir}/sys/
> --
> 1.7.2.3
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>