Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix #720 - Fix ABI etc for Travis builds #721

Closed
wants to merge 4 commits into from
Closed

Fix #720 - Fix ABI etc for Travis builds #721

wants to merge 4 commits into from

Conversation

barbeau
Copy link
Member

@barbeau barbeau commented Nov 23, 2016 •
edited
Loading

  • Add separate ANDOID_TAG ABI tag variable
  • Duplicate tools to get the new repository-11.xml and to install Android SDK tools 25.1.x
  • Change system image names to match new Android SDK
  • Change emulator start command to use new ABI tag variable to specify Google APIs
  • Switch to 64-bit emulator image
  • Remove extra API level emulator image

See travis-ci/travis-ci#6122 (comment) for details.

See also googlemaps/android-maps-utils#299.

@codingblazer codingblazer mentioned this pull request Nov 26, 2016
Closed
@barbeau barbeau force-pushed the travisAbi branch 2 times, most recently from 97ca7d1 to 5529ad1 Compare December 13, 2016 14:21
@barbeau
Copy link
Member Author

barbeau commented Dec 13, 2016

For this PR, to get Travis fully working I'm also going to comment out the unit test for Open311 support and HART, as they haven't gone to production yet and this test will fail until then. When HART goes into production, then we can uncomment the unit test.

barbeau added a commit that referenced this pull request Dec 13, 2016
@barbeau
Comment out HART Open311 unit test until HART goes to production
52ca44a 
See #721 (comment)
@barbeau barbeau changed the title WIP - Fix #720 - Fix ABI etc for Travis builds Fix #720 - Fix ABI etc for Travis builds Dec 13, 2016
@barbeau
Copy link
Member Author

barbeau commented Dec 13, 2016

Grrrr....now we're back to travis-ci/travis-ci#2634 (comment) again...

@barbeau
Copy link
Member Author

barbeau commented Jan 23, 2017

Given that Google still doesn't seem to have resolved the issues with API level 23-25 emulators that prevent them from running on Travis, I'm pushing a temporary workaround to the master branch in 03e262d that builds the projects on Travis, but does not run the unit tests. Builds don't require the emulator, only the unit tests, so this at least gives us the ability to see if builds themselves succeed.

@barbeau barbeau mentioned this pull request Mar 3, 2017
Closed
@barbeau
Overhaul Travis script
76612e2 
* Based on #720 (comment), and solution at https://github.com/albodelu/android-maps-utils/blob/8ab3efe962cfe59da6db748c176108d7e142e4e7/.travis.yml.

Change gradlew command to match previous

Don't fail builds for Lint errors
@barbeau barbeau mentioned this pull request Jul 20, 2018
Closed
@barbeau barbeau mentioned this pull request Jul 11, 2019
Merged
@barbeau
Copy link
Member Author

barbeau commented Jul 11, 2019

A lot has changed on Travis since this PR, so I'm closing it in favor of a new approach in #997.

@barbeau barbeau closed this Jul 11, 2019
@aaronbrethorst aaronbrethorst deleted the travisAbi branch January 26, 2024 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@barbeau