NodeJS v5 upgrade (from v4)

[danielweck]

Not an issue with Readium per-se, but some notes about the transition from NodeJS v4 to v5

https://nodejs.org

• v4 LTS Long Term Support (mature and dependable)
• v5 Stable (latest features)

On my Windows 10 machine, I used the MSI installer to setup v4 (I'd rather avoid upgrading to v5 for now).
On my OSX machine, I used the package installer to setup v4, but I wanted to upgrade to v5. Simply installing the v5 package over v4 did not work (failure to execute even the simplest NPM build command). So, I did the following and it all worked fine thereafter:

• sudo rm -rf /usr/local/lib/node_modules
• install NodeJS v5 OSX package
• cd readium-js-viewer + rm -r node_modules
• cd readium-js + rm -r node_modules
• cd readium-shared-js + rm -r node_modules
• cd readium-cfi-js + rm -r node_modules
• navigate back inside readium-js-viewer top-level folder, and invoke npm run prepare
• npm run dist+sourcemap

Success.

[danielweck]

Mhmm, I had to invoke npm run prepare twice in order for npm run http to succeed (the opener Node module was missing in the node_modules folder). I then realized that invoking plain npm run install did not install zip-js in readium-js, in fact it removed it! Subsequently running npm run update would fix that, but sometimes running the NPM update command several times in a row would mess-up the node_modules folder again!! I know that dependency management has changed in NodeJS v5 (new algorithm that attempts to avoid deep nesting of redundant shared dependencies). So perhaps this is the cause of the problem (this does not fill me with confidence regarding v5 though!). The package.json dependencies seem pretty innocuous:
https://github.com/readium/readium-js/blob/develop/package.json#L46

  "dependencies": {
    "parallelshell": "latest",
    "opener": "latest",
    "http-server": "danielweck/http-server"
  },
  "devDependencies": {
    "crypto-js": "latest",
    "zipjs": "danielweck/zip.js",
    "requirejs-text": "latest",
    "gift": "latest"
  },

Any idea?

[danielweck]

NPM v3 (which Node 5 ships with) is "BETA SOFTWARE" (quote):
https://github.com/npm/npm/releases/tag/v3.0.0