From c14aa2b6cdcea31b1a7cf624374bea413a8de83a Mon Sep 17 00:00:00 2001 From: Jacob Howard Date: Sun, 23 Jun 2013 15:52:32 +0200 Subject: 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 --- Library/Formula/root.rb | 1 + 1 file changed, 1 insertion(+) (limited to 'Library/Formula') 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 -- cgit v1.2.3