Opened 6 months ago

Closed 6 months ago

#10636 closed defect (fixed)

Building ffmpeg fails at the documentation build phase

Reported by: hydra3333 Owned by:
Priority: normal Component: build system
Version: git-master Keywords:
Cc: Sebastian Ramacher Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Summary of the bug:
Building ffmpeg fails at the documentation build phase.
Built via the ubiquitous MABS at ​https://github.com/m-ab-s/media-autobuild_suite
Error log at ​https://github.com/m-ab-s/media-autobuild_suite/issues/2551

10:42:35 ├ Running configure...
10:44:00 ├ Running make...
Likely error (tail of the failed operation logfile):
makeinfo: error parsing /build/ffmpeg-git/doc/t2h.pm: Undefined subroutine &Texinfo::Config::set_from_init_file called at /build/ffmpeg-git/doc/t2h.pm line 24.
makeinfo: error parsing /build/ffmpeg-git/doc/t2h.pm: Undefined subroutine &Texinfo::Config::set_from_init_file called at /build/ffmpeg-git/doc/t2h.pm line 24.
make: *** [/build/ffmpeg-git/doc/Makefile:75: doc/ffmpeg-all.html] Error 1
make: *** [/build/ffmpeg-git/doc/Makefile:71: doc/ffmpeg.html] Error 1
STRIP	libavutil/x86/tx_float.o
skipping strip -x libavutil/x86/tx_float.o
AR	libavutil/libavutil.a
HTML	doc/ffmpeg.html
makeinfo: error parsing /build/ffmpeg-git/doc/t2h.pm: Undefined subroutine &Texinfo::Config::set_from_init_file called at /build/ffmpeg-git/doc/t2h.pm line 24.
make: *** [/build/ffmpeg-git/doc/Makefile:71: doc/ffmpeg.html] Error 1

How to reproduce:
Attempt to build ffmpeg via MABS (it used to work fine)

Change History (7)

comment:1 by hydra3333, 6 months ago

Version: unspecified → git-master

comment:2 by Sebastian Ramacher, 6 months ago

This is caused by changes in texinfo 7.1 and also affects ffmpeg 6.0 and other releases.

comment:3 by hydra3333, 6 months ago

Ah. Thank you.

For the time being I suppose we could use a prior version of texinfo, but I guess changes to ffmpeg may be required to make the issue go away ...

Last edited 6 months ago by hydra3333 (previous) (diff)

comment:4 by Sebastian Ramacher, 6 months ago

I have applied this workaround in Debian: ​https://salsa.debian.org/multimedia-team/ffmpeg/-/blob/debian/master/debian/patches/0005-Patch-up-t2h-for-texinfo-7.1.patch

It's not perfect, but at least the documentation can be built again.

comment:5 by Sebastian Ramacher, 6 months ago

Cc: Sebastian Ramacher added

comment:6 by Frank Plowman, 6 months ago

Patch to fix this in a backwards-compatible manner at: ​http://ffmpeg.org/pipermail/ffmpeg-devel/2023-November/316605.html

comment:7 by hydra3333, 6 months ago

Resolution: → fixed
Status: new → closed

Beaut !
Thanks

Note: See TracTickets for help on using tickets.