Skip to content

Conversation

ongyx
Copy link

@ongyx ongyx commented Jun 12, 2021

This PR fixes building piskel (using grunt desktop).

When building normally, grunt will show

>> TypeError: os.tmpDir is not a function

which causes the tasks defined after to not be registered:

Warning: Task "nwjs:windows" not found.

This is caused by outdated dependencies relying on os.tempDir, which was deprecated in Node 7.
To fix this, just run patch_tempdir.sh at the root of this repo every time after you run npm install.
Basically, it recurses through node_modules and replaces os.tempDir with os.tempdir.

* package.json listed grunt ^0.4.5, which conflicts with grunt-nw-builder (requires grunt ^1.0.1).
* Node versions 7 and above deprecated os.tmpDir, which is used in several dependencies such as temp (outdated deps?)
  To patch this, run 'npm install' as usual, then execute the shell script 'patch_tmpdir.sh' in the root of this repo.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant