By default, npm install will install all modules listed as dependencies.
With the --production flag, npm will not install modules listed in
devDependencies.
I will just ignore the Warning on connect and not respin.
The warning is that connect is old from 2011, and I think is no longer need
it for cordova-js that I'm aware. we can remove it for the next release and
probably do a clean across all repos and see what else is stale to bump the
version or remove.
Also for next release lets "npm install --production" that way is less
files to be downloaded by end users when installing form npm.
On Tue, Sep 16, 2014 at 5:47 PM, Marcel Kinard <cmarc...@gmail.com> wrote:
> Sorry I'm not following exactly what you mean. Do you mean cordova-lib
> instead of cordova-cli?
>
> I also don't expect devDependencies to get installed when I do "npm -g
> install cordova@rc".
>
> cordova-js has the devDependency on connect, and does not have a
> shrinkwrap. cordova-lib has a dependency on cordova-js, but does have a
> shrinkwrap. When I generate the shrinkwrap in cordova-lib, all of the
> devDependencies in cordova-js are appearing in the cordova-lib shrinkwrap.
> That seems to be where it starts to go wrong.
>
> cd cordova-lib
> rm -r node_modules
> rm npm-shrinkwrap
> npm install
> npm shrinkwrap
>
> ...and I see the cordova-js devDependencies in the cordova-lib shrinkwrap.
> That's wrong, correct?
>
> It shouldn't matter if I had done an "npm link cordova-js" inside of
> cordova-lib, correct?
>
> I'm trying to figure out if I'm doing something wrong, or if the
> shrinkwrap command is misbehaving.
>
> On Sep 16, 2014, at 4:07 PM, Andrew Grieve <agri...@chromium.org> wrote:
>
> > You shouldn't be seeing devDependencies show up in cordova. I'd double
> make
> > sure that if you:
> >
> > cd cordova-cli
> > rm -r node_modules
> > rm npm-shrinkwrap.json
> > npm install
> > npm shrinkwrap
> >
> >
> > that you end up with an npm-shrinkwrap.json that includes the
> devDependency.
> >
> >
> > On Tue, Sep 16, 2014 at 4:01 PM, Marcel Kinard <cmarc...@gmail.com>
> wrote:
> >
> >> FYI, I am getting the following warning when I do "npm install
> cordova@rc
> >> ":
> >>
> >> npm WARN engine connect@1.8.5: wanted: {"node":">= 0.4.1 < 0.7.0"}
> >> (current: {"node":"0.10.31","npm":"1.4.23"})
> >>
> >> This is a devDependency in cordova-js, which I didn't change. If folks
> >> want this dependency changed to get rid of the warning, then it will
> mean
> >> respinning this tools release.
>
>
--
Carlos Santana
<csantan...@gmail.com>
