New Ticket     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #20799 (new defect)

Opened 2 years ago

Last modified 2 months ago

tk +quartz doesn't build 64-bit

Reported by: eallik@… Owned by: mww@…
Priority: Normal Milestone:
Component: ports Version: 1.8.99
Keywords: LP64 Cc: simon@…, cedayiv@…, kiwi.2008@…, jeffrey@…, tjb@…, vitaly@…, jpenney@…, rmsfisher@…, hgbrauen@…, markus@…, macports@…, itrekkie@…, palimm@…, anders.hasselqvist@…, yannick.cholette@…, adhocrocker@…, speed.rob@…, royliu@…, michael@…, ddollar@…, jarias01@…, gthyni@…, thimo@…, zeph0x@…, simpoir@…, verena.wolf@…, Damien@…, bgrupe@…, yacoob@…, tsunanet@…, jsg8pitt@…, eykosioux@…, jeremy.bethmont@…, zimmermann@…, yves@…, tim.stoop@…, stephen@…, mdippery@…, macports.org@…, jemaltz@…, markus.doits@…, giancarlo.degani@…, elvis.dowson@…, braden@…, elelay@…, rmstonecipher@…, joe@…, tascafoo@…, calsaiwing@…, toreador@…, a.y.harano@…, marc@…, macports@…, rizwank@…, obutovich@…, asahiro.g@…, kenny.macdermid@…, michelle.lynn.gill@…, Torsten.Maehne@…
Port: tk

Description (last modified by snc@…) (diff)

Using MacPorts trunk:

make: *** [tkMacOSXButton.o] Error 1

Attachments

tk+quartz_debug_build.txt Download (222.4 KB) - added by snc@… 2 years ago.
tk.txt Download (242.7 KB) - added by andreas@… 2 years ago.
x11_tk_Portfile.das.patch Download (1.8 KB) - added by kuba@… 2 years ago.
portfile patch to use github sources [disclaimer: I'm a total ports noob] -- wish starts up OK
tk-decarbon.patch Download (1.2 KB) - added by speed.rob@… 2 years ago.
Portfile patch. Changes the source to a tk branch with no carbon dependencies hosted on github.
Portfile-tk.diff Download (1.3 KB) - added by c.freden@… 2 years ago.
Patch for tk Portfile, fetches source from  http://opensource.apple.com/tarballs/tcl/tcl-87.tar.gz
tk-858-decarbon.patch Download (1.0 KB) - added by anders.hasselqvist@… 2 years ago.
Patch updated for tk 8.5.8. Fetching from github.
tk-859-decarbon.patch Download (1.0 KB) - added by tascafoo@… 16 months ago.
Patch updated for tk 8.5.9 (you probably also want tcl patch from bug 22954)

Change History

  Changed 2 years ago by jmr@…

  • keywords snowleopard added
  • owner changed from macports-tickets@… to mww@…
  • cc eallik@… removed

Changed 2 years ago by snc@…

  Changed 2 years ago by snc@…

  • description modified (diff)

follow-up: ↓ 4   Changed 2 years ago by jmr@…

  • keywords LP64 added; snowleopard removed
  • version changed from 1.8.0 to 1.8.99
  • summary changed from tk +quartz doesn't build on SL to tk +quartz doesn't build 64-bit

Smells like QuickDraw, which means this isn't 10.6-specific.

in reply to: ↑ 3   Changed 2 years ago by speed.rob@…

Replying to jmr@…:

Smells like QuickDraw, which means this isn't 10.6-specific.

Tk still relies on QuickDraw for --with-aqua? If that's the case this isn't going to be fixed any time soon.

  Changed 2 years ago by andreas@…

Hi Gang.

What do you mean by tk relies on quickdraw (what is qd anyway?)

Im trying to build python26 which relies on tk, and i have built it successfully on 10.5 before, like 2-3 months ago. But trying to build it on 10.6 now i get something like this attached log files.

Changed 2 years ago by andreas@…

  Changed 2 years ago by andreas@…

  • cc mcalhoun@…, kentk@…, mike@…, alexguo@…, robin@…, lstoll@…, tamyrvoll@…, john+macports@…, joshua_anderson@…, huahang.liu@…, faisal.moledina@…, tomvons@…, benjaminkreeger@…, m@…, andrius.laikina@…, dizzyd@…, xmitchx@…, treaves@…, shrift@…, tannhaus@…, tharant@…, hkroger@…, sjcjonker@…, macports@…, thedoobs@…, ram@…, gerald.gutierrez@…, pkutzner+macports@…, erik.labianca@…, issaco@…, luis.beca@…, brianm@…, david@…, macports@…, albert.veli@…, julien.lusson@…, me@…, randyoo@…, macports@…, rmsfisher@…, fmaillet@…, andy@…, macports.org@…, jlaurila@…, xgutter@…, brian.cunnie@…, conradwt@…, macsforever2000@…, macosforge@…, deepu.sudhakar@…, nicos_pavlov@…, jan@… added

Cc Me!

  Changed 2 years ago by simon@…

  • cc simon@… added

Cc Me!

  Changed 2 years ago by wsiegrist@…

  • cc wsiegrist@… added

Cc Me!

  Changed 2 years ago by wsiegrist@…

  • cc wsiegrist@… removed

Cc Me!

  Changed 2 years ago by jmr@…

  • cc andreas@… added; mcalhoun@…, kentk@…, mike@…, alexguo@…, robin@…, lstoll@…, tamyrvoll@…, john+macports@…, joshua_anderson@…, huahang.liu@…, faisal.moledina@…, tomvons@…, benjaminkreeger@…, m@…, andrius.laikina@…, dizzyd@…, xmitchx@…, treaves@…, shrift@…, tannhaus@…, tharant@…, hkroger@…, sjcjonker@…, macports@…, thedoobs@…, ram@…, gerald.gutierrez@…, pkutzner+macports@…, erik.labianca@…, issaco@…, luis.beca@…, brianm@…, david@…, macports@…, albert.veli@…, julien.lusson@…, me@…, randyoo@…, macports@…, rmsfisher@…, fmaillet@…, andy@…, macports.org@…, jlaurila@…, xgutter@…, brian.cunnie@…, conradwt@…, macsforever2000@…, macosforge@…, deepu.sudhakar@…, nicos_pavlov@…, jan@… removed

Cleaning up after cc malfunction.

  Changed 2 years ago by cedayiv@…

  • cc cedayiv@… added

Cc Me!

  Changed 2 years ago by kiwi.2008@…

  • cc kiwi.2008@… added

Cc Me!

  Changed 2 years ago by andreas@…

  • cc andreas@… removed

Cc Me!

  Changed 2 years ago by mnot@…

  • cc mnot@… added

Cc Me!

  Changed 2 years ago by jeffrey@…

  • cc jeffrey@… added

Cc Me!

  Changed 2 years ago by jabronson@…

  • cc jabronson@… added

Cc Me!

follow-up: ↓ 26   Changed 2 years ago by tjb@…

  Changed 2 years ago by tjb@…

  • cc tjb@… added

Cc Me!

  Changed 2 years ago by vitaly@…

  • cc vitaly@… added

Cc Me!

  Changed 2 years ago by jpenney@…

  • cc jpenney@… added

Cc Me!

  Changed 2 years ago by rmsfisher@…

  • cc rmsfisher@… added

Cc Me!

  Changed 2 years ago by blb@…

  • cc hgbrauen@… added

Cc reporter of dup #21439.

  Changed 2 years ago by markus@…

  • cc markus@… added

Cc Me!

  Changed 2 years ago by markus@…

  • cc markus@… removed

Cc Me!

  Changed 2 years ago by markus@…

  • cc markus@… added

Cc Me!

in reply to: ↑ 17 ; follow-ups: ↓ 27 ↓ 33   Changed 2 years ago by speed.rob@…

Replying to tjb@…:

Is this any help?

I downloaded the patched tk from Github and tried it out. Worked like a charm.

Maintainer, I believe you can close this ticket by updating the portfile to 8.5.7 and adding the patch.

in reply to: ↑ 26   Changed 2 years ago by cwt@…

Replying to speed.rob@…:

Replying to tjb@…:

Is this any help?

I downloaded the patched tk from Github and tried it out. Worked like a charm. Maintainer, I believe you can close this ticket by updating the portfile to 8.5.7 and adding the patch.

I downloaded tk from branch de-carbon-8-5 and it worked.

  Changed 2 years ago by tjb@…

How about someone posting an updated port file here since it doesn't look like this is going to get properly fixed anytime soon. Thanks in advance.

  Changed 2 years ago by macports@…

  • cc macports@… added

Cc Me!

  Changed 2 years ago by macports@…

Can someone explain how to install tk on OS X 10.6 so that macports sees and uses it? Speed.rob mentions adding a patch, but I don't see one?

  Changed 2 years ago by blb@…

  • cc lvcargnini@… added

Cc reporter of dup #21783.

  Changed 2 years ago by lvcargnini@…

  • cc lvcargnini@… removed

Cc Me!

in reply to: ↑ 26   Changed 2 years ago by lvcargnini@…

Could you attach the patch ?

Replying to speed.rob@…:

I downloaded the patched tk from Github and tried it out. Worked like a charm. Maintainer, I believe you can close this ticket by updating the portfile to 8.5.7 and adding the patch.

  Changed 2 years ago by itrekkie@…

  • cc itrekkie@… added

Cc Me!

Changed 2 years ago by kuba@…

portfile patch to use github sources [disclaimer: I'm a total ports noob] -- wish starts up OK

  Changed 2 years ago by palimm@…

  • cc palimm@… added

Cc Me!

  Changed 2 years ago by speed.rob@…

I tried out kuba's patch, but it just ended up not being able to fetch the data. I looked around on github and found that a neatly gzipped copy of the patched tk has already been created, but it's from April 15th. I'm not sure how much work has been done on this branch since then, but I tried it out and it built without a hitch.

Note: I'm not 100% sure this is a real fix. My only interest in the +quartz variant is preventing python dependencies from installing a bunch of X stuff I don't need, so I've done almost no testing.

Changed 2 years ago by speed.rob@…

Portfile patch. Changes the source to a tk branch with no carbon dependencies hosted on github.

  Changed 2 years ago by anders.hasselqvist@…

  • cc anders.hasselqvist@… added

Cc Me!

  Changed 2 years ago by whitley@…

  • cc whitley@… added

Cc Me!

  Changed 2 years ago by yannick.cholette@…

  • cc yannick.cholette@… added

Cc Me!

  Changed 2 years ago by adhocrocker@…

  • cc adhocrocker@… added

Cc Me!

  Changed 2 years ago by speed.rob@…

  • cc speed.rob@… added

Cc Me!

  Changed 2 years ago by macports@…

CC me!

  Changed 2 years ago by macports@…

  • cc macports@… added

Cc Me!

  Changed 2 years ago by macports@…

  • cc macports@… removed

Cc Me!

  Changed 2 years ago by macports@…

  • cc macports@… added

Cc Me!

  Changed 2 years ago by macports@…

  • cc macports@… removed

Cc Me!

Changed 2 years ago by c.freden@…

Patch for tk Portfile, fetches source from  http://opensource.apple.com/tarballs/tcl/tcl-87.tar.gz

follow-up: ↓ 49   Changed 2 years ago by c.freden@…

I couldn't get speed.rob's or kuba's patches to compile.

But if i edit the port file to fetch the tcl-8.5.7 tarball from  http://opensource.apple.com (the link tjb posted) it all compiles and installes. It is a big tarball though (~40MB) so it is far from optimal but at least it worked for me. Attached patch in above post.

  Changed 2 years ago by mnot@…

  • cc mnot@… removed

Cc Me!

in reply to: ↑ 47   Changed 2 years ago by jp@…

If I do that, I get a build error:

error: /Library/Frameworks/Tcl.framework directory doesn't contain tclConfig.sh

  Changed 2 years ago by jp@…

  • cc jp@… added

Cc Me!

  Changed 2 years ago by royliu@…

  • cc royliu@… added

Cc Me!

  Changed 2 years ago by farcaller@…

  • cc farcaller@… added

Cc Me!

  Changed 2 years ago by farcaller@…

  • cc farcaller@… removed

Cc Me!

  Changed 2 years ago by michael@…

  • cc michael@… added

Cc Me!

  Changed 2 years ago by ddollar@…

  • cc ddollar@… added

Cc Me!

  Changed 2 years ago by jarias01@…

  • cc jarias01@… added

Cc Me!

  Changed 2 years ago by gthyni@…

  • cc gthyni@… added

Cc Me!

  Changed 2 years ago by thimo@…

  • cc thimo@… added

Cc Me!

follow-up: ↓ 63   Changed 2 years ago by zeph0x@…

I was getting the exact error in the description on my Snow Leopard system. speed.rob's patch allowed me to compile tk +quartz but c.freden's did not. I didn't try kuba's.

  Changed 2 years ago by zeph0x@…

  • cc zeph0x@… added

Cc Me!

  Changed 2 years ago by simpoir@…

  • cc simpoir@… added

Cc Me!

  Changed 2 years ago by macsforever2000@…

  • cc verena.wolf@… added

Cc reporter of dup #22641.

in reply to: ↑ 59   Changed 2 years ago by mdippery@…

Replying to zeph0x@…:

I was getting the exact error in the description on my Snow Leopard system. speed.rob's patch allowed me to compile tk +quartz but c.freden's did not. I didn't try kuba's.

I, too, got Tk to compile using speed.rob's patch. c.freden's did not either, and I did not try kuba's. I'm using 10.6.2.

  Changed 2 years ago by mdippery@…

  • cc mdippery@… added

Cc Me!

  Changed 2 years ago by trajano@…

  • cc trajano@… added

Cc Me!

  Changed 2 years ago by mdippery@…

  • cc mdippery@… removed

Cc Me!

  Changed 2 years ago by trajano@…

  • cc trajano@… removed

Cc Me!

  Changed 2 years ago by farcaller@…

speed.rob's patch works for me with minor modifications.

  Changed 2 years ago by macsforever2000@…

  • cc Damien@…, bgrupe@… added

Cc from dup #23042.

  Changed 2 years ago by yacoob@…

  • cc yacoob@… added

Cc Me!

Changed 2 years ago by anders.hasselqvist@…

Patch updated for tk 8.5.8. Fetching from github.

  Changed 2 years ago by tsunanet@…

  • cc tsunanet@… added

Cc Me!

  Changed 2 years ago by jsg8pitt@…

  • cc jsg8pitt@… added

Cc Me!

  Changed 2 years ago by eykosioux@…

  • cc eykosioux@… added

Cc Me!

  Changed 2 years ago by eykosioux@…

  • cc eykosioux@… removed

Cc Me!

  Changed 2 years ago by eykosioux@…

  • cc eykosioux@… added

Cc Me!

follow-up: ↓ 80   Changed 2 years ago by dan@…

anders.hasselqvist's patch works for me.

  Changed 2 years ago by jp@…

  • cc jp@… removed

Cc Me!

  Changed 2 years ago by jeremy.bethmont@…

  • cc jeremy.bethmont@… added

Cc Me!

  Changed 2 years ago by zimmermann@…

  • cc zimmermann@… added

Cc Me!

in reply to: ↑ 76   Changed 2 years ago by zeph0x@…

Replying to dan@…:

anders.hasselqvist's patch works for me.

Worked for me too. Running 10.6.2.

  Changed 2 years ago by yves@…

  • cc yves@… added

Cc Me!

follow-ups: ↓ 84 ↓ 94   Changed 2 years ago by jabronson@…

tk-858-decarbon.patch Download worked for me

  Changed 2 years ago by tim.stoop@…

  • cc tim.stoop@… added

Cc Me!

in reply to: ↑ 82 ; follow-ups: ↓ 86 ↓ 89   Changed 2 years ago by tim.stoop@…

Replying to jabronson@…:

tk-858-decarbon.patch Download worked for me

Same for me.

  Changed 2 years ago by stephen@…

  • cc stephen@… added

Cc Me!

in reply to: ↑ 84 ; follow-up: ↓ 87   Changed 2 years ago by lawrence.bandeira@…

Replying to tim.stoop@…:

Replying to jabronson@…:

tk-858-decarbon.patch Download worked for me

Same for me.

How do I apply this patch? Can you tell me step by step? Thanks forwards

in reply to: ↑ 86   Changed 23 months ago by andre.david@…

Replying to lawrence.bandeira@…:

How do I apply this patch? Can you tell me step by step? Thanks forwards

See  http://guide.macports.org/chunked/development.patches.html#development.patches.applying

  Changed 23 months ago by mdippery@…

  • cc mdippery@… added

Cc Me!

in reply to: ↑ 84 ; follow-up: ↓ 90   Changed 22 months ago by n.a.hawes@…

Replying to tim.stoop@…:

Replying to jabronson@…:

tk-858-decarbon.patch Download worked for me

Same for me.

And me.

in reply to: ↑ 89   Changed 22 months ago by yannick.cholette@…

Replying to n.a.hawes@…:

Replying to tim.stoop@…:

Replying to jabronson@…:

tk-858-decarbon.patch Download worked for me

Same for me.

And me.

And me too!

  Changed 21 months ago by rmsfisher@…

Has anyone worked with the tk developers to reconcile the above referenced git fork with the project's trunk? It is unlikely that a checkout from a git fork will be found a suitably "stable" replacement for an official release tarball.

  Changed 21 months ago by macports.org@…

  • cc macports.org@… added

Cc Me!

  Changed 21 months ago by jemaltz@…

  • cc jemaltz@… added

Cc Me!

in reply to: ↑ 82   Changed 20 months ago by jabronson@…

Replying to jabronson@…:

tk-858-decarbon.patch Download worked for me

Looks like MacPorts Python 2.6.5 is no longer able to build _tkinter.so against my MacPorts tk+quartz install. Opened #25221 in case anyone else is affected.

  Changed 19 months ago by markus.doits@…

  • cc markus.doits@… added

Cc Me!

  Changed 19 months ago by giancarlo.degani@…

  • cc giancarlo.degani@… added

Cc Me!

  Changed 19 months ago by blb@…

  • cc elvis.dowson@… added

Cc reporter of dup #25756.

  Changed 19 months ago by camillo.rossi@…

I am trying to apply the tk-858-decarbon.patch but it does not work:

 sudo patch -p0 < ~/Desktop/tk-858-decarbon.patch
patching file Portfile
Hunk #1 FAILED at 14.
1 out of 1 hunk FAILED -- saving rejects to file Portfile.rej

Maybe my Portfile is different?

  Changed 18 months ago by braden@…

  • cc braden@… added

Cc Me!

  Changed 18 months ago by elelay@…

  • cc elelay@… added

Cc Me!

  Changed 17 months ago by rmstonecipher@…

  • cc rmstonecipher@… added

Cc Me!

  Changed 17 months ago by joe@…

  • cc joe@… added

Cc Me!

  Changed 16 months ago by tascafoo@…

  • cc tascafoo@… added

Cc Me!

Changed 16 months ago by tascafoo@…

Patch updated for tk 8.5.9 (you probably also want tcl patch from bug 22954)

  Changed 16 months ago by calsaiwing@…

  • cc calsaiwing@… added

Cc Me!

  Changed 15 months ago by toreador@…

  • cc toreador@… added

Cc Me!

  Changed 14 months ago by bm55b@…

tk version 8.6 has switched to Cocoa.  http://www.markroseman.com/tcl/guide86.html

Currently 8.6b1 (beta1) and 8.6b2 is coming.  http://www.tcl.tk/cgi-bin/tct/tip/311

I'm going to use X until we get the ports work for 8.6 done

  Changed 12 months ago by a.y.harano@…

  • cc a.y.harano@… added

Cc Me!

  Changed 9 months ago by marc@…

  • cc marc@… added

Cc Me!

  Changed 8 months ago by macports@…

  • cc macports@… added

Cc Me!

  Changed 8 months ago by rizwank@…

  • cc rizwank@… added

Cc Me!

  Changed 6 months ago by obutovich@…

  • cc obutovich@… added

Cc Me!

  Changed 6 months ago by asahiro.g@…

  • cc asahiro.g@… added

Cc Me!

  Changed 6 months ago by kenny.macdermid@…

  • cc kenny.macdermid@… added

Cc Me!

  Changed 5 months ago by michelle.lynn.gill@…

  • cc michelle.lynn.gill@… added

Cc Me!

follow-up: ↓ 118   Changed 4 months ago by tascafoo@…

The title of this ticket is misleading. Tk +quartz doesn't build with the same error as 32-bit too.

  Changed 4 months ago by whitley@…

  • cc whitley@… removed

Cc Me!

  Changed 3 months ago by Torsten.Maehne@…

  • cc Torsten.Maehne@… added

Cc Me!

in reply to: ↑ 115   Changed 2 months ago by ryandesign@…

Replying to tascafoo@…:

The title of this ticket is misleading. Tk +quartz doesn't build with the same error as 32-bit too.

The title of the ticket is accurate, for Snow Leopard. If you are on Lion, then you are experiencing the consequences of the fact that QuickDraw was removed in Lion. See #32329 which tracks this.

  Changed 2 months ago by jabronson@…

  • cc jabronson@… removed

Cc Me!

Note: See TracTickets for help on using tickets.