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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.