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

Bsneed/version fixes #966

Merged
merged 8 commits into from
Nov 12, 2020
Merged

Bsneed/version fixes #966

merged 8 commits into from
Nov 12, 2020

Conversation

migs647
Copy link
Contributor

@migs647 migs647 commented Nov 11, 2020

What does this PR do?
Fixes a number of inconsistencies with supported versions

Where should the reviewer start?
Test every platform and every version starting with Mac OS X 10.0

How should this be manually tested?
See above

Any background context you want to provide?
Cocoapods had issues when Analytics was linked as a 64bit library (iOS 11) with a 32 library (iOS 10). Mac OS would not work under building from source unless it was 10.13 for some, but would for swift package manager.

What are the relevant tickets?
https://segment.atlassian.net/browse/LIBMOBILE-211
https://segment.atlassian.net/browse/LIBMOBILE-214
https://segment.atlassian.net/browse/LIBMOBILE-212
https://segment.atlassian.net/browse/LIBMOBILE-222

Screenshots or screencasts (if UI/UX change)

Questions:

  • Does the docs need an update?
  • Are there any security concerns?
  • Do we need to update engineering / success?

@migs647 migs647 force-pushed the bsneed/version-fixes branch from 679a693 to 7c605f8 Compare November 11, 2020 18:01
@migs647 migs647 merged commit 9cab023 into master Nov 12, 2020
@migs647 migs647 deleted the bsneed/version-fixes branch November 12, 2020 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants