Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 498612 - dev-lang/tcl: hard mask USE=threading or use built-with-use dependencies instead of postinst message
Summary: dev-lang/tcl: hard mask USE=threading or use built-with-use dependencies inst...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: TCL/TK Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-19 19:36 UTC by Paweł Hajdan, Jr. (RETIRED)
Modified: 2022-09-11 20:00 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paweł Hajdan, Jr. (RETIRED) gentoo-dev 2014-01-19 19:36:42 UTC 
Instead of the following postinst message I got for dev-lang/tcl-8.5.13-r1:

PLEASE NOTE: You are compiling tcl-8.5.13 with
threading enabled.
Threading is not supported by all applications
that compile against tcl. You use threading at
your own discretion.

it'd be preferred to either hard mask USE=threading for tcl, or just adjust packages that break with it to require dev-lang/tcl[-threading] . See
http://devmanual.gentoo.org/general-concepts/dependencies/ .

The advantage is I can know about this at dependency resolution time, instead of things possibly randomly breaking later.
Comment 1 Justin Lecher (RETIRED) gentoo-dev 2014-01-19 19:41:41 UTC 
actually this is some ancient stuff from before I took over maintainership. And I personally never experienced any problems.

I will sleep over it, but I think solution 2 is the most appropriate.
Comment 2 Justin Lecher (RETIRED) gentoo-dev 2014-01-20 07:35:13 UTC 
+  20 Jan 2014; Justin Lecher <jlec@gentoo.org> tcl-8.5.13-r1.ebuild,
+  -tcl-8.5.14.ebuild, -tcl-8.5.15.ebuild, tcl-8.5.15-r1.ebuild,
+  -tcl-8.6.0-r1.ebuild, -tcl-8.6.1.ebuild:
+  Drop old; drop notification about threading support in favour of USE deps,
+  #498612
+