On Mon, Mar 30, 2020 at 5:28 PM Fabio Estevam <feste...@gmail.com> wrote: > > Hi Rob, > > On Mon, Mar 30, 2020 at 6:29 PM Fabio Estevam <feste...@gmail.com> wrote: > > > > When building kmscube in Buildroot for ARM the following > > errors are seen: > > > > ../common.c: In function 'get_time_ns': > > ../common.c:376:18: error: storage size of 'tv' isn't known > > struct timespec tv; > > ^~ > > ../common.c:377:2: warning: implicit declaration of function > > 'clock_gettime'; did you mean 'localtime'? [-Wimplicit-function-declaration] > > clock_gettime(CLOCK_MONOTONIC, &tv); > > ^~~~~~~~~~~~~ > > localtime > > ../common.c:377:16: error: 'CLOCK_MONOTONIC' undeclared (first use in this > > function) > > clock_gettime(CLOCK_MONOTONIC, &tv); > > > > Fix it by using the default for each compiler on every platform instead. > > > > Inspired by this gst-plugins-good commit: > > > > https://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/?id=19f6559582c73123a3ec1fcf5a6b8651fbc2e83f > > > > Signed-off-by: Fabio Estevam <feste...@gmail.com> > > --- > > meson.build | 6 +----- > > 1 file changed, 1 insertion(+), 5 deletions(-) > > > > diff --git a/meson.build b/meson.build > > index b8131db..0f52dfe 100644 > > --- a/meson.build > > +++ b/meson.build > > @@ -26,13 +26,9 @@ project( > > version : '0.0.1', > > license : 'MIT', > > meson_version : '>= 0.47', > > - default_options : ['c_std=c99', 'warning_level=2'] > > + default_options : ['warning_level=2'] > > c99 was set in commit 6cbd03ab9406 ("Makefile.am: Add -std=c99 to > CFLAGS") to fix build failure on mips64el: > > cube-tex.c:230:2: note: use option -std=c99 or -std=gnu99 to compile your code > > If we change c_std=gnu99 then we could make both mips64el and ARM happy. > > Will send a v2 with c_std=gnu99. >
thx.. I don't suppose I could talk you in to sending a gitlab merge request? BR, -R _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
