[TIMOB-25944] CLI: Block-scoped declaration error when using Node 4

GitHub Issue	n/a
Type	Bug
Priority	None
Status	Closed
Resolution	Won't Fix
Resolution Date	2018-10-18T09:46:25.000+0000
Affected Version/s	Release 7.1.1
Fix Version/s	n/a
Components	CLI
Labels	regression
Reporter	Ewan Harris
Assignee	Ewan Harris
Created	2018-04-06T18:56:39.000+0000
Updated	2018-10-18T13:04:25.000+0000

Description

Comments

Ewan Harris 2018-04-06

PR for node-appc: https://github.com/appcelerator/node-appc/pull/129 Will need titanium_mobile PRs after merge
Hans Knöchel 2018-04-06

Thought we require Node 8.x LTS since SDK 7?
Ewan Harris 2018-04-06

Appc CLI does, and we recommend it. But Titanium itself supports everything from 4 and up (and works on 7.1.0.GA)

Samir Mohammed 2018-09-17

When using node 4.6.1 and using ti clean the following error was shown:

   Axway:~ Samir$ ti clean
   SyntaxError: Unexpected token ...
       at exports.runInThisContext (vm.js:53:16)
       at Module._compile (module.js:373:25)
       at Object.Module._extensions..js (module.js:416:10)
       at Module.load (module.js:343:32)
       at Function.Module._load (module.js:300:12)
       at Module.require (module.js:353:17)
       at require (internal/module.js:12:17)
       at Object.<anonymous> (/usr/local/lib/node_modules/titanium/node_modules/node-appc/node_modules/fs-extra/lib/index.js:6:3)
       at Module._compile (module.js:409:26)
       at Object.Module._extensions..js (module.js:416:10)
   ---------------------------------------------
       at Object.exports.execFile (child_process.js:296:9)
       at exports.exec (child_process.js:111:18)
       at detectLocale (/usr/local/lib/node_modules/titanium/lib/titanium.js:69:4)
       at /usr/local/lib/node_modules/titanium/lib/titanium.js:84:2
       at Object.<anonymous> (/usr/local/lib/node_modules/titanium/lib/titanium.js:85:2)
       at Module._compile (module.js:409:26)
       at Object.Module._extensions..js (module.js:416:10)
       at Module.load (module.js:343:32)
       at Function.Module._load (module.js:300:12)

Ewan Harris 2018-10-18

Closing as wont fix, we'll be going to 6 as a minimum https://github.com/appcelerator/titanium_mobile/pull/10382/commits/2c0cd4f83673ea8fd3a9454d92167cb34faf4a44
Christopher Williams 2018-10-18

Note that while this is a major version change in terms of supported Node versions, Nodejs v4.x was EOL'd back at the end of April this year. It doesn't make sense to make efforts to continue to support an older EOL'd version of Node anymore (especially since the CLI requires Node 8.10 as it's minimum now!)

JSON Source