Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#25803 closed defect (fixed)

ffmpeg fails to build if texi2html 5.0 is installed

Reported by: takanori@… Owned by: dbevans (David B. Evans)
Priority: Normal Milestone:
Component: ports Version: 1.9.1
Keywords: Cc: jmroot (Joshua Root), raimue (Rainer Müller), sck-nogas (Scott C. Kennedy)
Port: ffmpeg texi2html

Description

After updating port:texi2html to @5.0, ffmpeg fails to build due to the following error.

Seems that this is because texi2html 5.0 does not support '-number' option any longer.

Attachments (3)

buildlog_ffmpeg.txt (785.8 KB) - added by takanori@… 14 years ago.
ffmpeg.diff (937 bytes) - added by takanori@… 14 years ago.
ffmpeg-docs.diff (1.6 KB) - added by ryandesign (Ryan Carsten Schmidt) 14 years ago.
a patch we could apply

Download all attachments as: .zip

Change History (11)

Changed 14 years ago by takanori@…

Attachment: buildlog_ffmpeg.txt added

comment:1 Changed 14 years ago by takanori@…

Here is a patch to avoid this problem.

# Note that the output (html) is not changed even if these '-monolithic' and '-number' options are removed.

Changed 14 years ago by takanori@…

Attachment: ffmpeg.diff added

comment:2 Changed 14 years ago by jmroot (Joshua Root)

This port is openmaintainer, so you should feel free to commit your fix.

comment:3 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Has duplicate #25885.

comment:4 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Has duplicate #25888.

comment:5 Changed 14 years ago by jeremyhu (Jeremy Huddleston Sequoia)

I added --disable-doc to configure.args (r70180) and will leave this with you to see if you want to add a variant for the documentation or just leave it as is (curently, documentation was only building when texi2html was present at build time... and it wasn't a build time dependent).

comment:6 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

On my Snow Leopard system at least, the system's texi2html is enough to build the documentation. So the documentation was getting built before, even if the texi2html port was not installed. When texi2html 5 is installed, then we need to patch the Makefile to make it continue to work. Attached is a patch which reenables the documentation, if we'd like to do that. We may want to use "port:texi2html" instead for consistency.

Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: ffmpeg-docs.diff added

a patch we could apply

comment:7 Changed 14 years ago by raimue (Rainer Müller)

Cc: raimue@… added
Resolution: fixed
Status: newclosed

As we don't know if the texi2html version on each version of Mac OS X produces the same output and parameters have changed between versions, we should rely on our own texi2html.

Similar patch committed in r70184.

comment:8 Changed 14 years ago by sck-nogas (Scott C. Kennedy)

Cc: sck@… added

Cc Me!

Note: See TracTickets for help on using tickets.