On Thu, Aug 09, 2018 at 06:07:54PM +0100, Filipe Manana wrote: > On Thu, Aug 9, 2018 at 4:21 PM, Zorro Lang <zlang@xxxxxxxxxx> wrote: > > xfstests build fails after glibc updated to 2.28: > > > > /tmp/cc3s96Dw.o: In function `main': > > /root/xfstests-dev/src/stat_test.c:690: undefined reference to `xfstests_statx' > > /tmp/cc3s96Dw.o: In function `get_reference': > > /root/xfstests-dev/src/stat_test.c:301: undefined reference to `xfstests_statx' > > /tmp/cc3s96Dw.o: In function `main': > > /root/xfstests-dev/src/stat_test.c:642: undefined reference to `xfstests_statx' > > collect2: error: ld returned 1 exit status > > gmake[2]: *** [Makefile:82: stat_test] Error 1 > > gmake[1]: *** [include/buildrules:31: src] Error 2 > > make: *** [Makefile:53: default] Error 2 > > > > Due to glibc-headers-2.28 defines struct statx and all related > > things, include STATX_TYPE. But xfstests src/statx.h won't build > > xfstests_statx() function STATX_TYPE has been defined, that doesn't > > make sense. > > > > If all statx things has been defined by glibc, we can skip xfstests > > internal statx definition, but xfstests_statx() function is still > > needed, because it's called by other programs. > > > > Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx> > > Lu sent the same fix 2 days ago: > > https://patchwork.kernel.org/patch/10558653/ OK, Great :) Thanks, Zorro > > > > --- > > > > CC David Howells <dhowells@xxxxxxxxxx>, the original author of src/statx.h. > > For this change won't break his original design. > > > > Thanks, > > Zorro > > > > src/statx.h | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/src/statx.h b/src/statx.h > > index 5d58208a..f6aaafd6 100644 > > --- a/src/statx.h > > +++ b/src/statx.h > > @@ -160,6 +160,8 @@ struct statx { > > > > #define STATX_ATTR_AUTOMOUNT 0x00001000 /* Dir: Automount trigger */ > > > > +#endif /* STATX_TYPE */ > > + > > static inline > > int xfstests_statx(int dfd, const char *filename, unsigned flags, > > unsigned int mask, struct statx *buffer) > > @@ -172,5 +174,4 @@ int xfstests_statx(int dfd, const char *filename, unsigned flags, > > #endif > > } > > > > -#endif /* STATX_TYPE */ > > #endif /* STATX_H */ > > -- > > 2.14.4 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe fstests" in > > the body of a message to majordomo@xxxxxxxxxxxxxxx > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > -- > Filipe David Manana, > > “Whether you think you can, or you think you can't — you're right.” > -- > To unsubscribe from this list: send the line "unsubscribe fstests" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html
