Eval list- and dict-typed option values, instead of JSON-parsing them.

Review Request #1443 — Created Dec. 8, 2014 and submitted

benjyw
pants
aae7049...
pants-reviews
davidt, patricklaw, zundel

This is to support list concatenation syntax in config files, e.g.,

dev_opts: %(basic_opts)s + %(big_mem_opts)s + ['-ldebug']

This is a legitimate use case that we actually have, but that I overlooked
when I made the JSON requirement in 8c3b5b33e8cabdf599ed1332067283ffc067a13a.

Ran options unit tests.

CI baking.

PA
  1. Ship It!

  2. 
      
BE
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as 676c0e80b90e2e6293f7204db8cb57a412c53d8f
Loading...