Files
poky/meta/recipes-extended/bash/bash/mkbuiltins_have_stringize.patch
Alexander Kanavin 5fc9579940 bash: upgrade 5.2.21 -> 5.2.32
(From OE-Core rev: f70eebdf5b60d0ee7b1bbcdff1135801b2654a08)

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
2024-08-23 09:12:37 +01:00

37 lines
1.3 KiB
Diff

From d11685286144c2e5630545e435d11387b2908fd0 Mon Sep 17 00:00:00 2001
From: Christopher Larson <chris_larson@mentor.com>
Date: Wed, 14 Nov 2012 07:55:09 -0700
Subject: [PATCH] bash: fix mkbuiltins build failure
On hosts with FORTIFY_SOURCES, stringize support is required, as it's used by
the macros to wrap functions (e.g. read and open in unistd.h). Those wrappers
use the STRING() macro from unistd.h. A header in the bash sources overrides
the unistd.h macro to 'x' when HAVE_STRINGIZE is not defined, causing the
wrappers to generate calls to 'xread' and 'xopen', which do not exist,
resulting in a failure to link.
Assume we have stringize support when cross-compiling, which works around the
issue.
It may be best for upstream to either give up on supporting compilers without
stringize support, or to not define STRING() at all when FORTIFY_SOURCES is
defined, letting the unistd.h one be used, instead.
Upstream-Status: Pending
---
builtins/mkbuiltins.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/builtins/mkbuiltins.c b/builtins/mkbuiltins.c
index f505ebd..b5b2eed 100644
--- a/builtins/mkbuiltins.c
+++ b/builtins/mkbuiltins.c
@@ -28,6 +28,7 @@
# define HAVE_STDLIB_H
# define HAVE_RENAME
+# define HAVE_STRINGIZE
#endif /* CROSS_COMPILING */
#if defined (HAVE_UNISTD_H)