On Mon, 2018-09-03 at 03:09 +0200, Thomas Monjalon wrote: > 31/08/2018 20:20, Luca Boccassi: > > Both a configuration-time "enable_docs" boolean option and an > > optional > > 'ninja doc' target are available. Note that due to a Meson bug for > > now > > the latter will only build, but not install the files. > > > > Signed-off-by: Luca Boccassi <bl...@debian.org> > > --- > > doc/api/generate_doxygen.sh | 10 ++++++++ > > doc/api/meson.build | 51 > > +++++++++++++++++++++++++++++++++++++ > > doc/build-sdk-meson.txt | 2 ++ > > doc/meson.build | 4 +++ > > meson.build | 3 +++ > > meson_options.txt | 2 ++ > > 6 files changed, 72 insertions(+) > > You use generate_doxygen.sh only with meson? > Shouldn't we use the same in mk/rte.sdkdoc.mk?
I was kinda hoping somebody would be able to suggest an alternative that avoids the need for generate_doxygen.sh - the issue being that with Meson's "custom_target" you can't specify a subdirectory in input/ouput (and the directory is generated at build time so can't have a meson.build in it), and also you cannot use the same "output" twice. So I can't see a way to have an additional custom_target to run the CSS generation, hence the script. Changing the makefile to use the script will make it a bit more complicated, as the doxygen file will need to be generated rather than passed by piping stout/in, so the rule will have to be split into 3: create directory -> create file -> call doxygen. Not sure it's worth it? -- Kind regards, Luca Boccassi
