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

Feature/v0.17.0 - AWSF Dev Days #59

Merged
merged 5 commits into from
Sep 25, 2024

Conversation

vade
Copy link
Contributor

@vade vade commented Sep 25, 2024

Link the Issue(s) this Pull Request is related to.

Fixes #58

Summarize your change.

This PR updates the Swift Bindings package to support OTIO v0.17.0

For Reviewers: Please note that this change introduces some minor changes to support OTIO's migration from otio::any to std::any as per similar changes in OpenTimelineIO/raven#46

To support std::any we also needed to update the Swift Package version to 5.5 (perhaps 5.4 is the minimal?) and update the cxx version, as well as change some search paths in the Package Definition.

We also had some changes to the Obj-C / C++ bridging code which references otio:any , as well as otio::optional and otio::nullopt to their std counterparts.

We also remove the any and optional lite package libraries, shim code, and references since we no longer need them

We presume things are working as swift test passes completely. We include no new tests as we aren't changing functionality on the exposed swift or otio api surface area.

Thank you!

Signed-off-by: vade <vade@vade.info>
Signed-off-by: vade <vade@vade.info>
Signed-off-by: vade <vade@vade.info>
Signed-off-by: vade <vade@vade.info>
Copy link
Member

@reinecke reinecke left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@reinecke reinecke merged commit 963be93 into OpenTimelineIO:main Sep 25, 2024
3 checks passed
@vade vade deleted the feature/v0.17.0 branch September 25, 2024 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants