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

Code cache breaks HMR #1554

Closed
darind opened this issue Dec 12, 2019 · 0 comments
Closed

Code cache breaks HMR #1554

darind opened this issue Dec 12, 2019 · 0 comments
Assignees
Labels
Milestone

Comments

@darind
Copy link
Collaborator

darind commented Dec 12, 2019

Environment
Provide version numbers for the following components (information can be retrieved by running tns info in your project folder or by inspecting the package.json of the project):

  • CLI: 6.2.2
  • Cross-platform modules: 6.2.3
  • Android Runtime: 6.2.0

Describe the bug

HMR does not work if code cache is enabled

To Reproduce

  1. Enable code cache in app/package.json:
{
  "main": "app.js",
  "android": {
    "v8Flags": "--nolazy --expose_gc",
    "markingMode": "none",
    "codeCache": "true"
  }
}
  1. Run the app: tns run android

  2. Make changes to some of the files and ensure that those changes are applied

  3. Stop the CLI with Ctrl+C

  4. Run the app again without making any changes: tns run android

  5. Make changes to some of the files

Expected behavior

The new changes are applied

Actual behavior

The new changes are never applied on the device even if the CLI reports that those changes have been sent and the app was refreshed.

@darind darind added the bug label Dec 12, 2019
@darind darind self-assigned this Dec 12, 2019
vmutafov added a commit that referenced this issue Dec 30, 2019
* bumping version

* fix(workers): Dispose the isolate when terminating workers (#1550)

* fix: Ensure js timestamp is smaller than corresponding code cache file timestamp (#1554)

* Fix Kotlin enum accessing issue (#1559)

* Update package.json version to 6.3.1

* Update CHANGELOG.md

Co-authored-by: Vasil Trifonov <v.trifonov@gmail.com>
Co-authored-by: Darin Dimitrov <darin.dimitrov@gmail.com>
@vmutafov vmutafov added this to the 6.3.1 milestone Jan 2, 2020
@vmutafov vmutafov closed this as completed Jan 2, 2020
vmutafov added a commit that referenced this issue Jan 31, 2020
* fix(workers): Dispose the isolate when terminating workers (#1550)

* Prepare 6.3.1 patch (#1561)

* bumping version

* fix(workers): Dispose the isolate when terminating workers (#1550)

* fix: Ensure js timestamp is smaller than corresponding code cache file timestamp (#1554)

* Fix Kotlin enum accessing issue (#1559)

* Update package.json version to 6.3.1

* Update CHANGELOG.md

Co-authored-by: Vasil Trifonov <v.trifonov@gmail.com>
Co-authored-by: Darin Dimitrov <darin.dimitrov@gmail.com>

Co-authored-by: Darin Dimitrov <darin.dimitrov@gmail.com>
Co-authored-by: Vasil Trifonov <v.trifonov@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants