Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (athena.apache.org: domain of stevengill97@gmail.com
 designates 209.85.192.49 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CABiQX1X3jrTDMphW820H1VOWXuqzYLwLs8PG1H+tHs34gqxwyw@mail.gmail.com>
References: 
 <CAMSbhBg_AVZst8SR5AixhonJ4=KnEGr+eHLoHui4wpYKKV_VEQ@mail.gmail.com>
 <CAMSbhBjxa8ytJeUhaD1VuuZ42dR19eb8scU3_A05q9ohPkaSZw@mail.gmail.com>
 <CABiQX1W-bBw-7qkPgYisQFBDHYPt=RD4jFRqCww_DRtdc1pdOA@mail.gmail.com>
 <CA+z8NA_AP8BuC4+ioD_W8vLKLpozZ=ptqv+Q58pTYzzkMEUCMA@mail.gmail.com>
 <CAEeF2TdJLSpFkkJHB3jsZ5qgK5L7Q_62q0Oz+Tpt26fe=6H+hA@mail.gmail.com>
 <CAMSbhBjBbgEFQkOK4yYtiuAEFDM1UO5v5KwekPmVN8YZAaQuDw@mail.gmail.com>
 <CAMSbhBhCx=L4FkT7Z=+0X7ud-sNAgPH-oTLKvmZyh1ECLoU-xQ@mail.gmail.com>
 <CAMSbhBjE9BajAxa3cBxBYVGBb_Yw-rjgCz+-32WtDD2JYC8uQA@mail.gmail.com>
 <CABiQX1X3jrTDMphW820H1VOWXuqzYLwLs8PG1H+tHs34gqxwyw@mail.gmail.com>
From: Steven Gill <stevengill97@gmail.com>
Date: Thu, 5 Mar 2015 15:18:36 -0800
Message-ID: 
 <CAMSbhBh0y95iuhf2fPGEULY4RJN1anFV96svX+0fvtRm6ASjzg@mail.gmail.com>
Subject: Re: [REVIEW] NPM Registry integration for fetching plugins and plugin
 rm
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=001a113549d6be7142051092ca93

--001a113549d6be7142051092ca93
Content-Type: text/plain; charset=ISO-8859-1

I will bump up the dep mapper to 1.0.0 and set it the dep to 1.x in
cordova-lib. Then I will merge the PR into master!

On Wed, Mar 4, 2015 at 5:55 PM, Andrew Grieve <agrieve@chromium.org> wrote:

> looks good. Should you un-pin the version of dep-mapper though?
>
> On Wed, Mar 4, 2015 at 8:03 PM, Steven Gill <stevengill97@gmail.com>
> wrote:
>
> > I have updated the PR based on feedback. Take a look. I'd like to merge
> it
> > into master if everyone feels good about it.
> >
> > https://github.com/apache/cordova-lib/pull/175/files
> >
> > On Tue, Mar 3, 2015 at 11:59 AM, Steven Gill <stevengill97@gmail.com>
> > wrote:
> >
> > > Thanks for all of the feedback so far! I will spend some time this
> > > afternoon making some changes.
> > >
> > > On Tue, Mar 3, 2015 at 9:13 AM, Steven Gill <stevengill97@gmail.com>
> > > wrote:
> > >
> > >> Darryl,
> > >>
> > >> +1 to what Michal said. We are still figuring out details for phase 2.
> > >> Support for package.json and installation from node_modules are two
> > things
> > >> I'd love to see supported.
> > >> On Mar 3, 2015 8:54 AM, "Michal Mocny" <mmocny@chromium.org> wrote:
> > >>
> > >>> Darryl,
> > >>>
> > >>> I believe that cordova-cli will continue to use `cordova plugin add`
> > and
> > >>> not use `node_modules` at all for phase 1 rollout.  It will use the
> > `npm`
> > >>> library to fetch into the existing plugins/ folder.  There will be no
> > >>> user
> > >>> visible change to project structure, though the plugins are being
> > renamed
> > >>> as part of the transition.
> > >>>
> > >>> -Michal
> > >>>
> > >>> On Tue, Mar 3, 2015 at 10:31 AM, Darryl Pogue <dvpdiner2@gmail.com>
> > >>> wrote:
> > >>>
> > >>> > Less a comment on the code and more of a question about the NPM
> > >>> > transition from an end-user pespective:
> > >>> >
> > >>> > Is Cordova going to automatically discover and use plugins that are
> > >>> > found in the node_modules folder (from package.json), or will it
> > still
> > >>> > keep its own plugins folder and require they be installed using
> > >>> > `cordova plugin add <name>`?
> > >>> >
> > >>> > On 3 March 2015 at 06:40, Andrew Grieve <agrieve@chromium.org>
> > wrote:
> > >>> > >
> > >>> > > Good stuff. Added some style comments, but logic lgtm!
> > >>> > >
> > >>> > > On Tue, Mar 3, 2015 at 3:55 AM, Steven Gill <
> > stevengill97@gmail.com>
> > >>> > wrote:
> > >>> > >
> > >>> > > > Also, Issue: https://issues.apache.org/jira/browse/CB-8551
> > >>> > > >
> > >>> > > > On Tue, Mar 3, 2015 at 12:54 AM, Steven Gill <
> > >>> stevengill97@gmail.com>
> > >>> > > > wrote:
> > >>> > > >
> > >>> > > > > PR: https://github.com/apache/cordova-lib/pull/175/files
> > >>> > > > >
> > >>> > > > > Please review, leave feedback and ask questions!
> > >>> > > > >
> > >>> > > >
> > >>> >
> > >>> >
> ---------------------------------------------------------------------
> > >>> > To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
> > >>> > For additional commands, e-mail: dev-help@cordova.apache.org
> > >>> >
> > >>> >
> > >>>
> > >>
> > >
> >
>

--001a113549d6be7142051092ca93--