Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id D716411690 for ; Wed, 23 Apr 2014 14:16:33 +0000 (UTC) Received: (qmail 95608 invoked by uid 500); 23 Apr 2014 14:16:33 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 95568 invoked by uid 500); 23 Apr 2014 14:16:33 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 95557 invoked by uid 99); 23 Apr 2014 14:16:32 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 23 Apr 2014 14:16:32 +0000 X-ASF-Spam-Status: No, hits=2.7 required=5.0 tests=FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_REPLY,HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of csantana23@gmail.com designates 74.125.82.182 as permitted sender) Received: from [74.125.82.182] (HELO mail-we0-f182.google.com) (74.125.82.182) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 23 Apr 2014 14:16:28 +0000 Received: by mail-we0-f182.google.com with SMTP id q59so931226wes.13 for ; Wed, 23 Apr 2014 07:16:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=k583MFslL2wfODjHv7IDk3RgpqNEsHg43NQBWgLxmtM=; b=fq/4D++BQYWpwJJFhvntjmXJhLAGzEFUWfYtm21VbUXqjw+7uyAjil5acJyxk2YqmV im6i8BLpBctiuLlEQVoMXPSP5HBd7T2SEbLOQGodxa9WAz+JHNd/1/zptG1m3rzg0Od7 WbDI2/upfzTUCZAcPOFBqs+WlqZSEXdbaeLTujnFwwxXjiLFe3oek+Xv6d6p0cKPARqf 4Jo1FKZiol2Hf9cg3B/9SFGUNmN8iiqg583oS9OWe+vY+M93eelin5Yjj5kgBMlca7CM 7kOQELhxmsPyihwIpfzaaCdZ/v2YNV6OQ40DfQAM0Q/UV+uhFZdDag2wBv+3sJWNvsiF 8HcQ== MIME-Version: 1.0 X-Received: by 10.180.93.41 with SMTP id cr9mr2026522wib.7.1398262566381; Wed, 23 Apr 2014 07:16:06 -0700 (PDT) Received: by 10.194.222.7 with HTTP; Wed, 23 Apr 2014 07:16:06 -0700 (PDT) In-Reply-To: References: <3AAB22CF-B009-4977-AE8A-23A10A54984C@gmail.com> Date: Wed, 23 Apr 2014 10:16:06 -0400 Message-ID: Subject: Re: [DISCUSS] Plugin master branches From: Carlos Santana To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=f46d04389357926f2204f7b65f80 X-Virus-Checked: Checked by ClamAV on apache.org --f46d04389357926f2204f7b65f80 Content-Type: text/plain; charset=UTF-8 +1 on using one branch "master" But, should we look into changing/enhancing the default behavior for "git" plugin install implementation to: if just a git url "https://github.com/apache/cordova-plugin-inappbrowser" it will query the tags/releases, and not pull latest commit and instead pull down latest release "r.0.4.0" from master. Or it's not worth? user can just do " https://github.com/apache/cordova-plugin-inappbrowser@r.0.4.0" --Carlos On Wed, Apr 23, 2014 at 9:30 AM, Michal Mocny wrote: > Also, it will only "break" new plugin installs. > > > On Tue, Apr 22, 2014 at 9:55 PM, Ian Clelland >wrote: > > > To be clear, this is just referring to Cordova CLI versions 3.0.0 - > 3.0.4, > > I think. By version 3.0.5, CLI had a dependency on plugman 0.10.0, which > > included the "plugman-registry" branch. (We didn't push anything to the > > registry until 3.1 was released, but we made sure that the infrastructure > > was ready a while before). > > > > If it is possible to use later versions of cordova-cli on a project that > > uses Cordova 3.0 engines, then we should be clear that we're not breaking > > Cordova 3.0 projects; just the oldest versions of the CLI, which > developers > > should be encouraged to upgrade in any case. > > > > > > > > On Tue, Apr 22, 2014 at 9:00 PM, Andrew Grieve > > wrote: > > > > > Didn't know about npm deprecate. Makes sense to me! > > > > > > On Tue, Apr 22, 2014 at 7:09 PM, Shazron wrote: > > > > Can we deprecate version 3.0? > > > > https://www.npmjs.org/doc/cli/npm-deprecate.html > > > > > > > > > > > > On Tue, Apr 22, 2014 at 4:00 PM, Anis KADRI > > > wrote: > > > > > > > >> +1 as well. This will break Cordova 3.0 though. Cordova versions >= > > 3.1 > > > are > > > >> fine because they support registries. Cordova 3.0 only supports git > > and > > > can > > > >> only fetch from master branches. > > > >> > > > >> > > > >> On Tue, Apr 22, 2014 at 11:32 AM, Joe Bowser > > wrote: > > > >> > > > >> > +1 > > > >> > > > > >> > On Tue, Apr 22, 2014 at 11:30 AM, Shazron > > wrote: > > > >> > > +1++ > > > >> > > > > > >> > > > > > >> > > On Tue, Apr 22, 2014 at 10:55 AM, Steven Gill < > > > stevengill97@gmail.com > > > >> > >wrote: > > > >> > > > > > >> > >> +1! > > > >> > >> > > > >> > >> > > > >> > >> On Tue, Apr 22, 2014 at 10:02 AM, James Jong < > > wjamesjong@gmail.com > > > > > > > >> > wrote: > > > >> > >> > > > >> > >> > +1 Making it easier and less confusing for our new > contributors > > > is > > > >> > good. > > > >> > >> > -James Jong > > > >> > >> > > > > >> > >> > On Apr 22, 2014, at 12:07 PM, Andrew Grieve < > > > agrieve@chromium.org> > > > >> > >> wrote: > > > >> > >> > > > > >> > >> > > +1! Certainly it's causing us a lot of pain still. Moving > to > > > >> > releasing > > > >> > >> > off > > > >> > >> > > of master seems like it would work fine. It's been working > > fine > > > >> for > > > >> > >> > > CLI/plugman, and they move much faster. > > > >> > >> > > > > > >> > >> > > > > > >> > >> > > On Tue, Apr 22, 2014 at 11:37 AM, Braden Shepherdson < > > > >> > >> > braden@chromium.org>wrote: > > > >> > >> > > > > > >> > >> > >> We originally needed the plugin releases to be on the > master > > > >> branch > > > >> > >> > because > > > >> > >> > >> there was no way to have CLI/Plugman fetch from other > > > branches. > > > >> > That > > > >> > >> is > > > >> > >> > no > > > >> > >> > >> longer the case. Further, you're correct that the > registry's > > > >> > tarballs > > > >> > >> is > > > >> > >> > >> the primary source now. Even if someone does have a git > > > >> dependency > > > >> > >> > >> somewhere, they can specify a branch (actually any ref) in > > the > > > >> > >> > > > > >> > >> > >> tag. Likewise the command line. > > > >> > >> > >> > > > >> > >> > >> I'm all for moving development into the master branch. > > > >> > >> > >> > > > >> > >> > >> Braden > > > >> > >> > >> > > > >> > >> > >> > > > >> > >> > >> On Tue, Apr 22, 2014 at 10:23 AM, Bryan Higgins < > > > >> > >> bryan@bryanhiggins.net > > > >> > >> > >>> wrote: > > > >> > >> > >> > > > >> > >> > >>> +1 > > > >> > >> > >>> > > > >> > >> > >>> I think the registry has been around for long enough that > > the > > > >> vast > > > >> > >> > >> majority > > > >> > >> > >>> of users won't be installing directly from git. > > > >> > >> > >>> > > > >> > >> > >>> > > > >> > >> > >>> On Tue, Apr 22, 2014 at 9:44 AM, Ian Clelland < > > > >> > >> iclelland@chromium.org > > > >> > >> > >>>> wrote: > > > >> > >> > >>> > > > >> > >> > >>>> I totally agree that we should do this. > > > >> > >> > >>>> > > > >> > >> > >>>> I think that once the current plugin release is > complete, > > I > > > can > > > >> > set > > > >> > >> up > > > >> > >> > >>> the > > > >> > >> > >>>> branches so that the master branch is for development, > and > > > we > > > >> > can go > > > >> > >> > >> from > > > >> > >> > >>>> there. > > > >> > >> > >>>> > > > >> > >> > >>>> Is it a requirement that plugins be tagged in git for > npm > > to > > > >> > >> function? > > > >> > >> > >> I > > > >> > >> > >>>> thought that the plugins were uploaded, zipped, to our > > couch > > > >> > server, > > > >> > >> > >> for > > > >> > >> > >>>> each release, and that there was no further > communication > > > with > > > >> > the > > > >> > >> git > > > >> > >> > >>>> repository? It shouldn't be a problem to go back and > make > > > sure > > > >> > >> they're > > > >> > >> > >>>> properly tagged, but I'm just wondering if it's still a > > > >> > necessity. > > > >> > >> > >>>> > > > >> > >> > >>>> Ian > > > >> > >> > >>>> > > > >> > >> > >>>> On Mon, Apr 21, 2014 at 9:27 PM, Jesse < > > > >> purplecabbage@gmail.com> > > > >> > >> > >> wrote: > > > >> > >> > >>>> > > > >> > >> > >>>>> I am seeing more and more pull requests that aren't > easy > > > >> merges > > > >> > >> > >> because > > > >> > >> > >>>>> people are starting their work from the master branch, > > and > > > not > > > >> > dev. > > > >> > >> > >>>>> > > > >> > >> > >>>>> We discussed *a long time ago* that at some point, we > > would > > > >> > >> consider > > > >> > >> > >>>> master > > > >> > >> > >>>>> to be the bleeding edge of each plugin, and we could > then > > > get > > > >> > rid > > > >> > >> of > > > >> > >> > >>> the > > > >> > >> > >>>>> dev branches. The requirements to make this possible > > > >> included, > > > >> > >> > >> using a > > > >> > >> > >>>>> branch/tag for every npm release of the plugin, and > > making > > > >> sure > > > >> > >> that > > > >> > >> > >>>> plugin > > > >> > >> > >>>>> dependencies were correctly mapped. > > > >> > >> > >>>>> > > > >> > >> > >>>>> Has anyone given this any more thought, and do we have > > any > > > >> idea > > > >> > >> when > > > >> > >> > >> we > > > >> > >> > >>>>> will make the switch? > > > >> > >> > >>>>> > > > >> > >> > >>>>> Cheers, > > > >> > >> > >>>>> Jesse > > > >> > >> > >>>>> > > > >> > >> > >>>>> > > > >> > >> > >>>>> @purplecabbage > > > >> > >> > >>>>> risingj.com > > > >> > >> > >>>>> > > > >> > >> > >>>> > > > >> > >> > >>> > > > >> > >> > >> > > > >> > >> > > > > >> > >> > > > > >> > >> > > > >> > > > > >> > > > > > > -- Carlos Santana --f46d04389357926f2204f7b65f80--