[ptxdist] [PATCHv4 1/1] linux-pam: this patch add Linux-PAM support to ptxdist

[ptxdist] [PATCHv4 1/1] linux-pam: this patch add Linux-PAM support to ptxdist

[Oliver Graute]

this patch add Linux-PAM support to ptxdist

Signed-off-by: Oliver Graute <oliver.graute@neuhaus.de>
---
 rules/linux-pam.in   |   13 ++++++++++++
 rules/linux-pam.make |   57 ++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 70 insertions(+)
 create mode 100644 rules/linux-pam.in
 create mode 100644 rules/linux-pam.make

diff --git a/rules/linux-pam.in b/rules/linux-pam.in
new file mode 100644
index 0000000..d76e4db
--- /dev/null
+++ b/rules/linux-pam.in
@@ -0,0 +1,13 @@
+## SECTION=networking
+
+config LINUX_PAM
+	tristate
+	prompt "linux-pam"
+	help
+	  Linux-PAM is a free implementation of the following DCE-RFC from
+	  Sunsoft. PAM provides a way to develop programs that are
+	  independent of authentication scheme. These programs need
+	  "authentication modules" to be attatched to them at run-time
+	  in order to work. Which authentication module is to be attatched
+	  is dependent upon the local system setup and is at the discretion
+	  of the local system administrator.
diff --git a/rules/linux-pam.make b/rules/linux-pam.make
new file mode 100644
index 0000000..bdb150d
--- /dev/null
+++ b/rules/linux-pam.make
@@ -0,0 +1,57 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2015 Dr. Neuhaus Telekommunikation GmbH, Hamburg Germany, Oliver Graute <oliver.graute@neuhaus.de>
+#
+# See CREDITS for details about who has contributed to this project.
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+PACKAGES-$(PTXCONF_LINUX_PAM) += linux-pam
+
+#
+# Paths and names
+#
+LINUX_PAM_VERSION	:= 1.1.8
+LINUX_PAM_MD5		:= 35b6091af95981b1b2cd60d813b5e4ee
+LINUX_PAM		:= Linux-PAM-$(LINUX_PAM_VERSION)
+LINUX_PAM_SUFFIX	:= tar.bz2
+LINUX_PAM_URL		:= http://www.linux-pam.org/library/$(LINUX_PAM).$(LINUX_PAM_SUFFIX)
+LINUX_PAM_SOURCE	:= $(SRCDIR)/$(LINUX_PAM).$(LINUX_PAM_SUFFIX)
+LINUX_PAM_DIR		:= $(BUILDDIR)/$(LINUX_PAM)
+LINUX_PAM_LICENSE	:= GPL, BSD
+
+# ----------------------------------------------------------------------------
+# Prepare
+# ----------------------------------------------------------------------------
+
+#
+# autoconf
+#
+LINUX_PAM_CONF_TOOL	:= autoconf
+
+# ----------------------------------------------------------------------------
+# Target-Install
+# ----------------------------------------------------------------------------
+
+$(STATEDIR)/linux-pam.targetinstall:
+	@$(call targetinfo)
+
+	@$(call install_init, linux-pam)
+	@$(call install_fixup, linux-pam,PRIORITY,optional)
+	@$(call install_fixup, linux-pam,SECTION,base)
+	@$(call install_fixup, linux-pam,AUTHOR,"<oliver.graute@neuhaus.de>")
+	@$(call install_fixup, linux-pam,DESCRIPTION,missing)
+
+	@$(call install_lib, linux-pam, 0, 0, 0644, libpam)
+
+	@$(call install_alternative, linux-pam, 0, 0, 0644, /etc/pam.conf)
+	@$(call install_finish, linux-pam)
+
+	@$(call touch)
+
+# vim: syntax=make
-- 
1.7.9.5


-- 
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] [PATCHv4 1/1] linux-pam: this patch add Linux-PAM support to ptxdist

[Marc Kleine-Budde]

[-- Attachment #1.1: Type: text/plain, Size: 4361 bytes --]

On 04/01/2015 04:27 PM, Oliver Graute wrote:
> this patch add Linux-PAM support to ptxdist
> 
> Signed-off-by: Oliver Graute <oliver.graute@neuhaus.de>
> ---
>  rules/linux-pam.in   |   13 ++++++++++++
>  rules/linux-pam.make |   57 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 70 insertions(+)
>  create mode 100644 rules/linux-pam.in
>  create mode 100644 rules/linux-pam.make
> 
> diff --git a/rules/linux-pam.in b/rules/linux-pam.in
> new file mode 100644
> index 0000000..d76e4db
> --- /dev/null
> +++ b/rules/linux-pam.in
> @@ -0,0 +1,13 @@
> +## SECTION=networking
> +
> +config LINUX_PAM
> +	tristate
> +	prompt "linux-pam"

I think a dependency to flex is missing, maybe it's only needed when
compiling with selinux support...:

libtool: link: arm-cortexa8-linux-gnueabihf-gcc -g -O2 -Wl,--as-needed
-Wl,--no-undefined -Wl,-O1 -o pam_conv1 pam_conv_l.o pam_conv_y.o
pam_conv_l.o: In function `yylex':
platform-1234/build-target/Linux-PAM-1.1.8/conf/pam_conv1/pam_conv_l.c:871:
undefined reference to `yywrap'

> +	help
> +	  Linux-PAM is a free implementation of the following DCE-RFC from
> +	  Sunsoft. PAM provides a way to develop programs that are
> +	  independent of authentication scheme. These programs need
> +	  "authentication modules" to be attatched to them at run-time
> +	  in order to work. Which authentication module is to be attatched
> +	  is dependent upon the local system setup and is at the discretion
> +	  of the local system administrator.
> diff --git a/rules/linux-pam.make b/rules/linux-pam.make
> new file mode 100644
> index 0000000..bdb150d
> --- /dev/null
> +++ b/rules/linux-pam.make
> @@ -0,0 +1,57 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2015 Dr. Neuhaus Telekommunikation GmbH, Hamburg Germany, Oliver Graute <oliver.graute@neuhaus.de>
> +#
> +# See CREDITS for details about who has contributed to this project.
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_LINUX_PAM) += linux-pam
> +
> +#
> +# Paths and names
> +#
> +LINUX_PAM_VERSION	:= 1.1.8
> +LINUX_PAM_MD5		:= 35b6091af95981b1b2cd60d813b5e4ee
> +LINUX_PAM		:= Linux-PAM-$(LINUX_PAM_VERSION)
> +LINUX_PAM_SUFFIX	:= tar.bz2
> +LINUX_PAM_URL		:= http://www.linux-pam.org/library/$(LINUX_PAM).$(LINUX_PAM_SUFFIX)
> +LINUX_PAM_SOURCE	:= $(SRCDIR)/$(LINUX_PAM).$(LINUX_PAM_SUFFIX)
> +LINUX_PAM_DIR		:= $(BUILDDIR)/$(LINUX_PAM)
> +LINUX_PAM_LICENSE	:= GPL, BSD
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +#
> +# autoconf
> +#
> +LINUX_PAM_CONF_TOOL	:= autoconf

There a lot of options to pam's ./configure, please provide sensible
defaults via CONF_OPT (at least for external packages) and make use of
the gobal ptxdist option:

--$(call ptx/wwo, PTXCONF_GLOBAL_SELINUX)-selinux
$(GLOBAL_LARGE_FILE_OPTION)

> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/linux-pam.targetinstall:
> +	@$(call targetinfo)
> +
> +	@$(call install_init, linux-pam)
> +	@$(call install_fixup, linux-pam,PRIORITY,optional)
> +	@$(call install_fixup, linux-pam,SECTION,base)
> +	@$(call install_fixup, linux-pam,AUTHOR,"<oliver.graute@neuhaus.de>")
> +	@$(call install_fixup, linux-pam,DESCRIPTION,missing)
> +
> +	@$(call install_lib, linux-pam, 0, 0, 0644, libpam)
> +
> +	@$(call install_alternative, linux-pam, 0, 0, 0644, /etc/pam.conf)

The install stage fails with:
Error: file not found:
platform-123/build-target/Linux-PAM-1.1.8/etc/pam.conf

Please provide a default projectroot/etc/pam.conf via this patch.

> +	@$(call install_finish, linux-pam)
> +
> +	@$(call touch)
> +
> +# vim: syntax=make
> 

Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

[ptxdist] Antwort: Re: [PATCHv4 1/1] linux-pam: this patch add Linux-PAM support to ptxdist

[Oliver.Graute]

[-- Attachment #1.1.1: Type: text/plain, Size: 1103 bytes --]



> > diff --git a/rules/linux-pam.in b/rules/linux-pam.in
> > new file mode 100644
> > index 0000000..d76e4db
> > --- /dev/null
> > +++ b/rules/linux-pam.in
> > @@ -0,0 +1,13 @@
> > +## SECTION=networking
> > +
> > +config LINUX_PAM
> > +   tristate
> > +   prompt "linux-pam"
>
> I think a dependency to flex is missing, maybe it's only needed when
> compiling with selinux support...:

flex support is necessary. i will add it with a select FLEX statement

> libtool: link: arm-cortexa8-linux-gnueabihf-gcc -g -O2 -Wl,--as-needed
> -Wl,--no-undefined -Wl,-O1 -o pam_conv1 pam_conv_l.o pam_conv_y.o
> pam_conv_l.o: In function `yylex':
>
platform-1234/build-target/Linux-PAM-1.1.8/conf/pam_conv1/pam_conv_l.c:871:
> undefined reference to `yywrap'

flex dependency will fix that error

> > +#
> > +# autoconf
> > +#
> > +LINUX_PAM_CONF_TOOL   := autoconf
>
> There a lot of options to pam's ./configure, please provide sensible
> defaults via CONF_OPT (at least for external packages) and make use of
> the gobal ptxdist option:
>

hm i don't know which ones are necessary here.

Best regards,

Oliver

[-- Attachment #1.1.2: Type: text/html, Size: 1520 bytes --]

[-- Attachment #1.2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 2446 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de