Skip to content

Commit

Permalink
Add depends_on('perl') to git package
Browse files Browse the repository at this point in the history
This commit changes the git package to depend_on('perl').  The system
perl is not always sufficient to install git (e.g. a CentOS7 system with
the development tools group installed has perl but not the
ExtUtils::MakeMaker package that git needs) and one can't always update
the system's perl.

This PR depends_on PR spack#1339, which adds a perl package to spack.
  • Loading branch information
hartzell committed Aug 3, 2016
1 parent 3e6ebd5 commit ac3d128
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions var/spack/repos/builtin/packages/git/package.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ class Git(Package):
depends_on("zlib")

# Use system perl for now.
# depends_on("perl")
depends_on("perl")
# depends_on("pcre")

def install(self, spec, prefix):
Expand All @@ -64,8 +64,8 @@ def install(self, spec, prefix):
"--without-pcre",
"--with-openssl=%s" % spec['openssl'].prefix,
"--with-zlib=%s" % spec['zlib'].prefix,
"--with-curl=%s" % spec['curl'].prefix,
"--with-expat=%s" % spec['expat'].prefix
"--with-expat=%s" % spec['expat'].prefix,
"--with-perl=%s" % join_path(spec['perl'].prefix.bin, 'perl'),
]

which('autoreconf')('-i')
Expand Down

0 comments on commit ac3d128

Please sign in to comment.