christopherpow / nesicide

Integrated Development Environment for the 8-bit Nintendo Entertainment System
315 stars 38 forks source link

Qt build errors on Mac OS #22

Closed kotutuloro closed 6 years ago

kotutuloro commented 6 years ago

When I try to build on Mac OS 10.12 Sierra, I'm getting a series of errors.

The first error I get is "Qt requires C++11 support" (I'm using Qt 5.9.1). I get a series of other Qt related errors about undeclared identifiers before hitting "fatal error: too many errors emitted".

I'm not at all versed in Qt or C++, so I'm having trouble figuring out if this is an issue on my end or not.

christopherpow commented 6 years ago

Are you at HEAD on master? Have you followed the README and downloaded/exploded the dependencies tarball? I had this issue about a month ago and I'm not recalling what the solution was, at the moment. I seem to remember it having something to do with the version of Boost. I've updated to Boost 1.64.0 in the latest dependencies package. Also, are you building using the build.sh script in build/? Or are you loading the subdirs project into Qt Creator and building in there?

kotutuloro commented 6 years ago

At HEAD on master I exploded the tarball with the dependencies and the version of Boost I have is 1.64.0. And I'm building in the build/ directory inside of nesicide.

I'm noticing that before the C++11 error, I'm also a ton of errors about unsigned ints. For example:

common/cnessystempalette.cpp:58:4: error: constant expression evaluates to -1381126912 which cannot be narrowed to type 'uint32_t' (aka 'unsigned int') [-Wc++11-narrowing] RGB_VALUE ( 173, 173, 173 ), ^~~~~~~

christopherpow commented 6 years ago

I am seeing this problem now. I recently upgraded from Qt 5.5 to Qt 5.9.1. I am looking into the solution. Also, I started getting the int problems too. I will push a fix for that. As for the c++11 thing, not sure on that yet.

On Tue, Sep 5, 2017 at 2:04 PM Kikelomo Otutuloro notifications@github.com wrote:

At HEAD on master I exploded the tarball with the dependencies and the version of Boost I have is 1.64.0. And I'm building in the build/ directory inside of nesicide.

I'm noticing that before the C++11 error, I'm also a ton of errors about unsigned ints. For example:

common/cnessystempalette.cpp:58:4: error: constant expression evaluates to -1381126912 which cannot be narrowed to type 'uint32_t' (aka 'unsigned int') [-Wc++11-narrowing] RGB_VALUE ( 173, 173, 173 ), ^~~~~~~

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/christopherpow/nesicide/issues/22#issuecomment-327272309, or mute the thread https://github.com/notifications/unsubscribe-auth/ABkI8M07l8TXocJTqOv6fh2e-c05GMx2ks5sfZssgaJpZM4PCPzG .

kotutuloro commented 6 years ago

(Sorry, I realized I haven't been getting notifications on this issue) I've pulled all of the recent changes, and I'm no longer getting this same error. But when I try to build now, 3 of the builds (ide, famitracker, and famiplayer) error out with:

In file included from cqtmfc_famitracker.cpp:1: ../../common/cqtmfc.h:2302:10: fatal error: 'windows.h' file not found

include

NES emulator builds and deploys correctly as far as I can tell. (Also, should I make a separate issue for this?)

christopherpow commented 6 years ago

I noticed that. I have a fix. I'm not on my computer now but will push when I can later tonight.

On Fri, Oct 6, 2017 at 5:26 PM Kikelomo Otutuloro notifications@github.com wrote:

(Sorry, I realized I haven't been getting notifications on this issue) I've pulled all of the recent changes, and I'm no longer getting this same error. But when I try to build now, 3 of the builds (ide, famitracker, and famiplayer) error out with:

In file included from cqtmfc_famitracker.cpp:1: ../../common/cqtmfc.h:2302:10: fatal error: 'windows.h' file not found

include

NES emulator builds and deploys correctly as far as I can tell.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/christopherpow/nesicide/issues/22#issuecomment-334884770, or mute the thread https://github.com/notifications/unsubscribe-auth/ABkI8Jmah5DZq16sonsfcQS2eIqZ2sV2ks5spqkjgaJpZM4PCPzG .

christopherpow commented 6 years ago

Please try now it should work.

kotutuloro commented 6 years ago

Not sure if the changes were pushed?

christopherpow commented 6 years ago

Ah. They were. But I need to upload a revised deps package. I should be able to do that this evening. Will let you know.

On Sun, Oct 8, 2017 at 11:23 AM Kikelomo Otutuloro notifications@github.com wrote:

Not sure if the changes were pushed?

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/christopherpow/nesicide/issues/22#issuecomment-335018041, or mute the thread https://github.com/notifications/unsubscribe-auth/ABkI8E4WYDG4uUsrmDjcEpmzkxBcBwMAks5sqPcTgaJpZM4PCPzG .

christopherpow commented 6 years ago

I have uploaded a new nesicide-deps.tar.bz2. Please download it, explode it, and try to build now. Thanks.

On Sun, Oct 8, 2017 at 11:23 AM, Kikelomo Otutuloro < notifications@github.com> wrote:

Not sure if the changes were pushed?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/christopherpow/nesicide/issues/22#issuecomment-335018041, or mute the thread https://github.com/notifications/unsubscribe-auth/ABkI8E4WYDG4uUsrmDjcEpmzkxBcBwMAks5sqPcTgaJpZM4PCPzG .

christopherpow commented 6 years ago

Actually, hang on. I need to do it once more.

On Sun, Oct 8, 2017 at 11:23 AM, Kikelomo Otutuloro < notifications@github.com> wrote:

Not sure if the changes were pushed?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/christopherpow/nesicide/issues/22#issuecomment-335018041, or mute the thread https://github.com/notifications/unsubscribe-auth/ABkI8E4WYDG4uUsrmDjcEpmzkxBcBwMAks5sqPcTgaJpZM4PCPzG .

kotutuloro commented 6 years ago

(Not sure if you were done with the deps but, the rtmidi dependency in the new tarball comes with a premade Makefile with the qmake path harcoded in, so it errored out. I went into /deps/rtmidi and removed the Makefile which solved it.)

I came across this error when building IDE, FamiTracker, and FamiPlayer:

Undefined symbols for architecture x86_64: "RtMidiIn::RtMidiIn(RtMidi::Api, std::1::basic_string<char, std::__1::char_traits, std::1::allocator >, unsigned int)", referenced from: CMIDI::CMIDI() in MIDI.o "MidiInApi::setCallback(void ()(double, std::1::vector<unsigned char, std::1::allocator >, void), void)", referenced from: CMIDI::OpenDevices() in MIDI.o CMIDI::ToggleInput() in MIDI.o "MidiInApi::cancelCallback()", referenced from: CMIDI::~CMIDI() in MIDI.o CMIDI::CloseDevices() in MIDI.o CMIDI::Shutdown() in MIDI.o CMIDI::SetInputDevice(int, bool) in MIDI.o CMIDI::SetOutputDevice(int) in MIDI.o CMIDI::ToggleInput() in MIDI.o "RtMidiOut::RtMidiOut(RtMidi::Api, std::1::basic_string<char, std::__1::char_traits, std::1::allocator >)", referenced from: CMIDI::CMIDI() in MIDI.o ld: symbol(s) not found for architecture x86_64 clang: error: linker command failed with exit code 1 (use -v to see invocation) make[1]: [release/libfamitracker.1.0.0.dylib] Error 1 make: [sub-------libs-famitracker-famitracker-lib-pro-make_first] Error 2

Adding a screenshot of everything it runs for FamiPlayer as an example in case that helps? screen shot 2017-10-10 at 2 58 43 pm

$ clang++ -v

Apple LLVM version 8.1.0 (clang-802.0.42) Target: x86_64-apple-darwin16.4.0 Thread model: posix InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

christopherpow commented 6 years ago

Grab everything fresh and try again. I am able to build in OSX.

christopherpow commented 6 years ago

Have you made any progress on this? I am able to build.

TroyLong commented 6 years ago

Yes. Thank you

On Wed, Oct 25, 2017 at 9:34 PM, Christopher Pow notifications@github.com wrote:

Have you made any progress on this? I am able to build.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/christopherpow/nesicide/issues/22#issuecomment-339532469, or mute the thread https://github.com/notifications/unsubscribe-auth/ANrPIlOLkxhsbjGjP4_w6Zz9MlKUKcHcks5sv-_SgaJpZM4PCPzG .

kotutuloro commented 6 years ago

Started from scratch and everything builds beautifully!

Thank you so, so much for all of your work by the way! I really appreciate it.

christopherpow commented 6 years ago

Thanks! Enjoy!