On Sun, 14 Feb 2021 13:52:25 -0800 Richard Henderson <richard.hender...@linaro.org> wrote:
> Well, first off, this fails to build with our gitlab CI. > > There are several problems, apart from the new bison parse.error > option previously reported: > > (1) bison and flex not added to dockerfiles, so the packages are not > available to CI. > > (2) idef-parser program not marked with "native: true", so we get an > early meson error about that. > > (3) flex = generator() does not specify both output files, so the > proper dependency on idef-parser.yy.h is not present. > > (4) idef-parser.yy.h is placed in the wrong directory, because you > used '%option header-file="xyz"' instead of the command-line > '--headerfile=@OUTPUT1@'. This results in an immediate compilation > failure, because the file isn't found. > > (5) There's an odd mismatch in file naming, in which you have > idef-lexer.lex.yy.c but idef-parser.yy.h. Not a bug, really, but it > looks odd when fixing 3 & 4. We've integrated the suggested changes and tested the v2 on the Gitlab CI. It should now work properly. -- Alessandro Di Federico rev.ng
