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

[macOS] Remove deprecated altool notarization support, disable rcodesign for C# version. #83482

Merged
merged 1 commit into from
Nov 3, 2023

Conversation

bruvzg
Copy link
Member

@bruvzg bruvzg commented Oct 17, 2023

  • Removes support for deprecated altool notarization.
  • Disables rcodesign support for C# version and adds an error message if GDExtension libraries are found during export, since rcodesign is not capable of signing bundles with embedded libraries.

@akien-mga akien-mga modified the milestones: 4.x, 4.2 Oct 17, 2023
Copy link
Member

@raulsntos raulsntos left a comment

Choose a reason for hiding this comment

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

What does it mean to disable rcodesign for C#? Is that going to cause issues for exported projects? Should there be a warning in this case too?

Also, we should support exporting non-C# projects from the C# version of the editor. What does this change mean for this case?

platform/macos/export/export_plugin.cpp Outdated Show resolved Hide resolved
@bruvzg bruvzg marked this pull request as ready for review November 2, 2023 08:19
@bruvzg bruvzg requested a review from a team as a code owner November 2, 2023 08:19
@YuriSizov YuriSizov merged commit 64d20e0 into godotengine:master Nov 3, 2023
15 checks passed
@YuriSizov
Copy link
Contributor

Thanks!

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.

4 participants