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

Work around DirectX libraries blocking iOS from building #5592

Merged
merged 4 commits into from
Dec 19, 2022

Conversation

frenzibyte
Copy link
Member

Attempt number 2, this time with not too extreme changes (see #5590 for previous approach).

Vortice.Direct3D11 and other DirectX libraries rely on SharpGen.Runtime for bindings generation, which makes it referenced during framework building stage, but fail on iOS during AOT compilation for unknown reasons.

I've went ahead and compiled empty assemblies for the faulty libraries and set up a target to copy the assemblies to the linking folder, while expecting it to fail due to the assemblies being... well, empty. But luckily enough, it passed AOT stage and finished building successfully.

Now consumers don't have to carry around any workarounds we add to our
libraries, as they're automatically applied during build using the
`osu.Framework.iOS.Workarounds.targets` file.
Copy link
Member

@peppy peppy left a comment

Choose a reason for hiding this comment

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

Seems much more palatable.

@peppy peppy requested a review from smoogipoo December 18, 2022 05:17
@frenzibyte frenzibyte self-assigned this Dec 18, 2022
Copy link
Contributor

@smoogipoo smoogipoo left a comment

Choose a reason for hiding this comment

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

I can't really test this, but it looks fine

@smoogipoo smoogipoo merged commit d264eb6 into ppy:master Dec 19, 2022
@frenzibyte frenzibyte deleted the fix-ios-2 branch December 19, 2022 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants