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

core-graphics: non-public API usage _CGContextSetFontSmoothingStyle #707

Closed
lucasfernog-crabnebula opened this issue Aug 30, 2024 · 10 comments

Comments

@lucasfernog-crabnebula
Copy link
Contributor

I'm trying to submit an iOS app but Apple is rejecting it:

ITMS-90338: Non-public API usage - The app references non-public symbols in <appName>: _CGContextSetFontSmoothingStyle. If method names in your source code match the private Apple APIs listed above, altering your method names will help prevent this app from being flagged in future submissions. In addition, note that one or more of the above APIs may be located in a static library that was included with your app. If so, they must be removed. For further information, visit the Technical Support Information at http://developer.apple.com/support/technical/

Even though my dependencies do not use the set_font_smoothing_style API, seems like Rust cannot strip it.

Similar to #656

I thought about deleting the API similar to #700 but seems like at least alacritty uses it. Maybe it makes sense to use a feature flag instead?

@waywardmonkeys
Copy link
Collaborator

Are you sure that alacrity is using it and if so, through core_graphics?

@lucasfernog-crabnebula
Copy link
Contributor Author

oh nvm they have their own bindings it seems https://github.com/search?q=set_font_smoothing_style&type=code

@lucasfernog-crabnebula
Copy link
Contributor Author

I guess we could follow that other PR and remove the API instead..

@waywardmonkeys
Copy link
Collaborator

And those are old forks… pretty sure the code is gone from current alacritty.

@waywardmonkeys
Copy link
Collaborator

I guess we could follow that other PR and remove the API instead..

I think I prefer this. If someone needs it, they can do a local binding… and GitHub search doesn’t seem to turn up users.

lucasfernog-crabnebula added a commit to crabnebula-dev/core-foundation-rs that referenced this issue Aug 30, 2024
The `CGContextSetFontSmoothingStyle` API is private, and Apple rejects it for the App Store.

See servo#707 for more information.
@lucasfernog-crabnebula
Copy link
Contributor Author

Alright I've opened a new PR.. let me know if there's anything else I can do.. this makes it impossible to ship any app to App Store :( and if this is released as 0.25 (it should right?) then it'll take some time to propagate this change to the ecosystem.

@jdm
Copy link
Member

jdm commented Aug 30, 2024

You should be able to rely on a fork with the 0.24 version and a cargo patch to override other copies in the ecosystem until that's complete.

@lucasfernog
Copy link

that's what i'm doing for now - but i need to instruct Tauri users to do the same

@lucasfernog
Copy link

thanks for the feedback and quick response btw!

github-merge-queue bot pushed a commit that referenced this issue Aug 30, 2024
The `CGContextSetFontSmoothingStyle` API is private, and Apple rejects it for the App Store.

See #707 for more information.
@atouchet
Copy link
Contributor

atouchet commented Sep 1, 2024

Done in #709.

@atouchet atouchet closed this as completed Sep 1, 2024
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

No branches or pull requests

5 participants