Just FYI -- this commit breaks the archlinux git package for tilemill. I don't know much about nodejs or about archlinux pkgbuild scripts -- I am sure it can/should be fixed over there so feel free to ignore me :). In case it is useful or interesting here is what I found:
The reason it fails is that I already have npm installed through nodejs, and the package nodejs "owns" all of npm's files. If I install another package (tilemill) that tries to overwrite npm's files, the package manager won't let me.
@uniphil - thanks for the report. Can you surface this problem as a new issue on its own and share the exact errors and console output you saw? What you describe makes me think that somehow the npm install npm is causing the new npm to be installed somewhere outside of the TileMill local node_modules which should not be happening. All this preinstall target is intended to do is install npm in tilemill/node_modules/npm, which was already happening before just at the install target: this changes it to happen sooner.
1434317Just FYI -- this commit breaks the archlinux git package for tilemill. I don't know much about nodejs or about archlinux pkgbuild scripts -- I am sure it can/should be fixed over there so feel free to ignore me :). In case it is useful or interesting here is what I found:
The reason it fails is that I already have
npminstalled throughnodejs, and the packagenodejs"owns" all of npm's files. If I install another package (tilemill) that tries to overwrite npm's files, the package manager won't let me.1434317@uniphil - thanks for the report. Can you surface this problem as a new issue on its own and share the exact errors and console output you saw? What you describe makes me think that somehow the
npm install npmis causing the new npm to be installed somewhere outside of the TileMill localnode_moduleswhich should not be happening. All thispreinstalltarget is intended to do is install npm intilemill/node_modules/npm, which was already happening before just at theinstalltarget: this changes it to happen sooner.