Fix the SetupPy target ownership check.

Review Request #4315 — Created Oct. 15, 2016 and submitted — Latest diff uploaded

jsirois
pants
jsirois/python/publish/fix_ownership_check
3968, 3969
3a022b9...
pants-reviews
benjyw, kwlzn
Previously the check was too restrictive and considered targets that did
not own files when the single publish ownership check is only intended
to prevent publishing the same file in more than one package.

 src/python/pants/backend/python/tasks/setup_py.py             | 14 ++++++------
 tests/python/pants_test/backend/python/tasks/test_setup_py.py | 58 ++++++++++++++++++++++++++++++++++++++++++++++++--
 2 files changed, 64 insertions(+), 8 deletions(-)

Tested this over in Aurora ad-hoc and it solved the publish problem
there as well as passing the new test emulating the Aurora prep_command
arrangement.

CI went green here:
https://travis-ci.org/pantsbuild/pants/builds/169610485

Loading...