[LTP] [PATCH 1/1] ci: Don't test doc generation on Debian oldstable

[LTP] [PATCH 1/1] ci: Don't test doc generation on Debian oldstable

[Petr Vorel]

HTML generation with asciidoc is broken on Debian 11 bullseye
(the new Debian oldstable):

a2x: ERROR: "xsltproc" --stringparam toc.section.depth 1 --stringparam
callout.graphics 0 --stringparam navig.graphics 0 --stringparam
admon.textlabel 1 --stringparam admon.graphics 0 --stringparam
chunk.section.depth 0  --output "/__w/ltp/ltp/docparse/metadata.html"
"/etc/asciidoc/docbook-xsl/xhtml.xsl"
"/__w/ltp/ltp/docparse/metadata.xml" returned non-zero exit status 5

First reason is that docbook2x is needed to get required XSL (broken
dependency, which is fixed on newer Debian versions).

But even with it being installed, it still does not work on GitHub
Actions, due somehow special network, e.g. ping 8.8.8.8 does not work
(firewall?), code working in distro running via podman locally fails on CI:

$ cd metadata && make
error : Unknown IO error
warning: failed to load external entity "http://docbook.sourceforge.net/release/xsl/current/xhtml/docbook.xsl"
compilation error: file /etc/asciidoc/docbook-xsl/xhtml.xsl line 12 element import
xsl:import : unable to load http://docbook.sourceforge.net/release/xsl/current/xhtml/docbook.xsl

Because asciidoc is tested on other distros, simply remove the test.

Reported-by: Li Wang <liwang@redhat.com>
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 .github/workflows/ci.yml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index d8e5dca86..d7e9f2dd0 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -1,4 +1,4 @@
-# Copyright (c) 2021 Petr Vorel <pvorel@suse.cz>
+# Copyright (c) 2021-2023 Petr Vorel <pvorel@suse.cz>
 
 name: "CI: docker based builds"
 on: [push, pull_request]
@@ -79,7 +79,6 @@ jobs:
           - container: "debian:oldstable"
             env:
               CC: clang
-              METADATA: asciidoc-pdf
 
           - container: "opensuse/leap"
             env:
-- 
2.40.1


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

Re: [LTP] [PATCH 1/1] ci: Don't test doc generation on Debian oldstable

[Li Wang]

Hi Petr,

Thanks so much for the debugging work.

Reviewed-by: Li Wang <liwang@redhat.com>

On Tue, Jun 20, 2023 at 12:36 AM Petr Vorel <pvorel@suse.cz> wrote:

> HTML generation with asciidoc is broken on Debian 11 bullseye
> (the new Debian oldstable):
>
> a2x: ERROR: "xsltproc" --stringparam toc.section.depth 1 --stringparam
> callout.graphics 0 --stringparam navig.graphics 0 --stringparam
> admon.textlabel 1 --stringparam admon.graphics 0 --stringparam
> chunk.section.depth 0  --output "/__w/ltp/ltp/docparse/metadata.html"
> "/etc/asciidoc/docbook-xsl/xhtml.xsl"
> "/__w/ltp/ltp/docparse/metadata.xml" returned non-zero exit status 5
>
> First reason is that docbook2x is needed to get required XSL (broken
> dependency, which is fixed on newer Debian versions).
>
> But even with it being installed, it still does not work on GitHub
> Actions, due somehow special network, e.g. ping 8.8.8.8 does not work
> (firewall?), code working in distro running via podman locally fails on CI:
>
> $ cd metadata && make
> error : Unknown IO error
> warning: failed to load external entity "
> http://docbook.sourceforge.net/release/xsl/current/xhtml/docbook.xsl"
> compilation error: file /etc/asciidoc/docbook-xsl/xhtml.xsl line 12
> element import
> xsl:import : unable to load
> http://docbook.sourceforge.net/release/xsl/current/xhtml/docbook.xsl
>
> Because asciidoc is tested on other distros, simply remove the test.
>
> Reported-by: Li Wang <liwang@redhat.com>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>  .github/workflows/ci.yml | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
> index d8e5dca86..d7e9f2dd0 100644
> --- a/.github/workflows/ci.yml
> +++ b/.github/workflows/ci.yml
> @@ -1,4 +1,4 @@
> -# Copyright (c) 2021 Petr Vorel <pvorel@suse.cz>
> +# Copyright (c) 2021-2023 Petr Vorel <pvorel@suse.cz>
>
>  name: "CI: docker based builds"
>  on: [push, pull_request]
> @@ -79,7 +79,6 @@ jobs:
>            - container: "debian:oldstable"
>              env:
>                CC: clang
> -              METADATA: asciidoc-pdf
>
>            - container: "opensuse/leap"
>              env:
> --
> 2.40.1
>
>

-- 
Regards,
Li Wang

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

Re: [LTP] [PATCH 1/1] ci: Don't test doc generation on Debian oldstable

[Petr Vorel]

Hi Li,

> Hi Petr,

> Thanks so much for the debugging work.

Thanks for your review, merged.
CI should be green again in few minutes.

Kind regards,
Petr

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