Fix Symbol not found: (_JSGlobalContextSetInspectable) #44185
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
facebook-github-bot
added
the
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D56425834 |
Summary: This change will fix a symbol not found for JSC Runtime. The `if` check was not a compile time check, therefore the symbol ended up in the binary even if it is not available. Following this post on [Apple forum](https://forums.developer.apple.com/forums/thread/749534), this changes should do the trick. ## Changelog [iOS][Fixed] - Fix Symbol not found: (_JSGlobalContextSetInspectable) Differential Revision: D56425834
cipolleschi
force-pushed
the
export-D56425834
branch
from
fe44f60
to
5fb18fb
Compare
|
This pull request was exported from Phabricator. Differential Revision: D56425834 |
Base commit: f453d7a |
|
This pull request has been merged in 224ac21. |
|
This pull request was successfully merged by @cipolleschi in 224ac21. When will my fix make it into a release? | How to file a pick request? |
Saadnajmi
reviewed
Apr 23, 2024
Comment on lines
379
to
+380
| #ifdef _JSC_HAS_INSPECTABLE | ||
| #if (__OSX_AVAILABLE_STARTING(MAC_NA, IPHONE_16_4)) |
There was a problem hiding this comment.
@cipolleschi Original author of _JSC_HAS_INSPECTABLE, should we move this check to the macros' definition (AKA, line 303)? I can do so in my already open PR where I refactor some of it #39549
There was a problem hiding this comment.
Hi @Saadnajmi, yes, we can. I just saw we had a check on the iOS version at line 303, and I think that that's what triggered the issue... The version is set to 160400, which is... 160.4!!!
I think it should be 16400 if we want to specify 16.4.
- #if __IPHONE_OS_VERSION_MAX_ALLOWED >= 160400
+ #if __IPHONE_OS_VERSION_MAX_ALLOWED >= 16400for example, the check for iOS 17, is 17000.
There was a problem hiding this comment.
Damn. I remember double checking that. Ok, I guess we don't need the double check then, and I'll fix the versions.
There was a problem hiding this comment.
Also closing this thread so we can keep discussion in #39549 (comment) , but as noted there, I think the existing macro is correct.
EDIT: I can't actually resolve this thread.. so.. just go comment on the other one :D
hurali97
pushed a commit
that referenced
this pull request
Apr 29, 2024
Summary: Pull Request resolved: #44185 This change will fix a symbol not found for JSC Runtime. The `if` check was not a compile time check, therefore the symbol ended up in the binary even if it is not available. Following this post on [Apple forum](https://forums.developer.apple.com/forums/thread/749534), this changes should do the trick. ## Changelog [iOS][Fixed] - Fix Symbol not found: (_JSGlobalContextSetInspectable) Reviewed By: hash3r Differential Revision: D56425834 fbshipit-source-id: a37af51b078bd47a938e6b65d9d8e0f7506e746f
Titozzz
pushed a commit
that referenced
this pull request
Apr 29, 2024
Summary: Pull Request resolved: #44185 This change will fix a symbol not found for JSC Runtime. The `if` check was not a compile time check, therefore the symbol ended up in the binary even if it is not available. Following this post on [Apple forum](https://forums.developer.apple.com/forums/thread/749534), this changes should do the trick. ## Changelog [iOS][Fixed] - Fix Symbol not found: (_JSGlobalContextSetInspectable) Reviewed By: hash3r Differential Revision: D56425834 fbshipit-source-id: a37af51b078bd47a938e6b65d9d8e0f7506e746f
cipolleschi
added a commit
that referenced
this pull request
May 1, 2024
Summary: Pull Request resolved: #44185 This change will fix a symbol not found for JSC Runtime. The `if` check was not a compile time check, therefore the symbol ended up in the binary even if it is not available. Following this post on [Apple forum](https://forums.developer.apple.com/forums/thread/749534), this changes should do the trick. ## Changelog [iOS][Fixed] - Fix Symbol not found: (_JSGlobalContextSetInspectable) Reviewed By: hash3r Differential Revision: D56425834 fbshipit-source-id: a37af51b078bd47a938e6b65d9d8e0f7506e746f
kosmydel
pushed a commit
to kosmydel/react-native
that referenced
this pull request
May 6, 2024
Summary: Pull Request resolved: facebook#44185 This change will fix a symbol not found for JSC Runtime. The `if` check was not a compile time check, therefore the symbol ended up in the binary even if it is not available. Following this post on [Apple forum](https://forums.developer.apple.com/forums/thread/749534), this changes should do the trick. ## Changelog [iOS][Fixed] - Fix Symbol not found: (_JSGlobalContextSetInspectable) Reviewed By: hash3r Differential Revision: D56425834 fbshipit-source-id: a37af51b078bd47a938e6b65d9d8e0f7506e746f
Saadnajmi
added a commit
to microsoft/react-native-macos
that referenced
this pull request
May 21, 2024
## Summary: Merge up to RN Core's `0.73.8` release, with one notable change: - Instead of picking up facebook#44185 , let's instead port facebook#39549 to React Native macOS, so that we have direct debugging with JSC working. ## Test Plan: CI should pass.
kosmydel
pushed a commit
to kosmydel/react-native
that referenced
this pull request
Jun 11, 2024
Summary: Pull Request resolved: facebook#44185 This change will fix a symbol not found for JSC Runtime. The `if` check was not a compile time check, therefore the symbol ended up in the binary even if it is not available. Following this post on [Apple forum](https://forums.developer.apple.com/forums/thread/749534), this changes should do the trick. ## Changelog [iOS][Fixed] - Fix Symbol not found: (_JSGlobalContextSetInspectable) Reviewed By: hash3r Differential Revision: D56425834 fbshipit-source-id: a37af51b078bd47a938e6b65d9d8e0f7506e746f
This was referenced Jun 28, 2024
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
This change will fix a symbol not found for JSC Runtime.
The
ifcheck was not a compile time check, therefore the symbol ended up in the binary even if it is not available.Following this post on Apple forum, this changes should do the trick.
Changelog
[iOS][Fixed] - Fix Symbol not found: (_JSGlobalContextSetInspectable)
Differential Revision: D56425834