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

Add SmartLink=false for macos target #1760

Closed
wants to merge 1 commit into from

Conversation

DmitriyKirakosyan
Copy link
Contributor

@DmitriyKirakosyan DmitriyKirakosyan commented Oct 20, 2023

  • Has CHANGELOG.md been updated?
  • Are tests passing locally?
  • Are the files formatted correctly?
  • Did you add unit tests if this modifies the Windows code?
  • Did you test your change with either the sample apps that are included in the repository or with a blank app that uses your change?

Description

The SmartLink properties have been removed in this GitHub pull request. However, it seems to be causing issues with macOS app validation for the App Store.

The previous pull request was not confirmed to have fixed any actual issues; it was based on the assumption that in some cases, the iOS app might not be able to compile due to the SmartLink flag being set to false. The change in the macOS app was made for consistency, so reverting this change should not cause any issues.

Related PRs or issues

AB#103183
#1753

Using default SmartLink (true) causes a rejection of a macos app,
on submission to App Store
@DmitriyKirakosyan DmitriyKirakosyan requested a review from a team as a code owner October 20, 2023 06:47
@DmitriyKirakosyan DmitriyKirakosyan deleted the fix/add-smartlink-for-macos branch March 18, 2024 11:54
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.

1 participant