Skip to content

Commit

Permalink
Merge pull request #3 from lsst/tickets/DM-11532
Browse files Browse the repository at this point in the history
Do not split the return value from get_include()
  • Loading branch information
timj committed Aug 9, 2017
2 parents c1bdd73 + 8aaab3e commit 4dbe59c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion ups/numpy.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,13 @@ class Configuration(lsst.sconsUtils.Configuration):
def configure(self, conf, packages, check=False, build=True):
lsst.sconsUtils.log.info("Configuring package '%s'." % self.name)
try:
# Returns the path to a single include directory
output = subprocess.check_output(["python", "-c",
"import numpy; print(numpy.get_include())"]).decode()
output = output.strip()
except subprocess.CalledProcessError:
return False
conf.env.AppendUnique(XCPPPATH=output.split())
conf.env.AppendUnique(XCPPPATH=[output])
return True

config = Configuration()

0 comments on commit 4dbe59c

Please sign in to comment.