Re: [LTP] [PATCH 7/7] docparse/Makefile: Do not abort on missing generators

Petr Vorel
2021-10-27T09:48:14Z

> Hi!
> > > Since we want to use the metadata.json in the testrunner it must bte
> > nit: typo s/bte/be

> > > build unconditionally.
> > According to docs:
> > --with-metadata-generator=asciidoc|asciidoctor
> > --disable-metadata Disable metadata generation (both HTML and PDF,
> > default no)
> > --disable-metadata-html Disable metadata HTML generation (default no)
> > --enable-metadata-pdf Enable metadata PDF generation (default no)

> > I'd like to have single option which does not create HTML and PDF (thus does not
> > require asciidoc|asciidoctor. I wonder if we could change build system to keep
> > --disable-metadata, build always docparse and json as we need, but with
> > ./configured --disable-metadata just not run HTML and PDF generation?

> I'm starting to think that the docparse.c should be moved to a
> metadata/paraser.c and that we should keep the documentation generators
> separate from the parser, that would allow us to keep all the configure
> switches while metadata.json would be build and installed
> unconditionally.

Good idea, go ahead with a patch please.

Kind regards,
Petr