Merge branch 'alfredo-gimenez-thrift' into develop

1 files changed, 10 insertions, 4 deletions

diff --git a/var/spack/packages/thrift/package.py b/var/spack/packages/thrift/package.py
index afde181ed9..0e15052f64 100644
--- a/var/spack/packages/thrift/package.py
+++ b/var/spack/packages/thrift/package.py
@@ -1,9 +1,9 @@
 from spack import *
 
 class Thrift(Package):
-    """The Apache Thrift software framework, for scalable cross-language services 
-    development, combines a software stack with a code generation engine to build 
-    services that work efficiently and seamlessly between C++, Java, Python, PHP, 
+    """The Apache Thrift software framework, for scalable cross-language services
+    development, combines a software stack with a code generation engine to build
+    services that work efficiently and seamlessly between C++, Java, Python, PHP,
     Ruby, Erlang, Perl, Haskell, C#, Cocoa, JavaScript, Node.js, Smalltalk, OCaml
      and Delphi and other languages."""
 
@@ -12,6 +12,8 @@ class Thrift(Package):
 
     version('0.9.2', '89f63cc4d0100912f4a1f8a9dee63678')
 
+    extends("python")
+
     depends_on("autoconf")
     depends_on("automake")
     depends_on("bison")
@@ -20,15 +22,19 @@ class Thrift(Package):
     depends_on("jdk")
     depends_on("libtool")
     depends_on("openssl")
+    depends_on("python")
 
     # Compilation fails for most languages, fortunately cpp installs fine
     # All other languages (yes, including C) are omitted until someone needs them
     def install(self, spec, prefix):
+        env["PY_PREFIX"]   = prefix
+        env["JAVA_PREFIX"] = prefix
+
         configure("--prefix=%s" % prefix,
                   "--with-boost=%s" % spec['boost'].prefix,
                   "--with-c=no",
                   "--with-go=no",
-                  "--with-python=no",
+                  "--with-python=yes",
                   "--with-lua=no",
                   "--with-php=no",
                   "--with-qt4=no",