Opened 11 years ago

Closed 10 years ago

#5987 closed defect (fixed)

MythMusic fails to build on windows

Reported by: Jonathan Martens <jonathan@…> Owned by: Nigel
Priority: minor Milestone: unknown
Component: Ports - Windows Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no

Description (last modified by Nigel)

It seems that MythMusic fails to build on Windows for (at least) two reasons, one is Goom failing to build as M_PI seems undeclared (patch will be attached).

The second is that it complains (after making Goom compile OK) that QHttp does not exist:

g++ -c -march=k8 -g -Wall -Wno-switch -Wpointer-arith -Wredundant-decls -Wno-non-virtual-dtor -funit-at-a-time -D__STDC_CONSTANT_MACROS -g -frtti -fexceptions -
mthreads -Wall -DUNICODE -DQT_LARGEFILE_SUPPORT -DUSING_MINGW -D_GNU_SOURCE -DPREFIX="U:/mythtv/build/" -DMMX -Di386 -DUSING_DIRECTX -D_FILE_OFFSET_BITS=64 -DQT_THREAD_SUPPORT -DQT_DLL -DQT_PLUGIN -DQT_QT3SUPPORT_LIB -DQT3_SUPPORT -DQT_SQL_
LIB -DQT_XML_LIB -DQT_OPENGL_LIB -DQT_GUI_LIB -DQT_CORE_LIB -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtCore' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtCore' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtGui' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtGui' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtOpenGL' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtOpenGL' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtXml' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtXml' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtSql' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/QtSql' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/Qt3Support' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include/Qt3Support ' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/include' -I'../../../build/include' -I'../../../build/include' -I'../../../build/include/mythtv' -I'/usr/include/taglib' -I'/usr/include/cdda' -I'u:/MSys/1.0/qt-win-opensource-src-4.4.3/
include/ActiveQt' -I'.' -I'.' -I'../../../../MSys/1.0/qt-win-opensource-src-4.4.3/mkspecs/win32-g++' -o cddecoder-windows.o cddecoder-windows.cpp
In file included from cddecoder-windows.cpp:19:
../../../build/include/mythtv/httpcomms.h:9:17: QHttp: No such file or directory

In file included from cddecoder-windows.cpp:19:
../../../build/include/mythtv/httpcomms.h:19: error: `QHttpRequestHeader' has not been declared
../../../build/include/mythtv/httpcomms.h:19: error: ISO C++ forbids declaration of `header' with no type
../../../build/include/mythtv/httpcomms.h:68: error: `QHttpRequestHeader' has not been declared
../../../build/include/mythtv/httpcomms.h:75: error: ISO C++ forbids declaration of `pAddlHdr' with no type
../../../build/include/mythtv/httpcomms.h:78: error: `QHttpRequestHeader' has not been declared
../../../build/include/mythtv/httpcomms.h:78: error: ISO C++ forbids declaration of `header' with no type
../../../build/include/mythtv/httpcomms.h:102: error: `QHttpRequestHeader' has not been declared
../../../build/include/mythtv/httpcomms.h:102: error: ISO C++ forbids declaration of `request' with no type
../../../build/include/mythtv/httpcomms.h:109: error: expected `,' or `...' before '&' token
../../../build/include/mythtv/httpcomms.h:109: error: ISO C++ forbids declaration of `QHttpResponseHeader' with no type
../../../build/include/mythtv/httpcomms.h:117: error: ISO C++ forbids declaration of `QHttp' with no type
../../../build/include/mythtv/httpcomms.h:117: error: expected `;' before '*' token
../../../build/include/mythtv/httpcomms.h:129: error: `QHttpRequestHeader' does not name a type
make: *** [cddecoder-windows.o] Error 1

Attachments (2)

MythTV-goom.patch (1.2 KB) - added by Jonathan Martens <jonathan@…> 11 years ago.
MythTV-goom.2.patch (1.3 KB) - added by Jonathan Martens <jonathan@…> 11 years ago.
expires earlier patch (but it seems I have no delete privileges) as this is from the top of the source tree

Download all attachments as: .zip

Change History (7)

Changed 11 years ago by Jonathan Martens <jonathan@…>

Attachment: MythTV-goom.patch added

Changed 11 years ago by Jonathan Martens <jonathan@…>

Attachment: MythTV-goom.2.patch added

expires earlier patch (but it seems I have no delete privileges) as this is from the top of the source tree

comment:1 Changed 11 years ago by Jonathan Martens <jonathan@…>

Forgot to mention that it also fails on a third reason as usleep seems undeclared in playbackbox.cpp, commenting it out (dirty hack off course) stops compile from failing there, this might need a compat.h file or even better proper includes as I believe MinGW provides this function as well (it seems to do so for other mythtv files as I need to patch it out in mythtv/libs/libmythdb/compat.h to make MythTV core build as described in #5980 (http://cvs.mythtv.org/trac/attachment/ticket/5980/MythTV-compat.h.patch).

comment:2 Changed 11 years ago by danielk

Owner: changed from danielk to Nigel
Status: newassigned

comment:3 Changed 10 years ago by Nigel

Description: modified (diff)

comment:4 Changed 10 years ago by Nigel

Component: windowsPorts - Windows

comment:5 Changed 10 years ago by Nigel

Resolution: fixed
Status: assignedclosed

(In [23277]) Fix missing M_PI on Windows. Closes #5987.

Note: See TracTickets for help on using tickets.