Re: [LTP] [PATCH v3 3/4] Makefile: Update 'doc' target, add 'doc-clean'

[Petr Vorel <pvorel@suse.cz>]

Hi Ricardo,

> On Mon Mar 24, 2025 at 8:40 PM -03, Petr Vorel wrote:
> > 'doc' target previously run docparse documentation. Point it to doc/
> > directory so that it build sphinx docs. doc/ dir has metadata/ dir as
> > dependency, no need to specify it. Call also '.venv' target.

> > NOTE: it's still possible to avoid virtualenv by calling 'make -C doc'

Thanks for testing.

> From the root dir ?

Yes, this is noted in the 4th patch
https://patchwork.ozlabs.org/project/ltp/patch/20250324234016.367228-5-pvorel@suse.cz/

> $ make -C doc
> make: Entering directory '/mnt/ext/src/linux/ltp/mail/doc'
> make -C /mnt/ext/src/linux/ltp/mail/metadata
> make[1]: Entering directory '/mnt/ext/src/linux/ltp/mail/metadata'
> /mnt/ext/src/linux/ltp/mail/metadata/parse.sh > ltp.json
> make[1]: Leaving directory '/mnt/ext/src/linux/ltp/mail/metadata'
> if [ -d .venv ]; then . .venv/bin/activate; fi; sphinx-build -b html . html
> /bin/sh: line 1: sphinx-build: command not found
> make: *** [Makefile:24: all] Error 127
> make: Leaving directory '/mnt/ext/src/linux/ltp/mail/doc'

As I wrote, I'll add sphinx to requrements.txt (as Andrea suggested).

Kind regards,
Petr

> > Add 'doc-clean': to remove only generated data (not optional .venv).

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp