Uh oh!
There was an error while loading. Please reload this page.
RNGP - Properly set the jsRootDir default value - #35992
Closed
cortinico wants to merge 1 commit into
Closed
Conversation
Summary: Fixessoftware-mansion/react-native-gesture-handler#2382 I've just realized that the default value fo `jsRootDir` is not entirely correct. That's the root of the folder where the codegen should run. For apps, it should be defaulted to `root` (i.e. ../../) For libraries, it should be defaulted to `../` (currently is root). This causes a problem where libraries without either a `codegenConfig` or a `react{ jsRootDir = ... }` specified in the build.gradle will be invoking the codegen and generating duplicated symbols. Changelog: [Android] [Fixed] - RNGP - Properly set the `jsRootDir` default value Differential Revision: D42806411 fbshipit-source-id: 44371edec593aba5a5115d05e6f8d2e86ec6dc11
facebook-github-bot
commented
Jan 27, 2023
Contributor
This pull request was exported from Phabricator. Differential Revision: D42806411 |
analysis-bot
commented
Jan 27, 2023
Base commit: a00cea4 |
facebook-github-bot
commented
Jan 30, 2023
Contributor
This pull request has been merged in c000409. |
kelset pushed a commit
that referenced
this pull request
Jan 30, 2023
Summary: Pull Request resolved: #35992Fixessoftware-mansion/react-native-gesture-handler#2382 I've just realized that the default value fo `jsRootDir` is not entirely correct. That's the root of the folder where the codegen should run. For apps, it should be defaulted to `root` (i.e. ../../) For libraries, it should be defaulted to `../` (currently is root). This causes a problem where libraries without either a `codegenConfig` or a `react{ jsRootDir = ... }` specified in the build.gradle will be invoking the codegen and generating duplicated symbols. Changelog: [Android] [Fixed] - RNGP - Properly set the `jsRootDir` default value Reviewed By: cipolleschi Differential Revision: D42806411 fbshipit-source-id: ffe45f9684a22494cc2e4d0a19de9077cb341365
hameezAhmad
commented
Apr 4, 2023
I am getting this Execution failed for task ':app:mergeLibDexDebug'. in react native 0.71.5 |
andrepporto
commented
Apr 25, 2023
Same thing. Execution failed for task ':app:mergeLibDexDebug'. in react native 0.71.5 |
cortinico
commented
Apr 25, 2023
ContributorAuthor
@andrepporto@hameezAhmad Please open a separate issue. Also it's unclear why it's related to this PR. |
This file contains hidden or 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
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
6 participants
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:
Fixessoftware-mansion/react-native-gesture-handler#2382
I've just realized that the default value fo
jsRootDiris not entirely correct.That's the root of the folder where the codegen should run.
For apps, it should be defaulted to
root(i.e. ../../)For libraries, it should be defaulted to
../(currently is root).This causes a problem where libraries without either a
codegenConfigor areact{ jsRootDir = ... }specified in the build.gradle will be invoking the codegen and generating duplicated symbols.
Changelog:
[Android] [Fixed] - RNGP - Properly set the
jsRootDirdefault valueDifferential Revision: D42806411