Opened 11 years ago

Closed 8 years ago

#20799 closed defect (fixed)

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@…, jpgoldberg (Jeffrey Goldberg), tjb@…, vitaly@…, jpenney (Jason Penney), rmsfisher@…, hgbrauen@…, markus@…, macports@…, itrekkie@…, palimm@…, nevil (Anders Hasselqvist), yannick.cholette@…, adhocrocker@…, speed.rob@…, royliu@…, michael@…, ddollar@…, jarias01@…, gthyni@…, thimo@…, zeph0x@…, simpoir@…, verena.wolf@…, Damien@…, bgrupe27, yacoob@…, tsunanet@…, jsg8pitt@…, eykosioux@…, jeremy.bethmont@…, zimmermann@…, yves@…, tim.stoop@…, stephen@…, mdippery@…, macports.org@…, jemaltz@…, markus.doits@…, giancarlo.degani@…, elvis.dowson@…, braden@…, elelay (Eric Le Lay), rmstonecipher@…, joe@…, avysk (Alexey Vyskubov), calsaiwing@…, toreador@…, a.y.harano@…, marc@…, macports@…, rizwank@…, obutovich@…, asahiro.g@…, kenny.macdermid@…, michelle.lynn.gill@…, maehne (Torsten Maehne), tonycpsu@…
Port: tk

Description (last modified by nerdling (Jeremy Lavergne))

Using MacPorts trunk:

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

Attachments (7)

tk+quartz_debug_build.txt (222.4 KB) - added by nerdling (Jeremy Lavergne) 11 years ago.
tk.txt (242.7 KB) - added by andreas@… 11 years ago.
x11_tk_Portfile.das.patch (1.8 KB) - added by KubaO (Kuba Ober) 11 years ago.
portfile patch to use github sources [disclaimer: I'm a total ports noob] -- wish starts up OK
tk-decarbon.patch (1.2 KB) - added by speed.rob@… 11 years ago.
Portfile patch. Changes the source to a tk branch with no carbon dependencies hosted on github.
Portfile-tk.diff (1.3 KB) - added by c.freden@… 11 years ago.
Patch for tk Portfile, fetches source from http://opensource.apple.com/tarballs/tcl/tcl-87.tar.gz
tk-858-decarbon.patch (1.0 KB) - added by nevil (Anders Hasselqvist) 11 years ago.
Patch updated for tk 8.5.8. Fetching from github.
tk-859-decarbon.patch (1.0 KB) - added by avysk (Alexey Vyskubov) 10 years ago.
Patch updated for tk 8.5.9 (you probably also want tcl patch from bug 22954)

Download all attachments as: .zip

Change History (128)

comment:1 Changed 11 years ago by jmroot (Joshua Root)

Cc: eallik@… removed
Keywords: snowleopard added
Owner: changed from macports-tickets@… to mww@…

Changed 11 years ago by nerdling (Jeremy Lavergne)

Attachment: tk+quartz_debug_build.txt added

comment:2 Changed 11 years ago by nerdling (Jeremy Lavergne)

Description: modified (diff)

comment:3 Changed 11 years ago by jmroot (Joshua Root)

Keywords: LP64 added; snowleopard removed
Summary: tk +quartz doesn't build on SLtk +quartz doesn't build 64-bit
Version: 1.8.01.8.99

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

comment:4 in reply to:  3 Changed 11 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.

comment:5 Changed 11 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 11 years ago by andreas@…

Attachment: tk.txt added

comment:6 Changed 11 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!

comment:7 Changed 11 years ago by simon@…

Cc: simon@… added

Cc Me!

comment:8 Changed 11 years ago by wsiegrist@…

Cc: wsiegrist@… added

Cc Me!

comment:9 Changed 11 years ago by wsiegrist@…

Cc: wsiegrist@… removed

Cc Me!

comment:10 Changed 11 years ago by jmroot (Joshua Root)

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.

comment:11 Changed 11 years ago by cedayiv@…

Cc: cedayiv@… added

Cc Me!

comment:12 Changed 11 years ago by kiwi.2008@…

Cc: kiwi.2008@… added

Cc Me!

comment:13 Changed 11 years ago by andreas@…

Cc: andreas@… removed

Cc Me!

comment:14 Changed 11 years ago by mnot@…

Cc: mnot@… added

Cc Me!

comment:15 Changed 11 years ago by jpgoldberg (Jeffrey Goldberg)

Cc: jeffrey@… added

Cc Me!

comment:16 Changed 11 years ago by jabronson@…

Cc: jabronson@… added

Cc Me!

comment:18 Changed 11 years ago by tjb@…

Cc: tjb@… added

Cc Me!

comment:19 Changed 11 years ago by vitaly@…

Cc: vitaly@… added

Cc Me!

comment:20 Changed 11 years ago by jpenney (Jason Penney)

Cc: jpenney@… added

Cc Me!

comment:21 Changed 11 years ago by rmsfisher@…

Cc: rmsfisher@… added

Cc Me!

comment:22 Changed 11 years ago by blb@…

Cc: hgbrauen@… added

Cc reporter of dup #21439.

comment:23 Changed 11 years ago by markus@…

Cc: markus@… added

Cc Me!

comment:24 Changed 11 years ago by markus@…

Cc: markus@… removed

Cc Me!

comment:25 Changed 11 years ago by markus@…

Cc: markus@… added

Cc Me!

comment:26 in reply to:  17 ; Changed 11 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.

comment:27 in reply to:  26 Changed 11 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.

comment:28 Changed 11 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.

comment:29 Changed 11 years ago by macports@…

Cc: macports@… added

Cc Me!

comment:30 Changed 11 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?

comment:31 Changed 11 years ago by blb@…

Cc: lvcargnini@… added

Cc reporter of dup #21783.

comment:32 Changed 11 years ago by lvcargnini@…

Cc: lvcargnini@… removed

Cc Me!

comment:33 in reply to:  26 Changed 11 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.

comment:34 Changed 11 years ago by itrekkie@…

Cc: itrekkie@… added

Cc Me!

Changed 11 years ago by KubaO (Kuba Ober)

Attachment: x11_tk_Portfile.das.patch added

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

comment:35 Changed 11 years ago by palimm@…

Cc: palimm@… added

Cc Me!

comment:36 Changed 11 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 11 years ago by speed.rob@…

Attachment: tk-decarbon.patch added

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

comment:37 Changed 11 years ago by nevil (Anders Hasselqvist)

Cc: anders.hasselqvist@… added

Cc Me!

comment:38 Changed 11 years ago by whitley@…

Cc: whitley@… added

Cc Me!

comment:39 Changed 11 years ago by yannick.cholette@…

Cc: yannick.cholette@… added

Cc Me!

comment:40 Changed 11 years ago by adhocrocker@…

Cc: adhocrocker@… added

Cc Me!

comment:41 Changed 11 years ago by speed.rob@…

Cc: speed.rob@… added

Cc Me!

comment:42 Changed 11 years ago by macports@…

CC me!

comment:43 Changed 11 years ago by macports@…

Cc: macports@… added

Cc Me!

comment:44 Changed 11 years ago by macports@…

Cc: macports@… removed

Cc Me!

comment:45 Changed 11 years ago by macports@…

Cc: macports@… added

Cc Me!

comment:46 Changed 11 years ago by macports@…

Cc: macports@… removed

Cc Me!

Changed 11 years ago by c.freden@…

Attachment: Portfile-tk.diff added

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

comment:47 Changed 11 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.

comment:48 Changed 11 years ago by mnot@…

Cc: mnot@… removed

Cc Me!

comment:49 in reply to:  47 Changed 11 years ago by jp@…

If I do that, I get a build error:

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

comment:50 Changed 11 years ago by jp@…

Cc: jp@… added

Cc Me!

comment:51 Changed 11 years ago by royliu@…

Cc: royliu@… added

Cc Me!

comment:52 Changed 11 years ago by farcaller@…

Cc: farcaller@… added

Cc Me!

comment:53 Changed 11 years ago by farcaller@…

Cc: farcaller@… removed

Cc Me!

comment:54 Changed 11 years ago by michael@…

Cc: michael@… added

Cc Me!

comment:55 Changed 11 years ago by ddollar@…

Cc: ddollar@… added

Cc Me!

comment:56 Changed 11 years ago by jarias01@…

Cc: jarias01@… added

Cc Me!

comment:57 Changed 11 years ago by gthyni@…

Cc: gthyni@… added

Cc Me!

comment:58 Changed 11 years ago by thimo@…

Cc: thimo@… added

Cc Me!

comment:59 Changed 11 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.

comment:60 Changed 11 years ago by zeph0x@…

Cc: zeph0x@… added

Cc Me!

comment:61 Changed 11 years ago by simpoir@…

Cc: simpoir@… added

Cc Me!

comment:62 Changed 11 years ago by mf2k (Frank Schima)

Cc: verena.wolf@… added

Cc reporter of dup #22641.

comment:63 in reply to:  59 Changed 11 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.

comment:64 Changed 11 years ago by mdippery@…

Cc: mdippery@… added

Cc Me!

comment:65 Changed 11 years ago by trajano@…

Cc: trajano@… added

Cc Me!

comment:66 Changed 11 years ago by mdippery@…

Cc: mdippery@… removed

Cc Me!

comment:67 Changed 11 years ago by trajano@…

Cc: trajano@… removed

Cc Me!

comment:68 Changed 11 years ago by farcaller@…

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

comment:69 Changed 11 years ago by mf2k (Frank Schima)

Cc: Damien@… bgrupe@… added

Cc from dup #23042.

comment:70 Changed 11 years ago by yacoob@…

Cc: yacoob@… added

Cc Me!

Changed 11 years ago by nevil (Anders Hasselqvist)

Attachment: tk-858-decarbon.patch added

Patch updated for tk 8.5.8. Fetching from github.

comment:71 Changed 11 years ago by tsunanet@…

Cc: tsunanet@… added

Cc Me!

comment:72 Changed 11 years ago by jsg8pitt@…

Cc: jsg8pitt@… added

Cc Me!

comment:73 Changed 11 years ago by eykosioux@…

Cc: eykosioux@… added

Cc Me!

comment:74 Changed 11 years ago by eykosioux@…

Cc: eykosioux@… removed

Cc Me!

comment:75 Changed 11 years ago by eykosioux@…

Cc: eykosioux@… added

Cc Me!

comment:76 Changed 11 years ago by ddennedy (Dan Dennedy)

anders.hasselqvist's patch works for me.

comment:77 Changed 11 years ago by jp@…

Cc: jp@… removed

Cc Me!

comment:78 Changed 11 years ago by jeremy.bethmont@…

Cc: jeremy.bethmont@… added

Cc Me!

comment:79 Changed 11 years ago by zimmermann@…

Cc: zimmermann@… added

Cc Me!

comment:80 in reply to:  76 Changed 11 years ago by zeph0x@…

Replying to dan@…:

anders.hasselqvist's patch works for me.

Worked for me too. Running 10.6.2.

comment:81 Changed 11 years ago by yves@…

Cc: yves@… added

Cc Me!

comment:82 Changed 11 years ago by jabronson@…

tk-858-decarbon.patch worked for me

comment:83 Changed 10 years ago by tim.stoop@…

Cc: tim.stoop@… added

Cc Me!

comment:84 in reply to:  82 ; Changed 10 years ago by tim.stoop@…

Replying to jabronson@…:

tk-858-decarbon.patch worked for me

Same for me.

comment:85 Changed 10 years ago by stephen@…

Cc: stephen@… added

Cc Me!

comment:86 in reply to:  84 ; Changed 10 years ago by lawrence.bandeira@…

Replying to tim.stoop@…:

Replying to jabronson@…:

tk-858-decarbon.patch worked for me

Same for me.

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

comment:87 in reply to:  86 Changed 10 years 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

comment:88 Changed 10 years ago by mdippery@…

Cc: mdippery@… added

Cc Me!

comment:89 in reply to:  84 ; Changed 10 years ago by n.a.hawes@…

Replying to tim.stoop@…:

Replying to jabronson@…:

tk-858-decarbon.patch worked for me

Same for me.

And me.

comment:90 in reply to:  89 Changed 10 years ago by yannick.cholette@…

Replying to n.a.hawes@…:

Replying to tim.stoop@…:

Replying to jabronson@…:

tk-858-decarbon.patch worked for me

Same for me.

And me.

And me too!

comment:91 Changed 10 years 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.

comment:92 Changed 10 years ago by macports.org@…

Cc: macports.org@… added

Cc Me!

comment:93 Changed 10 years ago by jemaltz@…

Cc: jemaltz@… added

Cc Me!

comment:94 in reply to:  82 Changed 10 years ago by jabronson@…

Replying to jabronson@…:

tk-858-decarbon.patch 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.

comment:95 Changed 10 years ago by markus.doits@…

Cc: markus.doits@… added

Cc Me!

comment:96 Changed 10 years ago by giancarlo.degani@…

Cc: giancarlo.degani@… added

Cc Me!

comment:97 Changed 10 years ago by blb@…

Cc: elvis.dowson@… added

Cc reporter of dup #25756.

comment:98 Changed 10 years 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?

comment:99 Changed 10 years ago by braden@…

Cc: braden@… added

Cc Me!

comment:100 Changed 10 years ago by elelay (Eric Le Lay)

Cc: elelay@… added

Cc Me!

comment:101 Changed 10 years ago by rmstonecipher@…

Cc: rmstonecipher@… added

Cc Me!

comment:102 Changed 10 years ago by joe@…

Cc: joe@… added

Cc Me!

comment:103 Changed 10 years ago by avysk (Alexey Vyskubov)

Cc: tascafoo@… added

Cc Me!

Changed 10 years ago by avysk (Alexey Vyskubov)

Attachment: tk-859-decarbon.patch added

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

comment:104 Changed 10 years ago by calsaiwing@…

Cc: calsaiwing@… added

Cc Me!

comment:105 Changed 10 years ago by toreador@…

Cc: toreador@… added

Cc Me!

comment:106 Changed 10 years 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

comment:107 Changed 9 years ago by a.y.harano@…

Cc: a.y.harano@… added

Cc Me!

comment:108 Changed 9 years ago by marc@…

Cc: marc@… added

Cc Me!

comment:109 Changed 9 years ago by macports@…

Cc: macports@… added

Cc Me!

comment:110 Changed 9 years ago by rizwank@…

Cc: rizwank@… added

Cc Me!

comment:111 Changed 9 years ago by obutovich@…

Cc: obutovich@… added

Cc Me!

comment:112 Changed 9 years ago by asahiro.g@…

Cc: asahiro.g@… added

Cc Me!

comment:113 Changed 9 years ago by kenny.macdermid@…

Cc: kenny.macdermid@… added

Cc Me!

comment:114 Changed 9 years ago by michelle.lynn.gill@…

Cc: michelle.lynn.gill@… added

Cc Me!

comment:115 Changed 9 years ago by avysk (Alexey Vyskubov)

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

comment:116 Changed 9 years ago by whitley@…

Cc: whitley@… removed

Cc Me!

comment:117 Changed 9 years ago by maehne (Torsten Maehne)

Cc: Torsten.Maehne@… added

Cc Me!

comment:118 in reply to:  115 Changed 9 years ago by ryandesign (Ryan Schmidt)

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.

comment:119 Changed 9 years ago by jabronson@…

Cc: jabronson@… removed

Cc Me!

comment:120 Changed 8 years ago by tonycpsu@…

Cc: tonycpsu@… added

Cc Me!

comment:121 Changed 8 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.