On Mon, Mar 25, 2024 at 10:53:22AM -0700, Khem Raj wrote: > basename prototype has been removed from string.h from latest musl [1] > compilers e.g. clang-18 flags the absense of prototype as error. therefore > include libgen.h for providing it. > > [1] https://git.musl-libc.org/cgit/musl/commit/?id=725e17ed6dff4d0cd22487bb64470881e86a92e7 Same question as I had for your xfsdump patchset -- the manpage for basename(3) mentions that old glibc posix implementations would modify the argument. Is that still true, or has that not been the case for long enough that it's ok not to care? --D > Signed-off-by: Khem Raj <raj.khem@xxxxxxxxx> > --- > src/splice-test.c | 1 + > src/stat_test.c | 1 + > src/t_ext4_dax_inline_corruption.c | 1 + > src/t_ext4_dax_journal_corruption.c | 1 + > src/t_mmap_collision.c | 1 + > 5 files changed, 5 insertions(+) > > diff --git a/src/splice-test.c b/src/splice-test.c > index eb863673..7b7094f5 100644 > --- a/src/splice-test.c > +++ b/src/splice-test.c > @@ -11,6 +11,7 @@ > #include <unistd.h> > #include <fcntl.h> > #include <err.h> > +#include <libgen.h> > > #include <stdlib.h> > #include <stdio.h> > diff --git a/src/stat_test.c b/src/stat_test.c > index 95cfdc4f..77e1acdd 100644 > --- a/src/stat_test.c > +++ b/src/stat_test.c > @@ -4,6 +4,7 @@ > * Written by David Howells (dhowells@xxxxxxxxxx) > */ > > +#include <libgen.h> > #include <stdarg.h> > #include <stdio.h> > #include <stdlib.h> > diff --git a/src/t_ext4_dax_inline_corruption.c b/src/t_ext4_dax_inline_corruption.c > index e1a39a6c..a40c5df5 100644 > --- a/src/t_ext4_dax_inline_corruption.c > +++ b/src/t_ext4_dax_inline_corruption.c > @@ -2,6 +2,7 @@ > /* Copyright (c) 2018 Intel Corporation. */ > #include <errno.h> > #include <fcntl.h> > +#include <libgen.h> > #include <stdio.h> > #include <stdlib.h> > #include <string.h> > diff --git a/src/t_ext4_dax_journal_corruption.c b/src/t_ext4_dax_journal_corruption.c > index ba7a96e4..53be10c2 100644 > --- a/src/t_ext4_dax_journal_corruption.c > +++ b/src/t_ext4_dax_journal_corruption.c > @@ -2,6 +2,7 @@ > /* Copyright (c) 2018 Intel Corporation. */ > #include <errno.h> > #include <fcntl.h> > +#include <libgen.h> > #include <stdio.h> > #include <stdlib.h> > #include <string.h> > diff --git a/src/t_mmap_collision.c b/src/t_mmap_collision.c > index c872f4e2..d9315fcf 100644 > --- a/src/t_mmap_collision.c > +++ b/src/t_mmap_collision.c > @@ -15,6 +15,7 @@ > */ > #include <errno.h> > #include <fcntl.h> > +#include <libgen.h> > #include <pthread.h> > #include <stdbool.h> > #include <stdio.h> > -- > 2.44.0 > >
