Source: bash
Version: 5.2.37-1.1
Severity: important
Tags: ftbfs upstream
User: helm...@debian.org
Usertags: rebootstrap

Hi Matthias,

not sure how bash got missed in the archive rebuild, but it also FTBFS
with C23. Some pieces were contributed in 2023 already, but as it stands
there still are incompatible function declarations such as

| ../.././builtins/mkbuiltins.c:267:29: error: too many arguments to function 
'xmalloc'; expected 0, have 1
|   267 |           error_directory = xmalloc (2 + strlen (argv[arg_index]));
|       |                             ^~~~~~~  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| ../.././builtins/mkbuiltins.c:62:14: note: declared here
|    62 | static char *xmalloc (), *xrealloc ();
|       |              ^~~~~~~

This particular one is fixed on the devel branch upstream, but not on
the master branch upstream. The relevant commit
57d4dc15ff35895a1c1248f948f59739ffb99fde is a grab bag that also covers
asan fixes as well as autotoolizing, so it likely is not backportable as
is.

Would you mind a temporarily selecting gnu17 explicitly?

--- a/debian/rules
+++ b/debian/rules
@@ -41,7 +41,7 @@
 endif
 
 
-dpkg_buildflags = DEB_BUILD_MAINT_OPTIONS="hardening=+all" 
DEB_CFLAGS_MAINT_APPEND="-Wall" dpkg-buildflags
+dpkg_buildflags = DEB_BUILD_MAINT_OPTIONS="hardening=+all" 
DEB_CFLAGS_MAINT_APPEND="-Wall -std=gnu17" dpkg-buildflags
 CFLAGS := $(shell $(dpkg_buildflags) --get CFLAGS)
 CPPFLAGS := $(shell $(dpkg_buildflags) --get CPPFLAGS)
 LDFLAGS := $(shell $(dpkg_buildflags) --get LDFLAGS)

This builds and has few chances of introducing regressions as it merely
selects what is the default to the default compiler for trixie.

Helmut

Reply via email to