[PATCH v1 2/2] meson.build: use local include path for tests
Peter Hutterer
peter.hutterer at who-t.net
Thu Mar 5 22:02:18 UTC 2020
On Thu, Mar 05, 2020 at 09:03:44PM +0100, Peter Seiderer wrote:
> Hello Peter,
>
> On Thu, 5 Mar 2020 13:47:08 +1000, Peter Hutterer <peter.hutterer at who-t.net> wrote:
>
> > On Mon, Mar 02, 2020 at 04:16:28PM +0100, Peter Seiderer wrote:
> > > Fixes:
> > >
> > > ../test/test-common.h: In function ‘print_event’:
> > > ../test/test-common.h:97:6: error: ‘const struct input_event’ has no member named ‘input_event_sec’
> > > ev->input_event_sec,
> > > ^
> > > ../test/test-common.h:98:6: error: ‘const struct input_event’ has no member named ‘input_event_usec’
> > > ev->input_event_usec,
> > > ^
> > > ../test/test-common.h:102:6: error: ‘const struct input_event’ has no member named ‘input_event_sec’
> > > ev->input_event_sec,
> > > ^
> > > ../test/test-common.h:103:6: error: ‘const struct input_event’ has no member named ‘input_event_usec’
> > > ev->input_event_usec,
> > > ^
> > >
> > > Signed-off-by: Peter Seiderer <ps.report at gmx.net>
> >
> > applied, thanks. FTR it's easier if you submit a PR to gitlab directly
> > because that way we have the CI etc. run automatically.
> >
> > https://gitlab.freedesktop.org/libevdev/libevdev/-/merge_requests/42
> >
>
> O.k., will do the next time...., did take a look at [1] first which states
> 'Patches ... should be submitted to the input-tools at lists.freedesktop.org mailing list',
> maybe 'or alternatively/preferred submitted via gitlab merge requests if possible'?
hoh, indeed, outdated information. I pushed out an update for that now,
thanks.
>
> > What I do wonder though is why the CI didn't catch this and whether it's
> > worth adding a test case for this. I guess they're just older kernel
> > headers?
>
> It was (by incident) the same toolchain as used for another buildroot autobuild
> failure/fix, as pointed out by Romain Naour using a '... old codesourcery
> toolchain using kernel headers 3.13 ...' (see [2])...
I think I'll skip trying to figure out a CI pipeline for that then :)
Cheers,
Peter
More information about the Input-tools
mailing list