Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#13302 closed Patch - Bug Fix (fixed)

Configure check_lib updates for FFMpeg merge

Reported by: Gary Buhrmaster Owned by: Peter Bennett
Priority: minor Milestone: 30.0
Component: MythTV - configure script Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The FFMpeg resync in 24f44de changed the function check_lib calling sequence (it adds a name first param). There are a couple of check_lib calls there were not converted in the MythTV configure script.

To reproduce failure for the first case (without patch): ./configure --enable-mythlogserver No reproducer provided for the second case (the external mpeg2 library support is still a WIP I believe).

Proposed patch:

diff --git a/mythtv/configure b/mythtv/configure
index 32e0098b9f..7bfac0307a 100755
--- a/mythtv/configure
+++ b/mythtv/configure
@@ -5874,7 +5874,7 @@ enabled ffmpeg_pthreads && enable pthreads
 # Check for libuuid for zeromq
 enabled mythlogserver || disable libuuid
 if enabled libuuid ; then
-  check_lib uuid/uuid.h uuid_generate -luuid || disable libuuid
+  check_lib uuid uuid/uuid.h uuid_generate -luuid || disable libuuid
   if disabled libuuid ; then
     die "ERROR: can not find libuuid. (debian uuid-dev; RedHat libuuid-devel)"
   fi
@@ -5987,7 +5987,7 @@ enabled opengl            && { check_lib opengl GL/glx.h glXGetProcAddress "-lGL
                                die "ERROR: opengl not found."
                              }
 if enabled libmpeg2external; then
-    check_lib mpeg2dec/mpeg2.h mpeg2_init -lmpeg2 || disable libmpeg2external
+    check_lib libmpeg2external mpeg2dec/mpeg2.h mpeg2_init -lmpeg2 || disable libmpeg2external
     if disabled libmpeg2external; then
         die "Warning! No suitable external libmpeg2 found, use internal copy."
     fi

Change History (4)

comment:1 by Peter Bennett, 7 years ago

This regex finds the offending calls

check_lib *[^ ]*.h

It only found those two.

This will not happen in future - we are no longer merging ffmpeg new configure into MythTV configure when we resync ffmpeg.

Last edited 7 years ago by Peter Bennett (previous) (diff)

comment:2 by Peter Bennett, 7 years ago

Resolution: Fixed
Status: newclosed

Fixed in 4b1afb5e480

configure: fix bad check_lib calls.

The FFMpeg resync in 24f44de changed the function check_lib calling sequence (it adds a name first param). There are a couple of check_lib calls there were not converted in the MythTV configure script.

comment:3 by Gary Buhrmaster <gary.buhrmaster@…>, 7 years ago

Resolution: Fixedfixed

In 4b1afb5e48/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:4 by Stuart Auchterlonie, 7 years ago

Milestone: needs_triage30.0
Note: See TracTickets for help on using tickets.