On Tue, Apr 7, 2020 at 5:19 AM Petr Štetiar <yn...@true.cz> wrote: > > Eneas U de Queiroz <cotequei...@gmail.com> [2020-04-06 17:10:30]: > > Hi, > > > TLDR: You can't review this by only looking at the patches. > > Just tried to build[1] test it on x86/64 sunxi/a53 imx6 ipq40xx and it fails > to build: > > make -s -C scripts/config conf CC=cc: build failed. > > 1. https://gitlab.com/ynezz/openwrt/pipelines/133543034 > > -- ynezz
Thanks for trying to test this. I'm in the dark here with exactly what went wrong, but I've caught an oversight on my part: Linux now requires flex & bison to build files that it used to ship prebuilt. I can either restore the previous behavior, or we can require them as well, and then I'll add a check for them--and later perhaps remove them from tools/? What do you think? BTW, here's the output of that make subcommand if you take out '-s' in include/toplevel.mk:107: cc -O2 -c -o conf.o conf.c cc -O2 -c -o confdata.o confdata.c cc -O2 -c -o expr.o expr.c bison -l -d -b parser parser.y flex -L -olexer.lex.c lexer.l cc -O2 -I ./. -c -o lexer.lex.o lexer.lex.c cc -O2 -I ./. -c -o parser.tab.o parser.tab.c cc -O2 -c -o preprocess.o preprocess.c cc -O2 -c -o symbol.o symbol.c cc -O2 -c -o util.o util.c cc conf.o confdata.o expr.o lexer.lex.o parser.tab.o preprocess.o symbol.o util.o -o conf rm lexer.lex.c That's why I assume it's missing bison & flex. Eneas _______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel