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.

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

Turf v2.0.0-alpha.2 #517

Merged
merged 3 commits into from
Feb 18, 2021
Merged

Turf v2.0.0-alpha.2 #517

merged 3 commits into from
Feb 18, 2021

Conversation

1ec5
Copy link
Contributor

@1ec5 1ec5 commented Feb 9, 2021

This PR requires Turf v2.0.0-alpha.1 v2.0.0-alpha.2 as a stopgap to unblock mapbox/mapbox-navigation-ios#2629 and mapbox/mapbox-navigation-ios#2794. This PR will be merged to the release-v2.0 branch, which will not be merged into the main branch until we eliminate the need for a binary Turf distribution (mapbox/turf-swift#133) when using SPM.

/cc @mapbox/navigation-ios @macdrevx

@1ec5 1ec5 added the build label Feb 9, 2021
@1ec5 1ec5 added this to the v2.0.0 milestone Feb 9, 2021
Copy link
Contributor Author

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v2.0.0-alpha.2 has been tagged. We need to upgrade to that tag, which has the correct build flags. We should also update the Cartfile and podspecs for consistency across dependency managers, especially since v2.0.0-alpha.2 has some unrelated type changes that haven’t been released as part of v1.3.0 yet.

@1ec5 1ec5 requested review from MaximAlien and chezzdev February 9, 2021 23:40
@1ec5
Copy link
Contributor Author

1ec5 commented Feb 9, 2021

Linux builds are failing:

error: artifact of binary target 'Turf' failed extraction: The operation could not be completed. (TSCBasic.Process.Error error 0.)

Per mapbox/turf-swift#133 (comment), we’ll need to either exclude tvOS, watchOS, and Linux from this branch of MapboxDirections or conditionally have those platforms depend on a different tag of MapboxDirections.

@zugaldia
Copy link
Member

we’ll need to either exclude tvOS, watchOS, and Linux

Sounds like a reasonable trade-off for the Public Preview, and one that we should reconsider before GA.

@1ec5 1ec5 merged commit 0351a3b into release-v2.0 Feb 18, 2021
@1ec5 1ec5 deleted the binary-turf branch February 18, 2021 20:08
@1ec5 1ec5 changed the title Turf v2.0.0-alpha.1 Turf v2.0.0-alpha.2 Feb 18, 2021
@1ec5 1ec5 restored the binary-turf branch February 18, 2021 22:58
@1ec5 1ec5 mentioned this pull request Mar 3, 2021
@truburt truburt modified the milestones: v2.0.0, v2.0.0 (Public Preview) Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants