Bug 62891 - noatun doesn't build with gcc-3.3.1
Summary: noatun doesn't build with gcc-3.3.1
Status: RESOLVED FIXED
Alias: None
Product: noatun
Classification: Miscellaneous
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Solaris
: NOR normal
Target Milestone: ---
Assignee: Multimedia Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-08-18 16:40 UTC by alancio
Modified: 2003-08-29 17:33 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch fixing the problem from Red Hat (rawhide .src.rpm) (982 bytes, patch)
2003-08-18 17:32 UTC, Ewald Snel
Details

Note You need to log in before you can comment on or make changes to this bug.
Description alancio 2003-08-18 16:40:09 UTC
Version:            (using KDE KDE 3.1.3)
Installed from:    Compiled From Sources
Compiler:          gcc-3.3.1 
OS:          Solaris

I am trying to build kdemultimedia with gcc-3.3.1, but I get the following error:

kdemultimedia-3.1.3/noatun/modules/winskin/vis/visQueue.cpp:20: error: ISO C++ forbids variable-size array

Is this a plugin or is it integral part of noatun?
If it is a plugin, how can I disable it so I can go on compiling the other things? make -k? setenv DO_NOT_COMPILE?


This is the complete output:

Making all in vismake[5]: Entering directory `/cesma/alan/kde-3.1.3/build/kdemultimedia-3.1.3/noatun/modules/winskin/vis'if /bin/bash ../../../../libtool --silent --mode=compile --tag=CXX g++ -DHAVE_CONFIG_H -I. -I../../../../../../source/kdemultimedia-3.1.3/noatun/modules/winskin/vis -I../../../.. -I/usr/local/kde-3.1.3/include/arts -I/usr/local/kde-3.1.3/include -I/usr/local/qt-3.1.2/include    -DQT_THREAD_SUPPORT -O3 -fexpensive-optimizations -mtune=ultrasparc -D_REENTRANT -D_POSIX_PTHREAD_SEMANTICS -DUSE_SOLARIS -DSVR4 -I/usr/demo/SOUND/include  -Wnon-virtual-dtor -Wno-long-long -Wundef -Wall -pedantic -W -Wpointer-arith -Wwrite-strings -DNDEBUG -DNO_DEBUG -O2 -O3 -fexpensive-optimizations -mtune=ultrasparc -fno-exceptions -fno-check-new -DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST  -MT visQueue.lo -MD -MP -MF ".deps/visQueue.Tpo" \  -c -o visQueue.lo `test -f '../../../../../../source/kdemultimedia-3.1.3/noatun/modules/winskin/vis/visQueue.cpp' || echo '../../../../../../source/kdemultimedia-3.1.3/noatun/modules/winskin/vis/'`../../../../../../source/kdemultimedia-3.1.3/noatun/modules/winskin/vis/visQueue.cpp; \
then mv ".deps/visQueue.Tpo" ".deps/visQueue.Plo"; \
else rm -f ".deps/visQueue.Tpo"; exit 1; \
fi../../../../../../source/kdemultimedia-3.1.3/noatun/modules/winskin/vis/visQueue.cpp: In
   constructor `VISQueue::VISQueue(int)':../../../../../../source/kdemultimedia-3.1.3/noatun/modules/winskin/vis/visQueue.cpp:20: error: ISO
   C++ forbids variable-size array
make[5]: *** [visQueue.lo] Error 1make[5]: Leaving directory `/cesma/alan/kde-3.1.3/build/kdemultimedia-3.1.3/noatun/modules/winskin/vis'
make[4]: *** [all-recursive] Error 1make[4]: Leaving directory `/cesma/alan/kde-3.1.3/build/kdemultimedia-3.1.3/noatun/modules/winskin'
make[3]: *** [all-recursive] Error 1make[3]: Leaving directory `/cesma/alan/kde-3.1.3/build/kdemultimedia-3.1.3/noatun/modules'
make[2]: *** [all-recursive] Error 1make[2]: Leaving directory `/cesma/alan/kde-3.1.3/build/kdemultimedia-3.1.3/noatun'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/cesma/alan/kde-3.1.3/build/kdemultimedia-3.1.3'
make: *** [all] Error 2
Comment 1 Ewald Snel 2003-08-18 17:32:36 UTC
Created attachment 2263 [details]
Patch fixing the problem from Red Hat (rawhide .src.rpm)

I have found this patch in the KDE Multimedia source RPM from Red Hat rawhide.
It seems to fix the problem with Red Hat's version of gcc 3.3.1.
Comment 2 Neil Stevens 2003-08-19 00:18:17 UTC
winskin is a UI plugin, so yes it can be skipped if you have to. 
 
If you really want that plugin, though, try compiling without -pedantic. 
Comment 3 alancio 2003-08-19 01:52:30 UTC
Thanks, it compiled without -pedantic. 
 
This is a bug because I didn't have -pedantic in CFLAGS, so if the Makefile 
autoconf/automake generates by itself has this flag then it should work. 
Comment 4 Charles Samuels 2003-08-29 17:33:07 UTC
looks like your patch was applied