Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#13109 closed Patch - Feature (Won't Fix)

Feature patch - Fix FIXME in libhdhomerun.pro

Reported by: Gary Buhrmaster <gary.buhrmaster@…> Owned by: David Hampton
Priority: minor Milestone: 30.0
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Feature patch - Fix FIXME in libhdhomerun.pro

Commit 5ed7db9 modified the mythtv specific libhdhomerun.pro file to work around an issue with QMAKE_STRIP being empty by commenting out the strip.

Re-enable the strip when possible by adding in a test (!isEmpty) to the scope.

Lightly tested.

Proposed Patch follows: 

diff --git a/mythtv/external/libhdhomerun/libhdhomerun.pro b/mythtv/external/libhdhomerun/libhdhomerun.pro
index c0251bbf48..6a33dfb815 100644
--- a/mythtv/external/libhdhomerun/libhdhomerun.pro
+++ b/mythtv/external/libhdhomerun/libhdhomerun.pro
@@ -89,8 +89,7 @@ Makefile.app {
 
     QMAKE_CLEAN += $(TARGET)
 
-    # FIXME QMAKE_STRIP is empty sometimes, skip for now
-    # unix:QMAKE_POST_LINK=$${QMAKE_STRIP} $(TARGET)
+    unix: !isEmpty(QMAKE_STRIP): QMAKE_POST_LINK=$${QMAKE_STRIP} $(TARGET)
     
     SOURCES += hdhomerun_config.c

Change History (3)

comment:1 by David Hampton, 7 years ago

Owner: set to David Hampton
Status: new → assigned

comment:2 by David Hampton, 7 years ago

Resolution: → Won't Fix
Status: assigned → closed

Commit 27fec2baba removed the embedded libhdhomerun. MythTV now uses the system supplied libhdhomerun library.

comment:3 by Stuart Auchterlonie, 7 years ago

Milestone: needs_triage → 30.0
Note: See TracTickets for help on using tickets.