Error out on duplicate artifacts for jar publish.

Review Request #3481 - Created Feb. 19, 2016 and submitted

Information
Garrett Malmquist
pants
gmalmquist/check-for-duplicate-provides
2941
Reviewers
pants-reviews
benjyw, patricklaw, stuhood, zundel
This adds some extra validation to check_targets to ensure that we
don't have multiple libraries which provide the same artifact.

A nicely formatted error message is output in the event of a
conflict:

    DuplicateArtifactError: Multiple targets define the same artifacts!

      com.example-nail -> internal -> http://example.com
(/private/var/folders/wf/q7mqymv13kq1w7vkzq6y0fr4000258/T/tmpZRPxjZ_BUILD_ROOT/pushdb)
is defined by:
        a:a
        conflict:conflict

Added a test to test_jar_publish.py.

CI went green here: https://travis-ci.org/gmalmquist/pants/builds/110481214

Issues

  • 0
  • 0
  • 1
  • 1
Description From Last Updated
Eric Ayers
Eric Ayers
Garrett Malmquist
Garrett Malmquist
Review request changed

Status: Closed (submitted)

Change Summary:

In commit 0420fc18d54c8ae992e498aedb8a681daceeddef; thanks Eric.

Stu Hood

Thanks! Could have sworn we had something like this already... *shrug.

Loading...