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
?
Your comment

This issue is archived.

To comment on this conversation send an email to 67973@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 67973
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch