2024 2002-04-23 02:11 0000 tcl-8.3.3.ebuild and tk-8.3.3.ebuild shouldn't cause tcl and tk to be threaded 2004-04-24 12:04:34 0000 1 1 1 Unclassified Gentoo Linux Development 1.1a x86 Linux RESOLVED FIXED P2 normal --- 17181 1 kevin@desertsol.com utx@penguin.cz agriffis@gentoo.org mholzer@gentoo.org mkennedy@gentoo.org perl@gentoo.org port001@gentoo.org seemant@gentoo.org kevin@desertsol.com 2002-04-23 02:11:04 0000 the tcl-8.3.3.ebuild and tk-8.3.3.ebuild automatically cause the --enable-threads flag on the ./configure line. (not affected by USE) I was trying to install lavaps-1.20 (http://www.isi.edu/~johnh/SOFTWARE/LAVAPS/index.html) which uses tcl/tk. lavaps built fine but crashed when trying to create a tcl intrepreter. I hacked the ebuilds for tcl and tk to not add the --enable-threads flag, emerged them again, rebuilt lavaps and it worked fine. I think the 2 ebuilds should be changed so that the use of the --enable-threads can be controlled by the USE environment variable. when tk was configuring, it even warns that enabling threads isn't a good idea. here's a stack dump from gdb when tcl/tk had threads enabled: 11:44pm1raven:/tmp/lavaps-1.20>gdb ./lavaps GNU gdb 5.1.1 Copyright 2002 Free Software Foundation, Inc. GDB is free software, covered by the GNU General Public License, and you are welcome to change it and/or distribute copies of it under certain conditions. Type "show copying" to see the conditions. There is absolutely no warranty for GDB. Type "show warranty" for details. This GDB was configured as "i686-pc-linux-gnu"... (gdb) run Starting program: /tmp/lavaps-1.20/./lavaps Program received signal SIG32, Real-time event 32. 0x40300ad9 in sigsuspend () from /lib/libc.so.6 (gdb) where #0 0x40300ad9 in sigsuspend () from /lib/libc.so.6 #1 0x4040ef59 in pthread_kill_other_threads_np () from /lib/libpthread.so.0 #2 0x4040e24b in pthread_create () from /lib/libpthread.so.0 #3 0x4014ac5d in Tcl_CreateThread () from /usr/lib/libtcl.so #4 0x4014b21f in Tcl_InitNotifier () from /usr/lib/libtcl.so #5 0x4012cc0a in TclInitNotifier () from /usr/lib/libtcl.so #6 0x4010f9aa in TclInitSubsystems () from /usr/lib/libtcl.so #7 0x400f3532 in Tcl_CreateInterp () from /usr/lib/libtcl.so #8 0x0804f4ef in tcl_blob::init () at tcl_blob.cc:335 #9 0x0804b4b1 in main (argc=1, argv=0xbffff9e4) at main.cc:127 #10 0x402ef3c1 in __libc_start_main () from /lib/libc.so.6 (gdb) The program is running. Exit anyway? (y or n) y drobbins@gentoo.org 2002-04-23 17:05:08 0000 sounds like we should disable threads with tcl/tk karltk@gentoo.org 2002-04-24 14:06:23 0000 Disabled in tcl-8.3.3; will be a use flag later on when we have a consistent policy for handling ebuild-specific use flags. port001@gentoo.org 2003-10-19 17:36:46 0000 Disabling threads blocks the aolserver (bug #17181) from being submited to portage because it requires tcl to be built with --enable-threads. Perhaps we could use an internal USE flag for the time being? lee@thaum.net 2003-10-22 12:15:29 0000 Created an attachment (id=19633) tcl ebuild with threads USE flag I've attached the modified tcl ebuild I'm using which has a 'threads' USE flag just in case its useful to anyone else. It makes merging <a href="http://bugs.gentoo.org/show_bug.cgi?id=17181">aolserver</a> easier. karltk@gentoo.org 2003-11-03 10:29:20 0000 Sorry. I don't have capacity to deal with the tcl and tk stuff anymore. seemant@gentoo.org 2003-11-21 23:03:35 0000 Stanislav, thoughts? port001@gentoo.org 2004-01-10 18:04:52 0000 Stanislav, if you don't have any objections I'm going to go with the local USE flag. This bug is holding up 4 others. mholzer@gentoo.org 2004-01-11 02:43:31 0000 so we could make "threads" as global flag it's also used in dev-lang/perl:threads - Enable Perl threads, has some compatibility problems mkennedy@gentoo.org 2004-01-11 16:13:50 0000 dev-lisp/clisp and dev-lisp/sbcl are two more ebuilds which can use a threads flag agriffis@gentoo.org 2004-01-17 10:32:01 0000 I'd vote for a global threads flag. With the upcoming portage it could even be set per-package. mholzer@gentoo.org 2004-01-17 10:38:48 0000 seemant: what do you think about a global threads USE-flag ? seemant@gentoo.org 2004-01-17 10:42:39 0000 marty, I don't know -- I think perl uses that -- but it's sort of a hidden flag because it could lead to dangerous things -- maybe sommething similar here? mholzer@gentoo.org 2004-01-17 11:03:50 0000 perl team, please tell us about the threads flag a little more esammer@gentoo.org 2004-01-18 03:03:31 0000 Threads in perl will cause a number of compatibility problems as most people know. This is even more problematic in terms of things like mod_perl-like environments, from what I've heard. Very few perl apps or modules make use of threads due to their buggy nature and I think most people are willing to hold out for perl6 for a decent thread implementation. As long as it's not on by default, a threads USE flag is fine, but I wouldn't be surprised if we see a slight upswing in perl bugs because of it. I believe that the perl ebuild currently uses a env variable to handle enabling and disabling of threads - maybe this could be used for tcl as well? My concern is that someone will enable a threads use flag for tcl and all of sudden have wacky problems with perl. port001@gentoo.org 2004-04-15 04:32:44 0000 If nobody has any objections, I'm going to goahead with this. I think i'll accompany a paused warning too. Anyone mind if I get rid of a few old ebuilds while I'm at it? port001@gentoo.org 2004-04-24 12:04:34 0000 in CVS. 19633 2003-10-22 12:15 0000 tcl ebuild with threads USE flag tcl-8.4.3.ebuild text/plain IyBDb3B5cmlnaHQgMTk5OS0yMDAzIEdlbnRvbyBUZWNobm9sb2dpZXMsIEluYy4KIyBEaXN0cmli dXRlZCB1bmRlciB0aGUgdGVybXMgb2YgdGhlIEdOVSBHZW5lcmFsIFB1YmxpYyBMaWNlbnNlIHYy CiMgJEhlYWRlcjogL2hvbWUvY3Zzcm9vdC9nZW50b28teDg2L2Rldi1sYW5nL3RjbC90Y2wtOC40 LjMuZWJ1aWxkLHYgMS40IDIwMDMvMDkvMzAgMjA6MzI6MzMga3VtYmEgRXhwICQKCgpTPSR7V09S S0RJUn0vJHtQTn0ke1BWfQpTUkNfVVJJPSJmdHA6Ly9mdHAuc2NyaXB0aWNzLmNvbS9wdWIvdGNs L3RjbDhfNC8ke1BOfSR7UFZ9LXNyYy50YXIuZ3oiCgoKSE9NRVBBR0U9Imh0dHA6Ly9kZXYuc2Ny aXB0aWNzLmNvbS9zb2Z0d2FyZS90Y2x0ay8iCgpERVNDUklQVElPTj0iVG9vbCBDb21tYW5kIExh bmd1YWdlIgoKIyBnZW50b29sa2l0IGlzIG5lZWRlZCBvbmx5IGZvciByZXZkZXAtcmVidWlsZApE RVBFTkQ9InZpcnR1YWwvZ2xpYmMKCT49YXBwLXBvcnRhZ2UvZ2VudG9vbGtpdC0wLjEuMjIiClJE RVBFTkQ9InZpcnR1YWwvZ2xpYmMiCgpTTE9UPSIwIgpMSUNFTlNFPSJCU0QiCktFWVdPUkRTPSJ+ eDg2IH5zcGFyYyB+bWlwcyIKIyB0aHJlYWRpbmcgaXMgbm90IHJlY29tbWVuZGVkIGFzIGl0IGJy ZWFrcyBzb21lIHBhY2thZ2VzCklVU0U9InRocmVhZHMiCgojIGh5cGVyLW9wdGltaXphdGlvbnMg dW50ZXN0ZWQuLi4KIwoKcGtnX3NldHVwKCkgewoJaWYgWyAtbiAiYHVzZSB0aHJlYWRzYCIgXQoJ dGhlbgoJCWV3YXJuICIiCgkJZXdhcm4gIlBMRUFTRSBOT1RFOiBZb3UgYXJlIGNvbXBpbGluZyB0 Y2wtOC40IHdpdGgiCgkJZXdhcm4gInRocmVhZGluZyBlbmFibGVkLiIKCQlld2FybiAiVGhyZWFk aW5nIGlzIG5vdCBzdXBwb3J0ZWQgYnkgYWxsIGFwcGxpY2F0aW9ucyIKCQlld2FybiAidGhhdCBj b21waWxlIGFnYWluc3QgdGNsLiBZb3UgdXNlIHRocmVhZGluZyBhdCIKCQlld2FybiAieW91ciBv d24gZGlzY3JldGlvbi4iCgkJZXdhcm4gIiIKCQlzbGVlcCAxMAoJZWxzZQoJCWV3YXJuICIiCgkJ ZXdhcm4gIlBMRUFTRSBOT1RFOiBJZiB5b3Ugd2FudCB0byBjb21waWxlIHRjbC04LjQgd2l0aCIK CQlld2FybiAidGhyZWFkaW5nIGVuYWJsZWQsIHlvdSBtdXN0IHJlc3RhcnQgdGhpcyBlbWVyZ2Ui CgkJZXdhcm4gIndpdGggVVNFPXRocmVhZHMgZW1lcmdlLi4uLiIKCQlld2FybiAiVGhyZWFkaW5n IGlzIG5vdCBzdXBwb3J0ZWQgYnkgYWxsIGFwcGxpY2F0aW9ucyIKCQlld2FybiAidGhhdCBjb21w aWxlIGFnYWluc3QgdGNsLiBZb3UgdXNlIHRocmVhZGluZyBhdCIKCQlld2FybiAieW91ciBvd24g ZGlzY3JldGlvbi4iCgkJZXdhcm4gIiIKCWZpCn0KCnNyY19jb21waWxlKCkgewoKCWxvY2FsIGxv Y2FsX2NvbmZpZ191c2U9IiIKCglpZiBbICJgdXNlIHRocmVhZHNgIiBdCgl0aGVuCgkJbG9jYWxf Y29uZmlnX3VzZT0iLS1lbmFibGUtdGhyZWFkcyIKCWZpCgoJY2QgJHtTfS91bml4CgkuL2NvbmZp Z3VyZSAtLWhvc3Q9JHtDSE9TVH0gXAoJCQkJLS1wcmVmaXg9L3VzciBcCgkJCQktLW1hbmRpcj0v dXNyL3NoYXJlL21hbiBcCgkJCQkke2xvY2FsX2NvbmZpZ191c2V9IFwKCQkJCXx8IGRpZQoKCWVt YWtlIENGTEFHUz0iJHtDRkxBR1N9IiB8fCBkaWUKCn0KCnNyY19pbnN0YWxsKCkgewoKCSNzaG9y dCB2ZXJzaW9uIG51bWJlcgoJbG9jYWwgdjEKCXYxPSR7UFYlLip9CgoJY2QgJHtTfS91bml4CglT PSBtYWtlIElOU1RBTExfUk9PVD0ke0R9IE1BTl9JTlNUQUxMX0RJUj0ke0R9L3Vzci9zaGFyZS9t YW4gaW5zdGFsbCB8fCBkaWUKCgkjIGZpeCB0aGUgdGNsQ29uZmlnLnNoIHRvIGVsaW1pbmF0ZSBy ZWZzIHRvIHRoZSBidWlsZCBkaXJlY3RvcnkKCXNlZCAtZSAicyxeVENMX0JVSUxEX0xJQl9TUEVD PSctTCR7U30vdW5peCxUQ0xfQlVJTERfTElCX1NQRUM9Jy1ML3Vzci9saWIsIiBcCgkJLWUgInMs XlRDTF9TUkNfRElSPScke1N9JyxUQ0xfU1JDX0RJUj0nL3Vzci9saWIvdGNsJHt2MX0vaW5jbHVk ZScsIiBcCgkJLWUgInMsXlRDTF9CVUlMRF9TVFVCX0xJQl9TUEVDPSctTCR7U30vdW5peCxUQ0xf QlVJTERfU1RVQl9MSUJfU1BFQz0nLUwvdXNyL2xpYiwiIFwKCQktZSAicyxeVENMX0JVSUxEX1NU VUJfTElCX1BBVEg9JyR7U30vdW5peCxUQ0xfQlVJTERfU1RVQl9MSUJfUEFUSD0nL3Vzci9saWIs IiBcCgkJLWUgInMsXlRDTF9MSUJfRklMRT0nbGlidGNsOC40Li5UQ0xfREJHWC4uc28nLFRDTF9M SUJfRklMRT1cImxpYnRjbDguNFwkXHtUQ0xfREJHWFx9LnNvXCIsIiBcCgkJJHtEfS91c3IvbGli L3RjbENvbmZpZy5zaCA+ICR7RH0vdXNyL2xpYi90Y2xDb25maWcuc2gubmV3CgltdiAke0R9L3Vz ci9saWIvdGNsQ29uZmlnLnNoLm5ldyAke0R9L3Vzci9saWIvdGNsQ29uZmlnLnNoCgoJIyBpbnN0 YWxsIHByaXZhdGUgaGVhZGVycwoJZG9kaXIgL3Vzci9saWIvdGNsJHt2MX0vaW5jbHVkZS91bml4 CglpbnN0YWxsIC1jIC1tMDY0NCAke1N9L3VuaXgvKi5oICR7RH0vdXNyL2xpYi90Y2wke3YxfS9p bmNsdWRlL3VuaXgKCWRvZGlyIC91c3IvbGliL3RjbCR7djF9L2luY2x1ZGUvZ2VuZXJpYwoJaW5z dGFsbCAtYyAtbTA2NDQgJHtTfS9nZW5lcmljLyouaCAke0R9L3Vzci9saWIvdGNsJHt2MX0vaW5j bHVkZS9nZW5lcmljCglybSAtZiAke0R9L3Vzci9saWIvdGNsJHt2MX0vaW5jbHVkZS9nZW5lcmlj L3RjbC5oCglybSAtZiAke0R9L3Vzci9saWIvdGNsJHt2MX0vaW5jbHVkZS9nZW5lcmljL3RjbERl Y2xzLmgKCXJtIC1mICR7RH0vdXNyL2xpYi90Y2wke3YxfS9pbmNsdWRlL2dlbmVyaWMvdGNsUGxh dERlY2xzLmgKCgkjIGluc3RhbGwgc3ltbGluayBmb3IgbGlicmFyaWVzCglkb3N5bSAvdXNyL2xp Yi9saWJ0Y2wke3YxfS5zbyAvdXNyL2xpYi9saWJ0Y2wuc28KCWRvc3ltIC91c3IvbGliL2xpYnRj bHN0dWIke3YxfS5hIC91c3IvbGliL2xpYnRjbHN0dWIuYQoKCWxuIC1zZiB0Y2xzaCR7djF9ICR7 RH0vdXNyL2Jpbi90Y2xzaAoKCWNkICR7U30KCWRvZG9jIFJFQURNRSBjaGFuZ2VzIGxpY2Vuc2Uu dGVybXMKCn0KCnBrZ19wb3N0aW5zdCgpIHsKCWV3YXJuCglld2FybiAiSWYgeW91J3JlIHVwZ3Jh ZGluZyBmcm9tIHRjbC04LjMsIHlvdSBtdXN0IHJlY29tcGlsZSB0aGUgb3RoZXIiCglld2FybiAi cGFja2FnZXMgb24geW91ciBzeXN0ZW0gdGhhdCBsaW5rIHdpdGggdGNsIGFmdGVyIHRoZSB1cGdy YWRlIgoJZXdhcm4gImNvbXBsZXRlcy4gIFRvIHBlcmZvcm0gdGhpcyBhY3Rpb24sIHBsZWFzZSBy dW4gcmV2ZGVwLXJlYnVpbGQuIgoJZXdhcm4gIklmIHlvdSBoYXZlIGRldi1sYW5nL3RrIGFuZCBk ZXYtdGNsdGsvdGNseCBpbnN0YWxsZWQgeW91IHNob3VsZCIKCWV3YXJuICJ1cGdyYWRlIHRoZW0g YmVmb3JlIHRoaXMgcmVjb21waWxhdGlvbiwgdG9vLCIKCWV3YXJuCglzbGVlcCA1Cn0K