search

cozy / cozy-debian

Cozy Debian package
13 stars 4 forks source link

Test npm version #18

Closed clochix closed 8 years ago

clochix commented 8 years ago

Some users have installed npm manually and encounter problem installing some packages. We should probably check the installed npm version (in the install script or in the diagnose script).

frankrousseau commented 8 years ago

I think it would be simpler to move Cozy to Node 4

clochix commented 8 years ago

Last week, some user had strange bugs when installing the .deb or updating their apps. It appears that what because they installed npm with npm itself, instead of using the one packaged with Debian. I think we'll still encounter that sort of problem after moving to Node 4, so it's worth adding two line of script to check the version of npm.

nledez commented 8 years ago

I think it fixed by 183adc0a671203deda9eac73f5cfd65a7552f001

https://github.com/cozy/cozy-debian/blob/master/debian/cozy-depends.postinst#L25-L35

We can install the package "cozy-depends" before continue installation.

WDYT @clochix?

Can we close the issue?

clochix commented 8 years ago

I do think this is definitively great ! Thanks !