Re: [libvirt PATCH 321/351] meson: docs: introduce meson-html-gen.py helper

On Tue, Jul 28, 2020 at 15:57:16 +0200, Pavel Hrdina wrote: > On Tue, Jul 28, 2020 at 03:18:23PM +0200, Peter Krempa wrote: > > On Thu, Jul 16, 2020 at 11:59:17 +0200, Pavel Hrdina wrote: > > > Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com&gt; > > > --- > > > docs/Makefile.am | 26 --------------------- > > > scripts/meson-html-gen.py | 49 +++++++++++++++++++++++++++++++++++++++ > > > scripts/meson.build | 1 + > > > 3 files changed, 50 insertions(+), 26 deletions(-) > > > create mode 100755 scripts/meson-html-gen.py [...] > > > +) > > > + > > > +html = subprocess.run( > > > + [args.xmllint, '--nonet', '--format', '-'], > > > + input=html_tmp.stdout, > > > + stdout=subprocess.PIPE, > > > + stderr=subprocess.PIPE, > > > > We can then do this as separate stage. > > What do you mean by separate stage? Not following here. I was arguing that the reformatting done by xmllint can be done as a separate build target thus eliminating the need to have this script completely.