Remove any special casing around the former android backend.

Review Request #3531 - Created March 3, 2016 and submitted

Information
Mateo Rodriguez
pants
3002
3530
Reviewers
pants-reviews
gmalmquist, jsirois, patricklaw, stuhood, zundel

This has to land along with the move to contrib
but I broke it up to make it easier to review.

This commit removes all the special casing that accrued
over the years as a consequence of having Android as a
separate backend. It also puts together some new
special casing that allows the android tests to still
be skipped in circumstances where that is needed.

  • This adds a flag to skip the new android contrib package
    when running CIs. Because OSX does not have container
    based CI it would have to download and bootstrap the SDK
    each run. That is not feasible for multiple reasons.

  • During the release of the package, it will not run the actual
    Android tests because the releaser will usually not have the
    SDK installed on their machine. This just checks to see if the
    various goals are installed.

CI.sh and release.sh passed.

A travis run with the code from both CR 3530 and 3531 is away at:
https://travis-ci.org/pantsbuild/pants/builds/113785902

Both of those commits will have to land at the same time.

Issues

  • 0
  • 1
  • 0
  • 1
Description From Last Updated
Eric Ayers
Mateo Rodriguez
Eric Ayers
Mateo Rodriguez
Mateo Rodriguez
Mateo Rodriguez
Mateo Rodriguez
Mateo Rodriguez
Eric Ayers
Stu Hood
Mateo Rodriguez
Mateo Rodriguez
Review request changed

Status: Closed (submitted)

Change Summary:

Thanks - submitted as c898eb2eeea218158679fb03bec031d05bab3320 a join of CR 3530 and CR 3531.

Loading...