* [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et
@ 2021-10-19 17:25 Fabrice Fontaine
2021-10-19 17:25 ` [Buildroot] [PATCH 2/3] package/heimdal: " Fabrice Fontaine
` (3 more replies)
0 siblings, 4 replies; 9+ messages in thread
From: Fabrice Fontaine @ 2021-10-19 17:25 UTC (permalink / raw)
To: buildroot; +Cc: Fabrice Fontaine
Don't install compile_et as it raises the following build failure with
samba4 since, at least, bump to version 4.14.7 in commit
630e85f8f5ed64933b45599fc4332d9609177c5c:
source4/heimdal/lib/asn1/asn1_err.c:47:23: error: 'link' redeclared as different kind of symbol
47 | static struct et_list link = { 0, 0 };
| ^~~~
In file included from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/bits/sigstksz.h:24,
from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/signal.h:328,
from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/sys/param.h:28,
from ../../lib/replace/../replace/replace.h:659,
from ../../source4/heimdal_build/config.h:10,
from source4/heimdal/lib/asn1/asn1_err.c:1:
/home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/unistd.h:819:12: note: previous declaration of 'link' was here
819 | extern int link (const char *__from, const char *__to)
| ^~~~
Fixes:
- http://autobuild.buildroot.org/results/d6de6ed59c553f6f413d280f3c65184945bb3850
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/e2fsprogs/e2fsprogs.mk | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
index ca891066a2..76215e8c34 100644
--- a/package/e2fsprogs/e2fsprogs.mk
+++ b/package/e2fsprogs/e2fsprogs.mk
@@ -90,5 +90,11 @@ define HOST_E2FSPROGS_INSTALL_CMDS
$(HOST_MAKE_ENV) $(MAKE1) -C $(@D) install install-libs
endef
+# Remove compile_et which raises a build failure with samba4
+define HOST_E2FSPROGS_REMOVE_COMPILE_ET
+ $(RM) $(HOST_DIR)/bin/compile_et
+endef
+HOST_E2FSPROGS_POST_INSTALL_HOOKS += HOST_E2FSPROGS_REMOVE_COMPILE_ET
+
$(eval $(autotools-package))
$(eval $(host-autotools-package))
--
2.33.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Buildroot] [PATCH 2/3] package/heimdal: install compile_et
2021-10-19 17:25 [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et Fabrice Fontaine
@ 2021-10-19 17:25 ` Fabrice Fontaine
2021-12-02 17:35 ` Peter Korsgaard
2021-12-12 11:49 ` Peter Korsgaard
2021-10-19 17:25 ` [Buildroot] [PATCH 3/3] package/samba4: drop third patch Fabrice Fontaine
` (2 subsequent siblings)
3 siblings, 2 replies; 9+ messages in thread
From: Fabrice Fontaine @ 2021-10-19 17:25 UTC (permalink / raw)
To: buildroot; +Cc: Fabrice Fontaine
compile_et is not installed anymore by heimdal since version 7.0.1 and
https://github.com/heimdal/heimdal/commit/e40ef47b57f3c62f94c4056f02ced3af2ed083ca
So remove the broken symlink and install it manually as it is needed for
samba4 because compile_et provided by e2fsprogs raises a build failure
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/heimdal/heimdal.mk | 15 +++++++++++----
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/package/heimdal/heimdal.mk b/package/heimdal/heimdal.mk
index 9a1a68f046..ce509c7afa 100644
--- a/package/heimdal/heimdal.mk
+++ b/package/heimdal/heimdal.mk
@@ -28,19 +28,26 @@ HOST_HEIMDAL_CONF_OPTS = \
--disable-ndbm-db \
--disable-heimdal-documentation
-HOST_HEIMDAL_CONF_ENV = MAKEINFO=true
+# Don't use compile_et from e2fsprogs as it raises a build failure with samba4
+HOST_HEIMDAL_CONF_ENV = ac_cv_prog_COMPILE_ET=no MAKEINFO=true
HEIMDAL_LICENSE = BSD-3-Clause
HEIMDAL_LICENSE_FILES = LICENSE
HEIMDAL_CPE_ID_VENDOR = heimdal_project
+# We need compile_et for samba4
+define HOST_HEIMDAL_INSTALL_COMPILE_ET
+ $(INSTALL) -m 0755 $(@D)/lib/com_err/compile_et \
+ $(HOST_DIR)/bin/compile_et
+endef
+
# We need asn1_compile in the PATH for samba4
define HOST_HEIMDAL_MAKE_SYMLINK
ln -sf $(HOST_DIR)/libexec/heimdal/asn1_compile \
$(HOST_DIR)/bin/asn1_compile
- ln -sf $(HOST_DIR)/bin/compile_et \
- $(HOST_DIR)/libexec/heimdal/compile_et
endef
-HOST_HEIMDAL_POST_INSTALL_HOOKS += HOST_HEIMDAL_MAKE_SYMLINK
+HOST_HEIMDAL_POST_INSTALL_HOOKS += \
+ HOST_HEIMDAL_INSTALL_COMPILE_ET \
+ HOST_HEIMDAL_MAKE_SYMLINK
$(eval $(host-autotools-package))
--
2.33.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Buildroot] [PATCH 3/3] package/samba4: drop third patch
2021-10-19 17:25 [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et Fabrice Fontaine
2021-10-19 17:25 ` [Buildroot] [PATCH 2/3] package/heimdal: " Fabrice Fontaine
@ 2021-10-19 17:25 ` Fabrice Fontaine
2021-12-02 17:35 ` Peter Korsgaard
2021-12-12 11:49 ` Peter Korsgaard
2021-12-02 17:33 ` [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et Peter Korsgaard
2021-12-12 11:49 ` Peter Korsgaard
3 siblings, 2 replies; 9+ messages in thread
From: Fabrice Fontaine @ 2021-10-19 17:25 UTC (permalink / raw)
To: buildroot; +Cc: Fabrice Fontaine
Drop third patch as it is not needed when using compile_et from heimdal
instead of compile_et from e2fprogs as spotted by upstream in:
https://bugzilla.samba.org/show_bug.cgi?id=13856#c20
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
.../samba4/0003-Fix-unistd.h-include.patch | 38 -------------------
...t.c-include-stdint.h-before-cmoka.h.patch} | 0
...patch => 0004-dcesrv_core-fix-build.patch} | 0
3 files changed, 38 deletions(-)
delete mode 100644 package/samba4/0003-Fix-unistd.h-include.patch
rename package/samba4/{0004-ldap_message_test.c-include-stdint.h-before-cmoka.h.patch => 0003-ldap_message_test.c-include-stdint.h-before-cmoka.h.patch} (100%)
rename package/samba4/{0005-dcesrv_core-fix-build.patch => 0004-dcesrv_core-fix-build.patch} (100%)
diff --git a/package/samba4/0003-Fix-unistd.h-include.patch b/package/samba4/0003-Fix-unistd.h-include.patch
deleted file mode 100644
index f01b899a18..0000000000
--- a/package/samba4/0003-Fix-unistd.h-include.patch
+++ /dev/null
@@ -1,38 +0,0 @@
-From 85c33f89df8607ace0707e1360ff0f79f3e8f5f3 Mon Sep 17 00:00:00 2001
-From: Andy Walsh <andy.walsh44+github@gmail.com>
-Date: Wed, 6 Nov 2019 19:28:30 +0100
-Subject: [PATCH] Fix unistd.h include
-
-Fixes build error
-
-source4/heimdal/lib/asn1/asn1_err.c:47:23: error: 'link' redeclared as different kind of symbol
- static struct et_list link = { 0, 0 };
-
-Downloaded from
-https://github.com/Andy2244/openwrt-extra/blob/samba-4.10/samba4/patches/006-samba-4-10-musl_rm_unistd_incl.patch
-
-Upstream bug report: https://bugzilla.samba.org/show_bug.cgi?id=13856
-
-Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
----
- lib/replace/replace.h | 4 ----
- 1 file changed, 4 deletions(-)
-
-diff --git a/lib/replace/replace.h b/lib/replace/replace.h
-index 1658465866e..e6768c4ae51 100644
---- a/lib/replace/replace.h
-+++ b/lib/replace/replace.h
-@@ -168,10 +168,6 @@
- #include <bsd/unistd.h>
- #endif
-
--#ifdef HAVE_UNISTD_H
--#include <unistd.h>
--#endif
--
- #ifdef HAVE_STRING_H
- #include <string.h>
- #endif
---
-2.20.1
-
diff --git a/package/samba4/0004-ldap_message_test.c-include-stdint.h-before-cmoka.h.patch b/package/samba4/0003-ldap_message_test.c-include-stdint.h-before-cmoka.h.patch
similarity index 100%
rename from package/samba4/0004-ldap_message_test.c-include-stdint.h-before-cmoka.h.patch
rename to package/samba4/0003-ldap_message_test.c-include-stdint.h-before-cmoka.h.patch
diff --git a/package/samba4/0005-dcesrv_core-fix-build.patch b/package/samba4/0004-dcesrv_core-fix-build.patch
similarity index 100%
rename from package/samba4/0005-dcesrv_core-fix-build.patch
rename to package/samba4/0004-dcesrv_core-fix-build.patch
--
2.33.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et
2021-10-19 17:25 [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et Fabrice Fontaine
2021-10-19 17:25 ` [Buildroot] [PATCH 2/3] package/heimdal: " Fabrice Fontaine
2021-10-19 17:25 ` [Buildroot] [PATCH 3/3] package/samba4: drop third patch Fabrice Fontaine
@ 2021-12-02 17:33 ` Peter Korsgaard
2021-12-12 11:49 ` Peter Korsgaard
3 siblings, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2021-12-02 17:33 UTC (permalink / raw)
To: Fabrice Fontaine; +Cc: buildroot
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
> Don't install compile_et as it raises the following build failure with
> samba4 since, at least, bump to version 4.14.7 in commit
> 630e85f8f5ed64933b45599fc4332d9609177c5c:
> source4/heimdal/lib/asn1/asn1_err.c:47:23: error: 'link' redeclared as different kind of symbol
> 47 | static struct et_list link = { 0, 0 };
> | ^~~~
> In file included from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/bits/sigstksz.h:24,
> from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/signal.h:328,
> from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/sys/param.h:28,
> from ../../lib/replace/../replace/replace.h:659,
> from ../../source4/heimdal_build/config.h:10,
> from source4/heimdal/lib/asn1/asn1_err.c:1:
> /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/unistd.h:819:12: note: previous declaration of 'link' was here
> 819 | extern int link (const char *__from, const char *__to)
> | ^~~~
> Fixes:
> - http://autobuild.buildroot.org/results/d6de6ed59c553f6f413d280f3c65184945bb3850
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
> package/e2fsprogs/e2fsprogs.mk | 6 ++++++
> 1 file changed, 6 insertions(+)
> diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
> index ca891066a2..76215e8c34 100644
> --- a/package/e2fsprogs/e2fsprogs.mk
> +++ b/package/e2fsprogs/e2fsprogs.mk
> @@ -90,5 +90,11 @@ define HOST_E2FSPROGS_INSTALL_CMDS
> $(HOST_MAKE_ENV) $(MAKE1) -C $(@D) install install-libs
> endef
> +# Remove compile_et which raises a build failure with samba4
> +define HOST_E2FSPROGS_REMOVE_COMPILE_ET
> + $(RM) $(HOST_DIR)/bin/compile_et
> +endef
> +HOST_E2FSPROGS_POST_INSTALL_HOOKS += HOST_E2FSPROGS_REMOVE_COMPILE_ET
This is not so great as you then end up if without compile_et is
e2fsprogs is (re)built after heimdall - But OK, heimdall depends on
e2fsprogs so in a normal build this will not trigger.
Committed, thanks.
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et
2021-10-19 17:25 [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et Fabrice Fontaine
` (2 preceding siblings ...)
2021-12-02 17:33 ` [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et Peter Korsgaard
@ 2021-12-12 11:49 ` Peter Korsgaard
3 siblings, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2021-12-12 11:49 UTC (permalink / raw)
To: Fabrice Fontaine; +Cc: buildroot
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
> Don't install compile_et as it raises the following build failure with
> samba4 since, at least, bump to version 4.14.7 in commit
> 630e85f8f5ed64933b45599fc4332d9609177c5c:
> source4/heimdal/lib/asn1/asn1_err.c:47:23: error: 'link' redeclared as different kind of symbol
> 47 | static struct et_list link = { 0, 0 };
> | ^~~~
> In file included from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/bits/sigstksz.h:24,
> from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/signal.h:328,
> from /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/sys/param.h:28,
> from ../../lib/replace/../replace/replace.h:659,
> from ../../source4/heimdal_build/config.h:10,
> from source4/heimdal/lib/asn1/asn1_err.c:1:
> /home/giuliobenetti/autobuild/run/instance-1/output-1/host/s390x-buildroot-linux-gnu/sysroot/usr/include/unistd.h:819:12: note: previous declaration of 'link' was here
> 819 | extern int link (const char *__from, const char *__to)
> | ^~~~
> Fixes:
> - http://autobuild.buildroot.org/results/d6de6ed59c553f6f413d280f3c65184945bb3850
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Committed to 2021.02.x and 2021.08.x, thanks.
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-12-12 11:50 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-19 17:25 [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et Fabrice Fontaine
2021-10-19 17:25 ` [Buildroot] [PATCH 2/3] package/heimdal: " Fabrice Fontaine
2021-12-02 17:35 ` Peter Korsgaard
2021-12-12 11:49 ` Peter Korsgaard
2021-10-19 17:25 ` [Buildroot] [PATCH 3/3] package/samba4: drop third patch Fabrice Fontaine
2021-12-02 17:35 ` Peter Korsgaard
2021-12-12 11:49 ` Peter Korsgaard
2021-12-02 17:33 ` [Buildroot] [PATCH 1/3] package/e2fsprogs: don't install compile_et Peter Korsgaard
2021-12-12 11:49 ` Peter Korsgaard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox