aboutsummaryrefslogtreecommitdiffstats
path: root/Library/Formula
diff options
context:
space:
mode:
authorJacob Howard2013-06-23 15:52:32 +0200
committerAdam Vandenberg2013-07-02 09:59:33 -0700
commitc14aa2b6cdcea31b1a7cf624374bea413a8de83a (patch)
treed35e4e67ee2fdb6345a644e956f311ec7c29a9ae /Library/Formula
parent0ee85a4956a89615dbea68e84c52258580795d9c (diff)
downloadhomebrew-c14aa2b6cdcea31b1a7cf624374bea413a8de83a.tar.bz2
ROOT: Add Python as a dependency to assure PyROOT bindings are correctly linked.
ROOT currently does not have a :python dependency, which causes ROOT to link against system python instead of homebrew python. This commit adds a :python dependency, which does not affect users who use system python (since homebrew will still link them to system python), but which does fix linking for users who use a brewed python. The logic behind making this dependency non-optional is: 1) Speaking from first-hand experience, most LHC analyses use PyROOT more than ROOT's built-in CINT interpreter, so it makes sense to have this always-on, as users will expect it. 2) For users who don't have a brewed python, this doesn't add any extra build dependencies. 3) The existing formula is compiled with the --all flag, and has no option to disable python bindings, so this really doesn't change the interface to the formula. Closes #20704. Signed-off-by: Adam Vandenberg <flangy@gmail.com>
Diffstat (limited to 'Library/Formula')
-rw-r--r--Library/Formula/root.rb1
1 files changed, 1 insertions, 0 deletions
diff --git a/Library/Formula/root.rb b/Library/Formula/root.rb
index 7a18c0698..e505c89e5 100644
--- a/Library/Formula/root.rb
+++ b/Library/Formula/root.rb
@@ -8,6 +8,7 @@ class Root < Formula
depends_on 'fftw' => :optional
depends_on :x11
+ depends_on :python
def install
#Determine architecture