Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stack audio errors when building TSC on MSYS2 on Windows #674

Closed
xet7 opened this issue Dec 17, 2019 · 2 comments
Closed

Stack audio errors when building TSC on MSYS2 on Windows #674

xet7 opened this issue Dec 17, 2019 · 2 comments
Labels
Bug
Milestone

Comments

@xet7
Copy link
Member

@xet7 xet7 commented Dec 17, 2019

2019-12-17 both branches devel and release-2.1.0 have this error:

[ 16%] Building CXX object CMakeFiles/tsc.dir/src/audio/sound_manager.cpp.obj
[ 16%] Building CXX object CMakeFiles/tsc.dir/src/audio/random_sound.cpp.obj
[ 16%] Building CXX object CMakeFiles/tsc.dir/src/campaign/campaign_loader.cpp.obj
In file included from C:/msys64/home/vagrant/TSC/tsc/src/audio/random_sound.cpp:20:
C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.hpp:203:14: error: 'stack' in namespace 'std' does not name a template type
  203 |         std::stack<NextMusicInfo> m_next_music;
      |              ^~~~~
In file included from C:/msys64/home/vagrant/TSC/tsc/src/audio/random_sound.cpp:20:
C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.hpp:27:1: note: 'std::stack' is defined in header '<stack>'; did you forget to '#include <stack>'?
   26 | #include <queue>
  +++ |+#include <stack>
   27 | #ifdef _WIN32
In file included from C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.cpp:17:
C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.hpp:203:14: error: 'stack' in namespace 'std' does not name a template type
  203 |         std::stack<NextMusicInfo> m_next_music;
      |              ^~~~~
In file included from C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.cpp:17:
C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.hpp:27:1: note: 'std::stack' is defined in header '<stack>'; did you forget to '#include <stack>'?
   26 | #include <queue>
  +++ |+#include <stack>
   27 | #ifdef _WIN32
C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.cpp: In member function 'bool TSC::cAudio::Play_Music(boost::filesystem::path, bool, bool, unsigned int)':
C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.cpp:351:9: error: 'm_next_music' was not declared in this scope
  351 |         m_next_music.emplace(filename, loops, fadein_ms);
      |         ^~~~~~~~~~~~
C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.cpp: In member function 'void TSC::cAudio::Update()':
C:/msys64/home/vagrant/TSC/tsc/src/audio/audio.cpp:570:33: error: 'm_next_music' was not declared in this scope
  570 |     if (!Is_Music_Playing() && !m_next_music.empty()) {
      |                                 ^~~~~~~~~~~~
make[2]: *** [CMakeFiles/tsc.dir/build.make:76: CMakeFiles/tsc.dir/src/audio/random_sound.cpp.obj] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [CMakeFiles/tsc.dir/build.make:63: CMakeFiles/tsc.dir/src/audio/audio.cpp.obj] Error 1
make[1]: *** [CMakeFiles/Makefile2:236: CMakeFiles/tsc.dir/all] Error 2
make: *** [Makefile:152: all] Error 2
@xet7 xet7 added this to the Version 2.1.0 milestone Dec 17, 2019
Quintus added a commit that referenced this issue Dec 17, 2019
@Quintus

This comment has been minimized.

Copy link
Member

@Quintus Quintus commented Dec 17, 2019

I've added a fix. Please retry compiling the release-2.1.0 branch.

@Quintus Quintus removed the Build system label Dec 17, 2019
@xet7

This comment has been minimized.

Copy link
Member Author

@xet7 xet7 commented Dec 17, 2019

Thanks! Now it works!

@xet7 xet7 closed this Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.