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

GCC9 Follow-up #2102

Merged
merged 2 commits into from
May 14, 2019
Merged

GCC9 Follow-up #2102

merged 2 commits into from
May 14, 2019

Conversation

uklotzde
Copy link
Contributor

@uklotzde uklotzde commented May 5, 2019

Not sure if Clang builds on macOS are affected by aborting with a Python error (as reported in #2101).

  • CCFLAGS -> CXXFLAGS to avoid warnings when compiling C files
  • Exclude from Clang builds that simply report an old GCC version (4.2.1 for 8.0.0)

@esbrandt
Copy link
Contributor

esbrandt commented May 5, 2019

With latest 3172494 , the build aborts

$  scons
scons: Reading SConscript files ...
INFO:root:Target Platform: osx
INFO:root:Target Machine: x86_64
INFO:root:Build: debug
INFO:root:Toolchain: gnu
INFO:root:Crosscompile: NO
INFO:root:Qt path: /usr/local/Cellar/qt/5.12.3/
Automatically detecting Mac OS X SDK.
XCode developer directory:/Applications/Xcode.app/Contents/Developer
Found OS X SDK:macosx10.14
Automatically selected OS X SDK:/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk
SConscript.env cwd: /Users/esbrandt/git/mixxxdj-mixxx/osx64_build
Checking whether the C++ compiler works... no
Building with CXX: clang++
A compiler with C++11 support is required.

After applying the patch from this PR, the error persist.

$ gcc --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/c++/4.2.1
Apple LLVM version 10.0.0 (clang-1000.11.45.5)
Target: x86_64-apple-darwin18.2.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

@uklotzde uklotzde added this to the 2.3.0 milestone May 6, 2019
@uklotzde
Copy link
Contributor Author

Merging this now, because it fixes a nasty warning when compiling C files. Clang was not affected.

@uklotzde uklotzde merged commit eea6e34 into mixxxdj:master May 14, 2019
@uklotzde uklotzde deleted the gcc9 branch June 1, 2019 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants