* [Buildroot] [PATCH 1/1] package/linux-pam: fix build with uclibc
@ 2020-06-11 11:28 Fabrice Fontaine
2020-06-14 16:40 ` Yann E. MORIN
0 siblings, 1 reply; 2+ messages in thread
From: Fabrice Fontaine @ 2020-06-11 11:28 UTC (permalink / raw)
To: buildroot
Fixes:
- http://autobuild.buildroot.org/results/92b3dd7c984d2b843ac9aacacd69eec99f28743e
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
...when-crypt-does-not-require-libcrypt.patch | 37 +++++++++++++++++++
package/linux-pam/linux-pam.mk | 2 +
2 files changed, 39 insertions(+)
create mode 100644 package/linux-pam/0001-configure-ac-fix-build-failure-when-crypt-does-not-require-libcrypt.patch
diff --git a/package/linux-pam/0001-configure-ac-fix-build-failure-when-crypt-does-not-require-libcrypt.patch b/package/linux-pam/0001-configure-ac-fix-build-failure-when-crypt-does-not-require-libcrypt.patch
new file mode 100644
index 0000000000..ebcb81aa91
--- /dev/null
+++ b/package/linux-pam/0001-configure-ac-fix-build-failure-when-crypt-does-not-require-libcrypt.patch
@@ -0,0 +1,37 @@
+From aef363c7e8e942224e6cffc4398366c6e5d31749 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Thu, 11 Jun 2020 00:04:32 +0200
+Subject: [PATCH] configure.ac: fix build failure when crypt() does not require
+ libcrypt
+
+Since commit 522246d20e4cd92fadc2d760228cb7e78cbeb4c5, the build fails
+if "none required" is returned by AC_SEARCH_LIBS for libcrypt.
+
+Resolves: https://github.com/linux-pam/linux-pam/pull/235
+Fixes: http://autobuild.buildroot.org/results/92b3dd7c984d2b843ac9aacacd69eec99f28743e
+Fixes: v1.4.0~228 ("Use cached 'crypt' library result correctly")
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+[Retrieved from:
+https://github.com/linux-pam/linux-pam/commit/aef363c7e8e942224e6cffc4398366c6e5d31749]
+---
+ configure.ac | 6 +++++-
+ 1 file changed, 5 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index ea08a7a3..c1862ea7 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -428,7 +428,11 @@ AS_IF([test "x$ac_cv_header_xcrypt_h" = "xyes"],
+ [crypt_libs="crypt"])
+
+ BACKUP_LIBS=$LIBS
+-AC_SEARCH_LIBS([crypt],[$crypt_libs], LIBCRYPT="${ac_cv_search_crypt}", LIBCRYPT="")
++AC_SEARCH_LIBS([crypt],[$crypt_libs])
++case "$ac_cv_search_crypt" in
++ -l*) LIBCRYPT="$ac_cv_search_crypt" ;;
++ *) LIBCRYPT="" ;;
++esac
+ AC_CHECK_FUNCS(crypt_r crypt_gensalt_r)
+ LIBS=$BACKUP_LIBS
+ AC_SUBST(LIBCRYPT)
diff --git a/package/linux-pam/linux-pam.mk b/package/linux-pam/linux-pam.mk
index 8f8d575102..57fb2c9cfd 100644
--- a/package/linux-pam/linux-pam.mk
+++ b/package/linux-pam/linux-pam.mk
@@ -20,6 +20,8 @@ LINUX_PAM_DEPENDENCIES = flex host-flex host-pkgconf \
$(TARGET_NLS_DEPENDENCIES)
LINUX_PAM_LICENSE = BSD-3-Clause
LINUX_PAM_LICENSE_FILES = Copyright
+# We're patching configure.ac
+LINUX_PAM_AUTORECONF = YES
LINUX_PAM_MAKE_OPTS += LIBS=$(TARGET_NLS_LIBS)
ifeq ($(BR2_PACKAGE_LIBSELINUX),y)
--
2.26.2
^ permalink raw reply related [flat|nested] 2+ messages in thread* [Buildroot] [PATCH 1/1] package/linux-pam: fix build with uclibc
2020-06-11 11:28 [Buildroot] [PATCH 1/1] package/linux-pam: fix build with uclibc Fabrice Fontaine
@ 2020-06-14 16:40 ` Yann E. MORIN
0 siblings, 0 replies; 2+ messages in thread
From: Yann E. MORIN @ 2020-06-14 16:40 UTC (permalink / raw)
To: buildroot
Fabrice, All,
On 2020-06-11 13:28 +0200, Fabrice Fontaine spake thusly:
> Fixes:
> - http://autobuild.buildroot.org/results/92b3dd7c984d2b843ac9aacacd69eec99f28743e
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Applied to master.
WE however have a whole lot of other failures on musl:
pam_faillock.c: In function 'check_local_user':
pam_faillock.c:365:10: warning: implicit declaration of function 'fgetpwent_r'; did you mean 'fgetpwent'? [-Wimplicit-function-declaration]
errn = fgetpwent_r(fp, &pw, buf, sizeof (buf), &pwp);
^~~~~~~~~~~
fgetpwent
Care to have a look in those, please?
Regards,
Yann E. MORIN.
> ---
> ...when-crypt-does-not-require-libcrypt.patch | 37 +++++++++++++++++++
> package/linux-pam/linux-pam.mk | 2 +
> 2 files changed, 39 insertions(+)
> create mode 100644 package/linux-pam/0001-configure-ac-fix-build-failure-when-crypt-does-not-require-libcrypt.patch
>
> diff --git a/package/linux-pam/0001-configure-ac-fix-build-failure-when-crypt-does-not-require-libcrypt.patch b/package/linux-pam/0001-configure-ac-fix-build-failure-when-crypt-does-not-require-libcrypt.patch
> new file mode 100644
> index 0000000000..ebcb81aa91
> --- /dev/null
> +++ b/package/linux-pam/0001-configure-ac-fix-build-failure-when-crypt-does-not-require-libcrypt.patch
> @@ -0,0 +1,37 @@
> +From aef363c7e8e942224e6cffc4398366c6e5d31749 Mon Sep 17 00:00:00 2001
> +From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +Date: Thu, 11 Jun 2020 00:04:32 +0200
> +Subject: [PATCH] configure.ac: fix build failure when crypt() does not require
> + libcrypt
> +
> +Since commit 522246d20e4cd92fadc2d760228cb7e78cbeb4c5, the build fails
> +if "none required" is returned by AC_SEARCH_LIBS for libcrypt.
> +
> +Resolves: https://github.com/linux-pam/linux-pam/pull/235
> +Fixes: http://autobuild.buildroot.org/results/92b3dd7c984d2b843ac9aacacd69eec99f28743e
> +Fixes: v1.4.0~228 ("Use cached 'crypt' library result correctly")
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +[Retrieved from:
> +https://github.com/linux-pam/linux-pam/commit/aef363c7e8e942224e6cffc4398366c6e5d31749]
> +---
> + configure.ac | 6 +++++-
> + 1 file changed, 5 insertions(+), 1 deletion(-)
> +
> +diff --git a/configure.ac b/configure.ac
> +index ea08a7a3..c1862ea7 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -428,7 +428,11 @@ AS_IF([test "x$ac_cv_header_xcrypt_h" = "xyes"],
> + [crypt_libs="crypt"])
> +
> + BACKUP_LIBS=$LIBS
> +-AC_SEARCH_LIBS([crypt],[$crypt_libs], LIBCRYPT="${ac_cv_search_crypt}", LIBCRYPT="")
> ++AC_SEARCH_LIBS([crypt],[$crypt_libs])
> ++case "$ac_cv_search_crypt" in
> ++ -l*) LIBCRYPT="$ac_cv_search_crypt" ;;
> ++ *) LIBCRYPT="" ;;
> ++esac
> + AC_CHECK_FUNCS(crypt_r crypt_gensalt_r)
> + LIBS=$BACKUP_LIBS
> + AC_SUBST(LIBCRYPT)
> diff --git a/package/linux-pam/linux-pam.mk b/package/linux-pam/linux-pam.mk
> index 8f8d575102..57fb2c9cfd 100644
> --- a/package/linux-pam/linux-pam.mk
> +++ b/package/linux-pam/linux-pam.mk
> @@ -20,6 +20,8 @@ LINUX_PAM_DEPENDENCIES = flex host-flex host-pkgconf \
> $(TARGET_NLS_DEPENDENCIES)
> LINUX_PAM_LICENSE = BSD-3-Clause
> LINUX_PAM_LICENSE_FILES = Copyright
> +# We're patching configure.ac
> +LINUX_PAM_AUTORECONF = YES
> LINUX_PAM_MAKE_OPTS += LIBS=$(TARGET_NLS_LIBS)
>
> ifeq ($(BR2_PACKAGE_LIBSELINUX),y)
> --
> 2.26.2
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-06-14 16:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-06-11 11:28 [Buildroot] [PATCH 1/1] package/linux-pam: fix build with uclibc Fabrice Fontaine
2020-06-14 16:40 ` Yann E. MORIN
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox