-
Notifications
You must be signed in to change notification settings - Fork 23
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
error: garbage collection is no longer supported #1
Comments
@petasis can I ask, what OS X versions does tkdnd aim to support? The latest binary on SourceForge is 10.8, so I presume that that's your development environment. So should it be at least your development environment, because you quite rightly want to be able to test and compile the software, and then potentially up to the latest OS X (currently 10.10)? |
Dear Andrew, In order to compile with the newer garbage collector, please edit CMakeLists.txt, lines 56-57 (comment them, both -fobjc-gc, -fno-objc-arc), and uncomment line 58 (-fobjc-arc). Best regards, |
Hello, I see radarhere added compatibility for later OSX last July. I just tried to compile on 10.11 using his forked "arc" branch and master branch as well and get same error as last year: checking for correct TEA configuration... ok (TEA 3.9) |
Are you saying that the output you've included is from my
is simple enough that I have trouble imagining a bug in it's purpose of changing the flag to |
yup... on your arc branch: megrimm-mbp:tkdnd megrimm$ make
On Wed, Mar 2, 2016 at 2:07 AM, Andrew Murray notifications@github.com
m.e.grimm, m.f.a, ed.m. |
Does the error also affects the cmake build? |
yes... similar error: megrimm-mbp:build megrimm$ cmake .. On Wed, Mar 2, 2016 at 2:15 PM, petasis notifications@github.com wrote:
m.e.grimm, m.f.a, ed.m. |
Hi. Would you be able to show me the output of these commands? You can copy paste them into your Terminal, or put them into a file and execute it.
Also, are you running |
megrimm-mbp:tkdnd megrimm$ ./test.sh ... and YES ./configure was run first in arc branch. hmmmmm..... thanks! On Thu, Mar 3, 2016 at 2:06 AM, Andrew Murray notifications@github.com
m.e.grimm, m.f.a, ed.m. |
Okay then. I've created a new branch, |
ok... there is the full output: megrimm-mbp:tkdnd megrimm$ git pull
checking for Tcl public headers... /Library/Frameworks/Tcl.framework/Headers define panic Tcl_Panic
/Library/Frameworks/Tcl.framework/Headers/tclDecls.h:2503:14: note: define panic Tcl_Panic
/Library/Frameworks/Tcl.framework/Headers/tclDecls.h:2503:2: note: expanded define panic Tcl_Panic
/Library/Frameworks/Tcl.framework/Headers/tclDecls.h:2503:3: note: expanded On Fri, Mar 4, 2016 at 5:55 AM, Andrew Murray notifications@github.com
m.e.grimm, m.f.a, ed.m. |
Okay. Now it looks like the changes from my branch are taking effect. I'm just going to focus on fixing your individual problem now, if that's alright. The latest errors you're experiencing are ones that I have found myself. My solution was to move the header files into the same directory. I have now made this change on the I've used header files from Tk/Tcl 8.6.1. It's possible you're using 8.5, and so that may need be to changed. However, I do feel that we are now close! |
This has been fixed. |
Original issue reported on code.google.com by
megr...@gmail.com
on 19 Nov 2014 at 8:12The text was updated successfully, but these errors were encountered: