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

Doctor could check for DEAD_CODE_STRIPPING=NO and warn users of possible build failures #1572

Closed
mikehardy opened this issue Mar 13, 2022 · 4 comments
Labels
command: doctor feature request no-stale-bot This issue cannot be marked as stale by stale bot

Comments

@mikehardy
Copy link
Contributor

Describe the Feature

Flipper and hermes when enabled on iOS can apparently cause a duplicate symbol problem for current versions in use by react-native.

Not sure what the underlying issue is, if it is fixed, and if it is fixed what version ranges the warning should be restricted to, but the basic test is:

  • scan pbxproj.proj and see if DEAD_CODE_STRIPPING is set to NO, if it is, warn the user of

Possible Implementations

Unsure, not sure if doctor is looking inside iOS projects yet, a simple text scan of the file seems sufficient though

Related Issues

facebook/react-native#32016
This was also noted here facebook/flipper#3117 (comment)

@mikehardy
Copy link
Contributor Author

This one is definitely needed for catalyst builds,

  • if SUPPORTS_MACCATALYST is YES
  • and use_flipper is on in the podfile
  • then DEAD_CODE_STRIPPING must be YES or your build will break

@github-actions
Copy link

There hasn't been any activity on this issue in the past 3 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 7 days.

@github-actions github-actions bot added the stale label Nov 26, 2022
@github-actions github-actions bot closed this as completed Dec 4, 2022
@thymikee thymikee reopened this Dec 5, 2022
@github-actions github-actions bot removed the stale label Dec 6, 2022
@TMisiukiewicz TMisiukiewicz added the no-stale-bot This issue cannot be marked as stale by stale bot label Dec 6, 2022
@szymonrybczak
Copy link
Collaborator

hey @mikehardy do you still this issue is still valid? happy to implement it 😅

@mikehardy
Copy link
Contributor Author

Hey @szymonrybczak 👋 well...flipper is out right? so I'm not sure this has value as it was a collision between the same symbol in flipper and hermes, but if only one of them is ever in play, I don't think this can happen

I'm going to close it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
command: doctor feature request no-stale-bot This issue cannot be marked as stale by stale bot
Projects
None yet
Development

No branches or pull requests

4 participants