Skip to content

Commit

Permalink
configure.py --tools should set a list instead of a string
Browse files Browse the repository at this point in the history
Currently the --tools option does not work because it is setting a string value
for 'build.tools'. It should be a list of strings instead.
  • Loading branch information
danc86 committed Apr 8, 2018
1 parent 08ad376 commit 51b17db
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/bootstrap/configure.py
Expand Up @@ -145,7 +145,7 @@ def v(*args):
o("full-bootstrap", "build.full-bootstrap", "build three compilers instead of two")
o("extended", "build.extended", "build an extended rust tool set")

v("tools", "build.tools", "List of extended tools will be installed")
v("tools", None, "List of extended tools will be installed")
v("build", "build.build", "GNUs ./configure syntax LLVM build triple")
v("host", None, "GNUs ./configure syntax LLVM host triples")
v("target", None, "GNUs ./configure syntax LLVM target triples")
Expand Down Expand Up @@ -321,6 +321,8 @@ def set(key, value):
set('target.{}.llvm-config'.format(build()), value + '/bin/llvm-config')
elif option.name == 'jemalloc-root':
set('target.{}.jemalloc'.format(build()), value + '/libjemalloc_pic.a')
elif option.name == 'tools':
set('build.tools', value.split(','))
elif option.name == 'host':
set('build.host', value.split(','))
elif option.name == 'target':
Expand Down

0 comments on commit 51b17db

Please sign in to comment.