[mythtv-commits] Ticket #7088: mythmusic's adding of the cdda includepath the qt project is broken

[MythTV]

#7088: mythmusic's adding of the cdda includepath the qt project is broken
---------------------------------+------------------------------------------
 Reporter:  pebender at san.rr.com  |        Owner:  janne  
     Type:  defect               |       Status:  closed 
 Priority:  minor                |    Milestone:  0.22   
Component:  Plugin - MythMusic   |      Version:  unknown
 Severity:  low                  |   Resolution:  fixed  
  Mlocked:  0                    |  
---------------------------------+------------------------------------------

Comment(by pebender at san.rr.com):

 As I mentioned in the bug report, exporting SYSROOT breaks many other
 things, which I I was suggested the fix.

 The problem is how SYSROOT and PREFIX are used.

 mythtv never prepends SYSROOT to PREFIX in of its *.pro files or
 Makefiles. However, it does prepend $sysroot to non $prefix path in the
 configure script. Therefore, for mythtv to cross-compile correctly,
 --sysroot must be set and --prefix must be set and prefixed by the value
 of --sysroot.

 However, mythplugins pepends SYSROOT to PREFIX in some of its *.pro and
 Makefiles but not in others. With SYSROOT exported, any *.pro or Makefile
 that prepends SYSROOT breaks. In addition, mythplugins and the themes
 configure scripts prepend $sysroot to $prefix.

 Because of these discrepancies, it is not clear to me the intended
 relationship between sysroot and prefix. When cross compiling, if prefix
 is intended to be a subdirectory of sysroot, then SYSROOT needs to be
 prepended to everywhere PREFIX is used in the mythtv and mythplugins *.pro
 and Makefiles and $sysroot needs to be prepended to $prefix in the
 configure scripts. When cross compiling, if prefix is intended to be
 independent of sysroot, then SYSROOT should not prepend PREFIX in any of
 mythplugins *.pro or Makefiles and $sysroot should not be prepended to
 $prefix in any of the configure scripts.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/7088#comment:4>
MythTV <http://www.mythtv.org/>
MythTV