Linux Test Project
 help / color / mirror / Atom feed
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

      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