Pass the BUILD file path to compile.

Review Request #1742 - Created Feb. 9, 2015 and submitted

Information
John Sirois
pants
jsirois/build_file/better_compile
1067
659318d...
Reviewers
pants-reviews
jinfeng, nhoward_tw
This produces SyntaxErrors with the path to the bad BUILD file included
in the error details.

 src/python/pants/base/build_file.py             |   2 +-
 tests/python/pants_test/base/BUILD              |   2 +-
 tests/python/pants_test/base/test_build_file.py | 211 +++++++++++++++++++++++++++-----------------------
 3 files changed, 115 insertions(+), 100 deletions(-)
CI went green here:
  https://travis-ci.org/pantsbuild/pants/builds/50115987

Issues

  • 0
  • 1
  • 0
  • 1
Description From Last Updated
John Sirois
Jin Feng
John Sirois
John Sirois
John Sirois
Review request changed

Status: Closed (submitted)

Patrick Lawson

   

Sorry for the late review--we should always be using six equivalents rather than introducing new deps on t.c.*.; e.g. https://pythonhosted.org/six/#six.exec_

  1. Fixed up here: https://rbcommons.com/s/twitter/r/1794/
Nick Howard (Twitter)
Ship It!
Loading...