On Mon, Jul 21, 2025 at 10:42:10PM +0100, Gavin Smith wrote: > On Mon, Jul 21, 2025 at 11:02:17PM +0200, Bruno Haible via Bug reports for > the GNU Texinfo documentation system wrote: > > The CI reports a compilation error on NetBSD 10.0: > > > > gcc -DHAVE_CONFIG_H -I. -I../../info -I.. -I../.. > > -I../../gnulib/lib -I../gnulib/lib > > -DLOCALEDIR=\"/usr/local/share/locale\" > > -DINFODIR=\"/usr/local/share/info\" > > -DINFODIR2=\"/usr/local/share/info\" > > -DSYSCONFDIR=\"/usr/local/etc\" -I/usr/pkg/include -Wall -g -O2 -MT > > run-external.o -MD -MP -MF .deps/run-external.Tpo -c -o run-external.o > > ../../info/run-external.c > > ../../info/run-external.c: In function ‘read_from_fd’: > > ../../info/run-external.c:124:18: error: storage size of ‘timeout’ isn’t > > known > > 124 | struct timeval timeout; > > | ^~~~~~~ > > > > 'struct timeval' is defined in <sys/time.h>. > > > > Bruno > > Thanks. I moved some code around and evidently didn't update the needed > #include's appropriately. It compiled on my system so I assumed it was > ok.
I don't really have a good idea how to manage header files in C programs especially when moving code about. I've downloaded and run 'deheader' (*) to try to help find if there were any includes that could be removed, manually reviewed its suggestions and made some changes. There are possibly other tools that do a similar thing. (*) http://www.catb.org/esr/deheader/
