-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Conversation
@1ec5 It looks OK. But only way to know for sure is on Travis. |
Not looking good on the Android side, @ljbade. Looks like toolchain.sh just stalls out. |
Hmm, OK I'll have to try it locally. |
Strange it doesn't hang locally but I get a bunch of weird messages:
|
OK got a compare link mapbox/mason@abdc131...41d8b2e |
Only change that looks interesting is mapbox/mason@abdc131...41d8b2e#diff-cc7a13abf233cbd4fc6762059aee5946R207 I think I need to clone mason on this computer to the right place. |
Hrm still does it. @springmeyer What do you think has broken? |
Per scrum, we’d like to feature iOS 9 SDK compatibility in the next release of GL for iOS. If we don’t figure the Android build failures by then, we’ll cut a branch based on the current submodule pin and cherry-pick just mapbox/mason@41d8b2e. |
@1ec5 Yeah I'd do that while we figure this out. |
Is this fixed in current master? |
Pulled in mason/mason@41d8b2eb28b3cd942d9f15583467b27c549675ea for mapbox/mason#106 and mason/mason@d8597d944533dbb5e28d08dfdba22a82c93171b7 for mapbox/mason#110. Fixes #1957 and fixes #1707.
@kkaefer, yes, that fixed it – thanks. I’ve also pulled in mason/mason@d8597d9 for mapbox/mason#110 to fix #1707. |
This PR pulls in mapbox/mason@41d8b2e for mapbox/mason#106, as well as plenty of other changes. @kkaefer @ljbade, is it safe to pull in all the Mason changes since mapbox/mason@abdc131?
Fixes #1957; fixes #1707.