docbook2x build failure

  • Done
  • quality assurance status badge
Details
3 participants
  • Greg Hogan
  • Bruno Victal
  • Steve George
Owner
unassigned
Submitted by
Greg Hogan
Severity
normal
G
G
Greg Hogan wrote on 22 Dec 2023 20:01
CA+3U0Zm91nfepEoA+z37kW2zCqCZwO4WSfnpzO=3DuC037Y-ug@mail.gmail.com
Build log attached. Am building from the most recent upstream commit
to core-updates (eff2e20688). bug#65479 "Docbook & XML/SGML
improvements" was merged to core-updates on October 10.

Toggle snippet (24 lines)
Making all in documentation
make[2]: Entering directory
'/tmp/guix-build-docbook2x-0.8.8.drv-0/docbook2X-0.8.8/xslt/documentation'
../../perl/db2x_xsltproc -C ../../xslt/catalog.xml -s ./ss-texi.xsl -o
docbook2man-xslt.txml docbook2man-xslt.xml \
-g output-file="docbook2man-xslt"
docbook2man-xslt.xml:1: parser error : Document is empty

^
unable to parse docbook2man-xslt.xml
make[2]: *** [Makefile:808: docbook2man-xslt.txml] Error 6
make[2]: Leaving directory
'/tmp/guix-build-docbook2x-0.8.8.drv-0/docbook2X-0.8.8/xslt/documentation'
make[1]: *** [Makefile:389: all-recursive] Error 1
make[1]: Leaving directory
'/tmp/guix-build-docbook2x-0.8.8.drv-0/docbook2X-0.8.8/xslt'
make: *** [Makefile:411: all-recursive] Error 1
error: in phase 'build': uncaught exception:
%exception #<&invoke-error program: "make" arguments: ("-j" "1")
exit-status: 2 term-signal: #f stop-signal: #f>
phase `build' failed after 0.4 seconds
command "make" "-j" "1" failed with status 2
build process 4 exited with status 256
B
B
Bruno Victal wrote on 29 Dec 2023 23:13
[PATCH core-updates 0/1] gnu: docbook2x: Fix build.
(name . Greg Hogan)(address . code@greghogan.com)
cover.1703887805.git.mirai@makinata.eu
Hi Greg,

Thanks for noticing, it turned out to be a regression introduced in v3.

Bruno Victal (1):
gnu: docbook2x: Fix build.

gnu/packages/docbook.scm | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)


base-commit: e769385a7e348c8ba97614d889d7e8625fd8226a
--
2.41.0
B
B
Bruno Victal wrote on 29 Dec 2023 23:13
[PATCH core-updates 1/1] gnu: docbook2x: Fix build.
(name . Greg Hogan)(address . code@greghogan.com)
9b0ef2546773ea8e8f8d7b843b4871deb8e52aa1.1703887805.git.mirai@makinata.eu
The documentation step fails to build if the original configure script is
removed.

* gnu/packages/docbook.scm (docbook2x)[source]: Preserve original configure
script.
[arguments]<#:phases>: Add 'autoreconf phase.
<#:modules>: Use %default-gnu-modules.
---
gnu/packages/docbook.scm | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)

Toggle diff (36 lines)
diff --git a/gnu/packages/docbook.scm b/gnu/packages/docbook.scm
index 3046846619..02e60222ac 100644
--- a/gnu/packages/docbook.scm
+++ b/gnu/packages/docbook.scm
@@ -912,21 +912,20 @@ (define-public docbook2x
(modules '((guix build utils)))
(snippet
;; Fix a failing test (maybe it worked with old texinfo?)
- #~(begin
- (substitute* "test/complete-manuals/at1.xml"
- (("<bridgehead>")
- "<bridgehead renderas=\"sect2\">"))
- ;; Force a new autoreconf run.
- (delete-file "configure")))))
+ #~(substitute* "test/complete-manuals/at1.xml"
+ (("<bridgehead>")
+ "<bridgehead renderas=\"sect2\">")))))
(outputs '("out" "doc"))
(build-system gnu-build-system)
(arguments
(list
- #:modules '((guix build gnu-build-system)
- (guix build utils)
- (srfi srfi-26))
+ #:modules `((srfi srfi-26)
+ ,@%default-gnu-modules)
#:phases
#~(modify-phases %standard-phases
+ (add-after 'unpack 'autoreconf
+ (lambda _
+ (invoke "autoreconf" "-vif")))
(add-after 'install 'move-doc
(lambda _
(let* ((old (string-append #$output "/share/doc"))
--
2.41.0
G
G
Greg Hogan wrote on 6 Mar 19:47 +0100
Re: [PATCH core-updates 0/1] gnu: docbook2x: Fix build.
(name . Bruno Victal)(address . mirai@makinata.eu)
CA+3U0Zk2GjqUgLGR84h9xFpoEvJfRPmjfnUz6VWG9rLG9uVWxQ@mail.gmail.com
On Fri, Dec 29, 2023 at 5:14?PM Bruno Victal <mirai@makinata.eu> wrote:
Toggle quote (16 lines)
>
> Hi Greg,
>
> Thanks for noticing, it turned out to be a regression introduced in v3.
>
> Bruno Victal (1):
> gnu: docbook2x: Fix build.
>
> gnu/packages/docbook.scm | 17 ++++++++---------
> 1 file changed, 8 insertions(+), 9 deletions(-)
>
>
> base-commit: e769385a7e348c8ba97614d889d7e8625fd8226a
> --
> 2.41.0

Hi Bruno,

Can this issue be closed? I am now able to build dockbook2x on
core-updates, and I believe this was fixed by Josselin Poiret in
d3a45711392fcecff640915c8bd881ff82559757 (also on 2023/12/29).


Greg
S
S
Steve George wrote on 22 Apr 11:39 +0200
block 70456 with 67973
(address . control@debbugs.gnu.org)
1713778748-3175-bts-steve@futurile.net
block 70456 with 67973
thanks
S
S
Steve George wrote on 23 Apr 15:26 +0200
Re: bug#67973: [PATCH core-updates 1/1] gnu: docbook2x: Fix build.
(name . Bruno Victal)(address . mirai@makinata.eu)
Zie2-KD0E0VGKY4V@dragon2
Hi,

Builds fine on current core-updates, so closing.

Steve / Futurile

On 29 Dec, Bruno Victal wrote:
Toggle quote (51 lines)
> The documentation step fails to build if the original configure script is
> removed.
>
> * gnu/packages/docbook.scm (docbook2x)[source]: Preserve original configure
> script.
> [arguments]<#:phases>: Add 'autoreconf phase.
> <#:modules>: Use %default-gnu-modules.
> ---
> gnu/packages/docbook.scm | 17 ++++++++---------
> 1 file changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/gnu/packages/docbook.scm b/gnu/packages/docbook.scm
> index 3046846619..02e60222ac 100644
> --- a/gnu/packages/docbook.scm
> +++ b/gnu/packages/docbook.scm
> @@ -912,21 +912,20 @@ (define-public docbook2x
> (modules '((guix build utils)))
> (snippet
> ;; Fix a failing test (maybe it worked with old texinfo?)
> - #~(begin
> - (substitute* "test/complete-manuals/at1.xml"
> - (("<bridgehead>")
> - "<bridgehead renderas=\"sect2\">"))
> - ;; Force a new autoreconf run.
> - (delete-file "configure")))))
> + #~(substitute* "test/complete-manuals/at1.xml"
> + (("<bridgehead>")
> + "<bridgehead renderas=\"sect2\">")))))
> (outputs '("out" "doc"))
> (build-system gnu-build-system)
> (arguments
> (list
> - #:modules '((guix build gnu-build-system)
> - (guix build utils)
> - (srfi srfi-26))
> + #:modules `((srfi srfi-26)
> + ,@%default-gnu-modules)
> #:phases
> #~(modify-phases %standard-phases
> + (add-after 'unpack 'autoreconf
> + (lambda _
> + (invoke "autoreconf" "-vif")))
> (add-after 'install 'move-doc
> (lambda _
> (let* ((old (string-append #$output "/share/doc"))
> --
> 2.41.0
>
>
>
>
S
S
Steve George wrote on 23 Apr 15:27 +0200
closing 67973
(address . control@debbugs.gnu.org)(address . 67973-submitter@debbugs.gnu.org)
1713878836-3687-bts-steve@futurile.net
close 67973
thanks
?