Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1614 closed defect (fixed)

Build failure in SoundManager.h

Reported by: DroppingIn Owned by: stwf
Priority: Should Have Milestone: Alpha 11
Component: Core engine Keywords:
Cc: Patch:

Description

Compilation has failed since the new sound system was committed/merged. Compilation fails even when --without-sound is passed.

In file included from ../../../source/soundmanager/SoundManager.cpp:22:0: ../../../source/soundmanager/data/SoundData.h:52:2: error: ‘CStrW’ does not name a type ../../../source/soundmanager/data/SoundData.h:60:2: error: ‘CStrW’ does not name a type make[1]: * [obj/engine_Release/SoundManager.o] Error 1

Full log attached.

Attachments (1)

build.log (507.3 KB ) - added by DroppingIn 12 years ago.
Build log

Download all attachments as: .zip

Change History (7)

by DroppingIn, 12 years ago

Attachment: build.log added

Build log

comment:1 by historic_bruno, 12 years ago

There's no --without-sound option :) Did you mean --without-audio? In any case it won't make a difference, that file still has to compile and the new sound system doesn't fully respect that option yet.

Which version of GCC are you using?

in reply to:  1 comment:2 by DroppingIn, 12 years ago

Replying to historic_bruno:

There's no --without-sound option :) Did you mean --without-audio? In any case it won't make a difference, that file still has to compile and the new sound system doesn't fully respect that option yet.

That's what I meant!

Which version of GCC are you using?

4.7.1.

comment:3 by stwf, 12 years ago

Owner: set to stwf
Status: newassigned

comment:4 by ben, 12 years ago

Resolution: fixed
Status: assignedclosed

In 12566:

Sound system patch by stwf:
Fixes sound system to work with archives (use VFS files instead of POSIX), fixes #1604.
Improves error handling and logging, refs #1594.
Allows sound to be disabled with -nosound/-quickstart runtime options or --without-audio build option, fixes #1609, #1614.
Experimentally increases default buffer size to help prevent sound stoppages.

comment:5 by historic_bruno, 12 years ago

Component: Build & PackagesCore engine
Milestone: BacklogAlpha 11

The problematic code has been removed as of r12566, you should now be able to build with --without-audio and it will work :) If not, reopen the ticket with the relevant info.

comment:6 by ben, 12 years ago

In 12577:

Adds some missing includes, refs #1614

Note: See TracTickets for help on using tickets.