Ah, it makes sense in both platforms and plugins. Thanks,
-----Original Message----- From: Steven Gill [mailto:stevengil...@gmail.com] Sent: Thursday, October 22, 2015 4:46 PM To: dev@cordova.apache.org Subject: Re: npm install on cordova platform/plugin add Yeah. Cordova-common is the main reason for platforms. IMO It doesn't make sense to check it into platforms. It makes sense to have it as a bundleDependency but if you cordova platform add it via git url, the dependency won't be around. Hence the npm install On Oct 22, 2015 4:24 PM, "Darryl Pogue" <dar...@dpogue.ca> wrote: > One example would be the ios-sim and ios-deploy node modules that the > iOS platform needs for launching the simulator and deploying to a device. > Currently these are bundled dependencies and checked in to git. > > Another example is the Platform API that's been added to > cordova-android, which depends on cordova-common. > > On 22 October 2015 at 16:17, Mefire O. <ommen...@microsoft.com> wrote: > > > I can definitely see the case for adding deps specified in plugins' > > package.json, which is to make 'tools plugins' development easier. > > > > But I don't understand the use case for platforms. What scenario are > > we trying to fulfill there ? > > > > Thanks, > > Omar Mefire > > > > -----Original Message----- > > From: Steven Gill [mailto:stevengil...@gmail.com] > > Sent: Thursday, October 22, 2015 2:54 PM > > To: dev@cordova.apache.org > > Subject: npm install on cordova platform/plugin add > > > > We discussed running `npm install` on `cordova plugin add` at the > > Cordova F2F. > > > > I want to see how people feel about this. > > > > For installing a platform via git url, the dependency won't exist > > unless we check it in to the repo. This can cause problems. > > bundleDependencies > are > > only useful for installing via npm > > > > For platforms: > > > > i believe this is how it would work: > > > > Adding via `npm` > > > > 1) `cordova platform add ios` > > > > 2) cordova fetches `cordova-ios` from npm, adds it to global cordova > > directory > > > > 3) I'd like tools to run `npm install` , but these deps are most > > likely already included in fetched version due to > > `bundleDependecies`. So it wouldn't do anything > > > > Adding locally > > > > 1) `cordova platform add /YourLocalCordovaIOS` > > > > 2) cordova uses your `cordova-ios` > > > > 3) tools to run `npm install` which should have no impact (Assuming > you've > > done it manually) > > > > Adding via Git url > > > > 1) `cordova platform add > > > https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithu > b.com%2fapache%2fcordova-ios%60&data=01%7c01%7commenjik%40microsoft.co > m%7c8b52f2866d674b08c10b08d2db2b5347%7c72f988bf86f141af91ab2d7cd011db4 > 7%7c1&sdata=zdpqgA08JQxJ4h5DpWCpBiggl1RTZu%2fMN8gE%2fbXHPG8%3d > > > > 2) cordova clones it (i believe into global cordova directory) > > > > 3) tools to run `npm install` which is important because packages > > aren't bundled unless we manually check them in > > > > For Plugins > > It would allow plugins to have 3rd party npm dependencies and not > > have to check them in. This will be even more important once we > > switch to browserify method. > > > > Thoughts? > > >
