* [Buildroot] [PATCH 1/1] package/apr-util: fix ldap detection
@ 2018-05-09 19:34 Bernd Kuhls
2018-05-12 7:01 ` Thomas Petazzoni
2018-05-28 14:23 ` Peter Korsgaard
0 siblings, 2 replies; 3+ messages in thread
From: Bernd Kuhls @ 2018-05-09 19:34 UTC (permalink / raw)
To: buildroot
configure checks if ldap_set_rebind_proc takes 2 or 3 arguments
http://svn.apache.org/viewvc/apr/apr-util/tags/1.6.1/build/apu-conf.m4?revision=1812528&view=markup#l370
It uses the macro APR_TRY_COMPILE_NO_WARNING which adds -Werror to
treat all warnings as errors when gcc is used:
http://svn.apache.org/viewvc/apr/apr/tags/1.6.3/build/apr_common.m4?revision=1812527&view=markup#l504
In some buildroot configs a compiler warning occurs during this check:
http://autobuild.buildroot.net/results/241/241ed78b93ce86c859e175530fa485711ff61615//apr-util-1.6.1/config.log
/home/rclinux/rc-buildroot-test/scripts/instance-1/output/host/arm-buildroot-linux-uclibcgnueabihf/sysroot/usr/include/features.h:218:5:
error: #warning requested reentrant code, but thread support was disabled [-Werror=cpp]
cc1: all warnings being treated as errors
Due to this warning 'ac_cv_ldap_set_rebind_proc_style' is set to two
instead of three leading to a build error later on. This patch forces
ac_cv_ldap_set_rebind_proc_style=three to be inline with openldap which
fixes
http://autobuild.buildroot.net/results/241/241ed78b93ce86c859e175530fa485711ff61615/
This solution was inspired by a discussion on the fink mailinglist:
https://sourceforge.net/p/fink/mailman/message/31720482/
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/apr-util/apr-util.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/apr-util/apr-util.mk b/package/apr-util/apr-util.mk
index 973194b915..b006964ccb 100644
--- a/package/apr-util/apr-util.mk
+++ b/package/apr-util/apr-util.mk
@@ -48,6 +48,7 @@ APR_UTIL_CONF_OPTS += --without-sqlite3
endif
ifeq ($(BR2_PACKAGE_OPENLDAP),y)
+APR_UTIL_CONF_ENV += ac_cv_ldap_set_rebind_proc_style=three
APR_UTIL_CONF_OPTS += \
--with-ldap \
--with-ldap-include="$(STAGING_DIR)/usr/include/" \
--
2.14.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH 1/1] package/apr-util: fix ldap detection
2018-05-09 19:34 [Buildroot] [PATCH 1/1] package/apr-util: fix ldap detection Bernd Kuhls
@ 2018-05-12 7:01 ` Thomas Petazzoni
2018-05-28 14:23 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2018-05-12 7:01 UTC (permalink / raw)
To: buildroot
Hello,
On Wed, 9 May 2018 21:34:16 +0200, Bernd Kuhls wrote:
> configure checks if ldap_set_rebind_proc takes 2 or 3 arguments
> http://svn.apache.org/viewvc/apr/apr-util/tags/1.6.1/build/apu-conf.m4?revision=1812528&view=markup#l370
>
> It uses the macro APR_TRY_COMPILE_NO_WARNING which adds -Werror to
> treat all warnings as errors when gcc is used:
> http://svn.apache.org/viewvc/apr/apr/tags/1.6.3/build/apr_common.m4?revision=1812527&view=markup#l504
>
> In some buildroot configs a compiler warning occurs during this check:
> http://autobuild.buildroot.net/results/241/241ed78b93ce86c859e175530fa485711ff61615//apr-util-1.6.1/config.log
>
> /home/rclinux/rc-buildroot-test/scripts/instance-1/output/host/arm-buildroot-linux-uclibcgnueabihf/sysroot/usr/include/features.h:218:5:
> error: #warning requested reentrant code, but thread support was disabled [-Werror=cpp]
> cc1: all warnings being treated as errors
>
> Due to this warning 'ac_cv_ldap_set_rebind_proc_style' is set to two
> instead of three leading to a build error later on. This patch forces
> ac_cv_ldap_set_rebind_proc_style=three to be inline with openldap which
> fixes
> http://autobuild.buildroot.net/results/241/241ed78b93ce86c859e175530fa485711ff61615/
>
> This solution was inspired by a discussion on the fink mailinglist:
> https://sourceforge.net/p/fink/mailman/message/31720482/
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/apr-util/apr-util.mk | 1 +
> 1 file changed, 1 insertion(+)
Applied to master, thanks.
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH 1/1] package/apr-util: fix ldap detection
2018-05-09 19:34 [Buildroot] [PATCH 1/1] package/apr-util: fix ldap detection Bernd Kuhls
2018-05-12 7:01 ` Thomas Petazzoni
@ 2018-05-28 14:23 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-05-28 14:23 UTC (permalink / raw)
To: buildroot
>>>>> "Bernd" == Bernd Kuhls <bernd.kuhls@t-online.de> writes:
> configure checks if ldap_set_rebind_proc takes 2 or 3 arguments
> http://svn.apache.org/viewvc/apr/apr-util/tags/1.6.1/build/apu-conf.m4?revision=1812528&view=markup#l370
> It uses the macro APR_TRY_COMPILE_NO_WARNING which adds -Werror to
> treat all warnings as errors when gcc is used:
> http://svn.apache.org/viewvc/apr/apr/tags/1.6.3/build/apr_common.m4?revision=1812527&view=markup#l504
> In some buildroot configs a compiler warning occurs during this check:
> http://autobuild.buildroot.net/results/241/241ed78b93ce86c859e175530fa485711ff61615//apr-util-1.6.1/config.log
> /home/rclinux/rc-buildroot-test/scripts/instance-1/output/host/arm-buildroot-linux-uclibcgnueabihf/sysroot/usr/include/features.h:218:5:
> error: #warning requested reentrant code, but thread support was disabled [-Werror=cpp]
> cc1: all warnings being treated as errors
> Due to this warning 'ac_cv_ldap_set_rebind_proc_style' is set to two
> instead of three leading to a build error later on. This patch forces
> ac_cv_ldap_set_rebind_proc_style=three to be inline with openldap which
> fixes
> http://autobuild.buildroot.net/results/241/241ed78b93ce86c859e175530fa485711ff61615/
> This solution was inspired by a discussion on the fink mailinglist:
> https://sourceforge.net/p/fink/mailman/message/31720482/
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
Committed to 2018.02.x, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-05-28 14:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-09 19:34 [Buildroot] [PATCH 1/1] package/apr-util: fix ldap detection Bernd Kuhls
2018-05-12 7:01 ` Thomas Petazzoni
2018-05-28 14:23 ` Peter Korsgaard
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.