On 20/08/2020 14:42, Hans Verkuil wrote: > On 06/08/2020 17:55, Ariel D'Alessandro wrote: >> Supports building libraries and tools found in contrib/, lib/ and >> utils/ directories, along with the implemented gettext translations. >> >> Also supports generating plain HTML documentation and manual pages using >> Doxygen. >> >> Co-developed-by: Ezequiel Garcia <ezequiel@xxxxxxxxxxxxx> >> Signed-off-by: Ezequiel Garcia <ezequiel@xxxxxxxxxxxxx> >> Acked-by: Kieran Bingham <kieran.bingham+renesas@xxxxxxxxxxxxxxxx> >> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@xxxxxxxxxxxxxxxx> >> Acked-by: Gregor Jasny <gjasny@xxxxxxxxxxxxxx> >> [Gregor: Control symbol visibility] >> Signed-off-by: Gregor Jasny <gjasny@xxxxxxxxxxxxxx> >> Signed-off-by: Ariel D'Alessandro <ariel@xxxxxxxxxxxxxxxxxxxx> >> --- >> .gitignore | 1 + >> Makefile.am | 3 +- >> README.md | 85 +- >> contrib/Makefile.am | 3 +- >> contrib/cobalt-ctl/Makefile.am | 2 + I would also suggest that all the Makefile.am changes are split off into another patch as well. Those make it harder to review as well since most (all?) of those changes just add meson.build. Regards, Hans >> contrib/cobalt-ctl/meson.build | 8 + >> contrib/decode_tm6000/Makefile.am | 2 + >> contrib/decode_tm6000/meson.build | 14 + >> contrib/gconv/Makefile.am | 2 +- >> contrib/gconv/meson.build | 44 + >> contrib/meson.build | 13 + >> contrib/rds-saa6588/Makefile.am | 2 + >> contrib/rds-saa6588/meson.build | 7 + >> contrib/test/Makefile.am | 3 +- >> contrib/test/meson.build | 143 + >> contrib/xc3028-firmware/Makefile.am | 2 +- >> contrib/xc3028-firmware/meson.build | 11 + >> doc/Doxyfile.in | 2351 +++++++++++++++++ > > 2351 lines to add this file?! > > Is this really needed? And if so, please add this in a separate patch in the > next version. This huge file pollutes the diff, making it very hard to review. > > Regards, > > Hans >