mps| I've got this trying to build u-boot sandbox `include/os.h:269:40: error: unknown type name 'loff_t'; did you mean 'off_t'?`
@dalias| mps, what function is it trying to use loff_t with ?
... mps| dalias: let me look
... mps| dalias: int os_get_filesize(const char *fname, loff_t *size);
... mps| file include/os.h in u-boot
@dalias| ok, looks like they're just misusing loff_t for something it's not intended for because they want to pretend they're not relying on off_t being 64-bit
@dalias| loff_t basically just exists as a type for kernel interfaces that always use 64-bit offsets regardless of glibc possibly having 32-bit ones
@dalias| but if you don't want to change this function's interface contract you can try to get loff_t from somewhere
@dalias| musl only exposes it with the functions specified to use loff_t
@dalias| which are in fcntl.h
@dalias| and only under _GNU_SOURCE (which they're conditional on)
... mps| dalias: thanks. someone on #u-boot channel proposed that we should consider adding patch that conditionally defines loff_t on Alpine
... mps| I'm not sure about this
@dalias| ...
@dalias| a patch for uboot or for system headers?
... mps| iiuc poster in system headers
@dalias| the very existence of loff_t is a linuxism
@dalias| so them using it for their os_get_filesize abstraction makes no sense
@dalias| the type here should be off_t or intmax_t or something
... mps| dalias: I don't think it is much important, it is not used to build any u-boot binary for boards/devices but only in building u-boot sandbox
@dalias| yeah
@dalias| do they use autoconf or something else?
@dalias| you could probably just add -Dloff_t=off_t to CFLAGS, btw
... mps| no, make $device_defconfig like linux kernel
@dalias| :-P