Skip to content

Commit

Permalink
Fix react-native-git-upgrade cache busting
Browse files Browse the repository at this point in the history
Summary:
**Motivation**

This PR fixes #12420.

For the first time, the version 0.41.2 includes a change in the generator (see #12162). The [require cache busting](https://github.com/facebook/react-native/blob/master/react-native-git-upgrade/cliEntry.js#L157-L163) of `react-native-git-upgrade`, designed to face this situation , was ineffective.

The entry in `require.cache` is the file path including the `.js` extension. We have to delete this entry with the same key.

**Test plan**

- Publish `react-native-git-upgrade` to Sinopia,
- Follow the reproduction steps of #12420
- 👉 The name of the app shouldn't be changed
Closes #12422

Differential Revision: D4585549

Pulled By: mkonicek

fbshipit-source-id: 508ac925c17d02b7739d47f9351a5aa336589f2e
  • Loading branch information
ncuillery authored and facebook-github-bot committed Feb 19, 2017
1 parent 8c0e6ec commit 927b356
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion react-native-git-upgrade/cliEntry.js
Expand Up @@ -160,7 +160,7 @@ function runCopyAndReplace(generatorDir, appName) {
* This file could have changed between these 2 versions. When generating the new template,
* we don't want to load the old version of the generator from the cache
*/
delete require.cache[copyProjectTemplateAndReplacePath];
delete require.cache[require.resolve(copyProjectTemplateAndReplacePath)];
const copyProjectTemplateAndReplace = require(copyProjectTemplateAndReplacePath);
copyProjectTemplateAndReplace(
path.resolve(generatorDir, '..', 'templates', 'HelloWorld'),
Expand Down
2 changes: 1 addition & 1 deletion react-native-git-upgrade/package.json
@@ -1,6 +1,6 @@
{
"name": "react-native-git-upgrade",
"version": "0.2.6",
"version": "0.2.7",
"license": "BSD-3-Clause",
"description": "The React Native upgrade tool",
"main": "cli.js",
Expand Down

0 comments on commit 927b356

Please sign in to comment.