Regression from PR #47458: Cannot install React Native from Git anymore
Personne n'a encore pris cette issue.
- Langage dominant
- C++
- Étoiles
- 127k
- Forks
- 25.3k
- Merge moyen
- 1 j 23 h
- PR mergées (30 j)
- 4
Description
Description
This PR updates shouldSkipGenerationForRncore as such it checks if .../ReactCommon exists before deciding to skip rncore generation, whereas previously it would check .../ReactCommon/react/renderer/components/rncore specifically.
This change breaks installations from Git.
I noticed this while upgrading from 0.76.2 to 0.78.1.
Per my debugging:
- Previous behavior:
- When installing from NPM:
rncoreis pre-generated using theprepackscript - When installing from Git:
rncoregets generated when runningpod install
- When installing from NPM:
- Current behavior:
- When installing from NPM:
rncoreis pre-generated using theprepackscript - When installing from Git:
rncorewill NOT get generated when runningpod installbecause.../ReactCommonis detected as exists even though.../ReactCommon/react/renderer/components/rncoredoes not
- When installing from NPM:
Steps to reproduce
npx @react-native-community/cli@latest init BuggyReactNative
cd ./BuggyReactNative/
npm install 'https://gitpkg.vercel.app/facebook/react-native/packages/react-native?0.78-stable'
cd ./ios/
bundle exec pod install
cd ..
ls ./node_modules/react-native/ReactCommon/react/renderer/components/rncore
# 💥 no such file or directory
# 💥 Xcode won't build the project
React Native Version
0.78.1
Affected Platforms
Runtime - iOS, Other (please specify)
Output of npx @react-native-community/cli info
Not relevant
Stacktrace or Logs
Not relevant
Reproducer
Run the reproducer bash script so you can test with your toolchain not mine
Screenshots and Videos
No response
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez dans packages/react-native/scripts/codegen/generate-artifacts-executor.js, au niveau de shouldSkipGenerationForRncore, et comparez le comportement antérieur référencé avec la vérification actuelle de ReactCommon. Utilisez le reproducer de l’installation via Git de l’issue et bundle exec pod install pour vérifier le chemin de génération. C’est terminé lorsque le répertoire rncore existe après l’installation et que le projet iOS peut être compilé.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, react-native
- Domaine
- build-system, mobile
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- Calme
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 65/100