From: Petr Vorel <pvorel@suse.cz>
To: Andrea Cervesato <andrea.cervesato@suse.com>
Cc: ltp@lists.linux.it
Subject: Re: [LTP] [PATCH 0/6] Remove asciidoc{,tor} doc
Date: Tue, 18 Feb 2025 13:28:44 +0100 [thread overview]
Message-ID: <20250218122844.GA2473190@pevik> (raw)
In-Reply-To: <6ecf5daa-95e3-43b9-9d04-fd7d2b2a0778@suse.com>
Hi Andrea, all,
> Hi Petr,
> On 2/7/25 11:55, Petr Vorel wrote:
> > Hi,
> > This is a follow up for Andrea's 1bf344a3db ("doc: add tests catalog page").
> > Besides these changes I plan:
> > 1) Configure readthedocs to generate html/pdf doc for user to be able to
> > download it.
> +1
Done already.
> > 2) Generate html/pdf documentation as a part of the release process
> > (build it locally with sphinx), to be able to add it to the release
> > as we did with the doc generated by asciidoc{,tor}.
> This is a bit tricky. Did you try already?
No. Not a big priority, but I'd like to do it before the release so that we can
upload generated archive again.
> > 3) Andrea refused to do have virtualenv support in Makefile.
> > https://lore.kernel.org/ltp/3032b376-8f7a-4b1c-8422-f5a61e59b680@suse.com/
> > I would still prefer to have it, as optional, when some variable is
> > passed to make. It's better than copy paste the commands all the time,
> > also it would help 2).
> Yeah, we can have sphinx + dependences packages installed in the system, via
> pip or using a Dockerfile. virtualenv is just an optional tool we don't want
> to be dependent from.
FYI implemented here (let's keep the discussion there)
https://patchwork.ozlabs.org/project/ltp/patch/20250211233552.1990618-1-pvorel@suse.cz/
...
> > delete mode 100755 docparse/testinfo.pl
> > delete mode 100644 m4/ax_prog_perl_modules.m4
> > delete mode 100644 m4/ltp-docparse.m4
> > rename {docparse => metadata}/README.md (100%)
> The rest looks good. Feel free to push it:
> Acked-by: Andrea Cervesato <andrea.cervesato@suse.com>
Thanks! FYI merged.
Kind regards,
Petr
> Andrea
--
Mailing list info: https://lists.linux.it/listinfo/ltp
prev parent reply other threads:[~2025-02-18 12:29 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-07 10:55 [LTP] [PATCH 0/6] Remove asciidoc{,tor} doc Petr Vorel
2025-02-07 10:55 ` [LTP] [PATCH 1/6] create-tarballs-metadata.sh: Remove metadata generation Petr Vorel
2025-02-07 10:55 ` [LTP] [PATCH 2/6] github: Stop building asciidoc{, tor} documentation Petr Vorel
2025-02-07 10:55 ` [LTP] [PATCH 3/6] ci: Don't install asciidoc{,tor} Petr Vorel
2025-02-07 10:55 ` [LTP] [PATCH 4/6] doc: Replace docparse info with sphinx Petr Vorel
2025-02-07 10:55 ` [LTP] [PATCH 5/6] Move docparse/README.md to metadata/ Petr Vorel
2025-02-07 10:55 ` [LTP] [PATCH 6/6] docparse: Remove Petr Vorel
2025-02-07 13:03 ` [LTP] [PATCH 0/6] Remove asciidoc{,tor} doc Andrea Cervesato via ltp
2025-02-18 12:28 ` Petr Vorel [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250218122844.GA2473190@pevik \
--to=pvorel@suse.cz \
--cc=andrea.cervesato@suse.com \
--cc=ltp@lists.linux.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox