-
Notifications
You must be signed in to change notification settings - Fork 135
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
Compile errors on Mac with latest (apparent typo introduced by #33 in revision 24) #37
Comments
Comment 1. originally posted by magreenblatt on 2014-02-05T19:21:56.000Z: Thanks for the report, but your patch is incorrect. Compile error fixed with correct changes in revision 26. |
|
Original comment by Anonymous. Comment 2. originally posted by christophe.cornu on 2014-02-05T20:42:32.000Z: Ack. Are you able to build a release version of the native libraries? I'm able to build the debug version only. For the release configuration, the post build step "Strip if needed" fails for me on Build target jcef_helper. A simple yes or no will be great :-) Build target jcef_helper ProcessInfoPlistFile "./xcodebuild/Release/jcef Helper.app/Contents/Info.plist" ./third_party/cef/macosx64/cefclient/mac/helper-Info.plist CompileC ./xcodebuild/jcef.build/Release/jcef_helper.build/Objects-normal/x86_64/jcef_helper.o native/jcef_helper.cpp normal x86_64 c++ com.apple.compilers.llvmgcc42 Ld "./xcodebuild/Release/jcef Helper.app/Contents/MacOS/jcef Helper" normal x86_64 PhaseScriptExecution "Postbuild \"Fix Framework Link\"" ./xcodebuild/jcef.build/Release/jcef_helper.build/Script-213024CF2B2594275141F091.sh PhaseScriptExecution "Postbuild \"Change Mach-O Flags\"" ./xcodebuild/jcef.build/Release/jcef_helper.build/Script-A543731CC89BD4BC25109641.sh PhaseScriptExecution "Postbuild \"Strip If Needed\"" ./xcodebuild/jcef.build/Release/jcef_helper.build/Script-96A927E9C987A9DE4CFD9A33.sh Traceback (most recent call last): |
Original report by Anonymous.
Original issue 37 created by christophe.cornu on 2014-02-03T22:09:51.000Z:
What steps will reproduce the problem?
What is the expected output? What do you see instead?
CefContext.cpp doesn't build
What version of the product are you using? On what operating system?
Latest - revision 25
Please provide any additional information below.
It's a typo with regard to the placement of the if defined(OS_MACOSX). There's a mismatch between { and } as a result. See patch though it's probably just as easy to fix it in the editor by hand.
BTW can you confirm you are using LLVM GCC 4.2 and not Apple's one?
The text was updated successfully, but these errors were encountered: