Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#6053 closed defect (invalid)

MythMusic compile problem with --enable-aac

Reported by: otto at kolsi dot fi Owned by: stuartm
Priority: minor Milestone: 0.21.1
Component: mythmusic Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

If mythplugins is configured with --enable-aac, following compile error occurs. System is Fedora 9.

g++ -c -pipe -march=pentium4 -Wall -Wno-switch -Wpointer-arith -Wredundant-decls -Wno-non-virtual-dtor -funit-at-a-time -D__STDC_CONSTANT_MACROS -fomit-frame-pointer -O3 -fomit-frame-pointer -I/usr/include/SDL -D_GNU_SOURCE=1 -D_REENTRANT -D_REENTRANT -Wall -W -DPIC -fPIC -D_GNU_SOURCE -DPREFIX="/usr/local" -DMMX -Di386 -D_FILE_OFFSET_BITS=64 -DQT_NO_DEBUG -DQT_PLUGIN -DQT_QT3SUPPORT_LIB -DQT3_SUPPORT -DQT_SQL_LIB -DQT_XML_LIB -DQT_OPENGL_LIB -DQT_GUI_LIB -DQT_CORE_LIB -I/usr/lib/qt4/mkspecs/linux-g++ -I. -I/usr/include/QtCore -I/usr/include/QtCore -I/usr/include/QtGui -I/usr/include/QtGui -I/usr/include/QtOpenGL -I/usr/include/QtOpenGL -I/usr/include/QtXml -I/usr/include/QtXml -I/usr/include/QtSql -I/usr/include/QtSql -I/usr/include/Qt3Support -I/usr/include/Qt3Support -I/usr/include -I/usr/local/include -I/usr/local/include -I/usr/include -I/usr/local/include/mythtv -I/usr/include/libvisual-0.4 -I/usr/include/taglib -I/usr/include/cdda -I/usr/X11R6/include -I. -I. -o aacdecoder.o aacdecoder.cpp
aacdecoder.cpp: In member function ‘bool aacDecoder::initializeMP4()’:
aacdecoder.cpp:317: error: invalid conversion from ‘uint32_t*’ to ‘long unsigned int*’
aacdecoder.cpp:317: error:   initializing argument 4 of ‘char NeAACDecInit2(void*, unsigned char*, long unsigned int, long unsigned int*, unsigned char*)’
make[2]: *** [aacdecoder.o] Error 1
make[2]: Leaving directory `/mnt/myth/trunk/mythplugins/mythmusic/mythmusic'
make[1]: *** [sub-mythmusic-make_default] Error 2
make[1]: Leaving directory `/mnt/myth/trunk/mythplugins/mythmusic'
make: *** [sub-mythmusic-make_default] Error 2

Without --enable-aac option mythplugins compile without errors.

Attachments (2)

aacdecoder.patch (871 bytes) - added by Paul Mason <latepaul@…> 11 years ago.
aacdecoder.2.patch (367 bytes) - added by Paul Mason <latepaul@…> 11 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 11 years ago by stuartm

Milestone: unknown0.21.1
Owner: changed from Isaac Richards to stuartm
Status: newassigned

I'll be removing the aac decoder from trunk very soon, but if someone submits a patch for -fixes we'll fix it there.

comment:2 Changed 11 years ago by stuartm

Status: assignedinfoneeded

comment:3 in reply to:  1 Changed 11 years ago by otto at kolsi dot fi

Replying to stuartm:

I'll be removing the aac decoder from trunk very soon, but if someone submits a patch for -fixes we'll fix it there.

--enable-aac works fine with -fixes in the same machine, so maybe this ticket can be closed then?

Changed 11 years ago by Paul Mason <latepaul@…>

Attachment: aacdecoder.patch added

Changed 11 years ago by Paul Mason <latepaul@…>

Attachment: aacdecoder.2.patch added

comment:4 Changed 11 years ago by Paul Mason <latepaul@…>

second patch replaces the first one which had a diff due to a typo.

comment:5 Changed 11 years ago by stuartm

Resolution: invalid
Status: infoneededclosed

comment:6 Changed 11 years ago by Britney Fransen <britney.fransen@…>

Using .21-fixes and --enable-aac I just saw this same error. Applying the second patch fixed the error and it then compiled without error.

Note: See TracTickets for help on using tickets.