Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (nike.apache.org: domain of stevengill97@gmail.com
 designates 209.85.216.46 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAMSbhBiOAOtvsFNNb4mZtYZTa2J3s+-SUAKaefXWzGjp9NTyMg@mail.gmail.com>
References: 
 <CAMSbhBiHwS8sZ+3-q2dw7Vq4XSLPv3Qw_2D80VZrU3vLVYFGpw@mail.gmail.com>
 <CAHcDwFwBs-9tbWBAq-VkbWZUuxVAH1pFkUW2S3QCBNaTBK99jA@mail.gmail.com>
 <a4641794ab634b26b51225388388e9c9@BLUPR03MB359.namprd03.prod.outlook.com>
 <CAEeRA4qGFDUcg9CE4T0P1AFsJGH80WsCML4XzSohcNuKtSiiHA@mail.gmail.com>
 <CAMTxO43om2Q072vuLrREMVG=7QA2KLtiZt4buTtYQ0NBMRR2uw@mail.gmail.com>
 <CAEeRA4qRmyTje-HkXfXUFDKR0Zp2KsUdr2Hm0LruLtmnuoVRYQ@mail.gmail.com>
 <CAK_TSXKM+sQeCrpsOyxNQTWvJGSFdYhRfQSNYtkXtk+F_KQLLw@mail.gmail.com>
 <CAEeRA4pkUVi1P5+-HF-zVtmgv9ChyXzbsbs1HpzWAmvcugxQgw@mail.gmail.com>
 <CAMSbhBivbmrTqDxn7p5+AHBCq2Nf+O4Of0fc3TD2GOKHTSZkig@mail.gmail.com>
 <CAMSbhBj7w0299EUw8vacJV=P0x4pioCosuReDYZYSqxRn7rNQw@mail.gmail.com>
 <CAE6O_-ajQS2TMT0YwfK0+dr=KsGwGfhV7mJKH9mQZHoP0J4aBQ@mail.gmail.com>
 <CAMSbhBiOAOtvsFNNb4mZtYZTa2J3s+-SUAKaefXWzGjp9NTyMg@mail.gmail.com>
From: Steven Gill <stevengill97@gmail.com>
Date: Tue, 8 Apr 2014 13:51:24 -0600
Message-ID: 
 <CAMSbhBi7DzhXyb5jBmvQPMnt08FN7FxuD3ySFyvUaTrzA9M5Wg@mail.gmail.com>
Subject: Re: [VOTE] cordoav-cli@3.4.1-0.1.0,
 cordova-plugman@0.21.0 and cordova-ios@3.4.1
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=001a1133cbc847920404f68d502f

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

This vote has failed and has been broken into separate vote threads.


On Sun, Apr 6, 2014 at 10:44 PM, Steven Gill <stevengill97@gmail.com> wrote:

> Just chatted with Ian at the bar about this.
>
> I think the best way to move forward is to vote on plugman first, release
> it and then make sure cli has proper references and vote on that.
>
> I'll send out a thread now
>
>
> On Sun, Apr 6, 2014 at 9:39 PM, Shazron <shazron@gmail.com> wrote:
>
>> Hi Steve,
>> Not sure what the next steps are -- are the -1s being resolved so we can
>> get this out? Let me know how I can help.
>>
>>
>> On Fri, Apr 4, 2014 at 10:45 AM, Steven Gill <stevengill97@gmail.com>
>> wrote:
>>
>> > Sorry, didn't mean to hit send on that last message quite so quickly.
>> >
>> > We could add release branches but it would require us to change our
>> release
>> > process for them.
>> >
>> > Isn't master only broken until we release. If the process is completed
>> > within 24hours, is having master broken a huge deal? Most people will
>> use
>> > npm to install cordova anyways.
>> >
>> >
>> >
>> > On Fri, Apr 4, 2014 at 10:43 AM, Steven Gill <stevengill97@gmail.com>
>> > wrote:
>> >
>> > > We don't have release branches for CLI + Plugman.
>> > >
>> > >
>> > > On Fri, Apr 4, 2014 at 10:40 AM, Bryan Higgins <
>> bryan@bryanhiggins.net
>> > >wrote:
>> > >
>> > >> SGTM
>> > >>
>> > >>
>> > >> On Fri, Apr 4, 2014 at 1:38 PM, Ian Clelland <iclelland@chromium.org
>> >
>> > >> wrote:
>> > >>
>> > >> > What about only adding the shrinkwrap file to git just before
>> release,
>> > >> on
>> > >> > the release branch? Then only published releases on npm (and on
>> > >> > dist.apache.org) will include that file, and the only people who
>> have
>> > >> to
>> > >> > delete it manually are those people testing the release candidate.
>> > >> Anyone
>> > >> > getting master from git will not have that file.
>> > >> >
>> > >> >
>> > >> > On Fri, Apr 4, 2014 at 1:25 PM, Bryan Higgins <
>> bryan@bryanhiggins.net
>> > >> > >wrote:
>> > >> >
>> > >> > > Right, but someone outside of this group stumbling upon the repo
>> on
>> > >> > GitHub
>> > >> > > won't be able to use it unless they know to delete the shrinkwrap
>> > >> file.
>> > >> > I'd
>> > >> > > rather the process be a bit more cumbersome for us and preserve
>> the
>> > >> > > integrity of repo.
>> > >> > >
>> > >> > >
>> > >> > > On Fri, Apr 4, 2014 at 1:11 PM, David Kemp <drkemp@chromium.org>
>> > >> wrote:
>> > >> > >
>> > >> > > > If we make the proposed change to the CLI package.json (change
>> the
>> > >> > > plugman
>> > >> > > > dependency to ">=0.21.0-rc"),
>> > >> > > > then all that is required for testing is to delete the
>> shrinkwrap
>> > >> file.
>> > >> > > >
>> > >> > > > That means a little less hackery to test the package that is
>> being
>> > >> > voted
>> > >> > > > on.
>> > >> > > >
>> > >> > > > We have currently made those changes on the fly in the CI to
>> get
>> > it
>> > >> > > working
>> > >> > > > again.
>> > >> > > >
>> > >> > > >
>> > >> > > >
>> > >> > > > On Fri, Apr 4, 2014 at 12:57 PM, Bryan Higgins <
>> > >> bryan@bryanhiggins.net
>> > >> > > > >wrote:
>> > >> > > >
>> > >> > > > > The other problem we face is the actual repo being unusable
>> for
>> > a
>> > >> > > period
>> > >> > > > of
>> > >> > > > > time unless you know how to fiddle with package.json /
>> > >> > > > npm-shrinkwrap.json.
>> > >> > > > >
>> > >> > > > > At the very least this should be pointed out on the README,
>> but
>> > >> even
>> > >> > > that
>> > >> > > > > doesn't seem friendly to potential contributors.
>> > >> > > > >
>> > >> > > > > I'm in favour of bumping the version number in CLI after the
>> > vote
>> > >> for
>> > >> > > > cases
>> > >> > > > > like this where we're releasing tools together. Those testing
>> > the
>> > >> CLI
>> > >> > > for
>> > >> > > > > release can bump up the plugman version locally.
>> > >> > > > >
>> > >> > > > > Otherwise we should vote on plugman first, then update CLI
>> and
>> > >> vote
>> > >> > on
>> > >> > > > > that...
>> > >> > > > >
>> > >> > > > >
>> > >> > > > > On Fri, Apr 4, 2014 at 12:33 PM, Parashuram Narasimhan (MS
>> OPEN
>> > >> > TECH) <
>> > >> > > > > panarasi@microsoft.com> wrote:
>> > >> > > > >
>> > >> > > > > > Can we also include the Windows Phone and Windows Platform,
>> > >> > > > specifically
>> > >> > > > > > for the following bugs/pull requests
>> > >> > > > > >
>> > >> > > > > > 1.      Windows 8 -
>> > >> > > https://github.com/apache/cordova-windows/pull/20-
>> > >> > > > > > This is more of a regression in case VS2013 is installed.
>> With
>> > >> the
>> > >> > > > build
>> > >> > > > > > announcement yesterday, this could be higher priority. Note
>> > that
>> > >> > this
>> > >> > > > is
>> > >> > > > > > not yet merged in.
>> > >> > > > > > 2.      Windows Phone 8 -
>> > >> > > > https://github.com/apache/cordova-wp8/pull/28-
>> > >> > > > > > This could affect the users using the emulator.
>> > >> > > > > >
>> > >> > > > > > -----Original Message-----
>> > >> > > > > > From: Axel Nennker [mailto:ignisvulpis@gmail.com]
>> > >> > > > > > Sent: Friday, April 4, 2014 9:21 AM
>> > >> > > > > > To: dev
>> > >> > > > > > Subject: Re: [VOTE] cordoav-cli@3.4.1-0.1.0,
>> > >> > > cordova-plugman@0.21.0and
>> > >> > > > > > cordova-ios@3.4.1
>> > >> > > > > >
>> > >> > > > > > Is there a chance to get the cb-2606 patch for launcer icon
>> > >> support
>> > >> > > > into
>> > >> > > > > a
>> > >> > > > > > release some time no too far?
>> > >> > > > > >
>> > >> > > > > > Axel
>> > >> > > > > > Am 04.04.2014 10:01 schrieb "Steven Gill" <
>> > >> stevengill97@gmail.com
>> > >> > >:
>> > >> > > > > >
>> > >> > > > > > > Please review and vote on the release of this
>> cordova-cli,
>> > >> > > > > > > cordova-plugman and cordova-ios release.
>> > >> > > > > > >
>> > >> > > > > > > cordova-cli@3.4.1-0.1.0, cordova-plugman@0.21.0 and
>> > >> > > > cordova-ios@3.4.1
>> > >> > > > > > > have been published here:
>> > >> > > > > > > *https://dist.apache.org/repos/dist/dev/cordova/CB-6245/
>> > >> > > > > > > <https://dist.apache.org/repos/dist/dev/cordova/CB-6245/
>> >*
>> > >> > > > > > >
>> > >> > > > > > >
>> > >> > > > > > > The packages were published from their corresponding git
>> > tags:
>> > >> > > > > > >     cordova-cli: 3.4.1-0.1.0 (b769a304be)
>> > >> > > > > > >     cordova-plugman: 0.21.0 (b2f3a130d3)
>> > >> > > > > > >     cordova-ios: 3.4.1 (a96d2360fa)
>> > >> > > > > > >
>> > >> > > > > > > Upon a successful vote I will upload the cli & plugman
>> > >> archives
>> > >> > to
>> > >> > > > > > > dist/ and publish them to npm. Cordova-ios will be
>> uploaded
>> > to
>> > >> > > > > > > dist/platforms. I will then post the corresponding blog
>> > post.
>> > >> > > > > > >
>> > >> > > > > > > Voting will go on for a minimum of 24 hours.
>> > >> > > > > > >
>> > >> > > > > > > I vote +1.
>> > >> > > > > > >
>> > >> > > > > > > If people want individual vote threads for each item,
>> let me
>> > >> know
>> > >> > > and
>> > >> > > > > > > I will create them instead of this thread.
>> > >> > > > > > >
>> > >> > > > > >
>> > >> > > > >
>> > >> > > >
>> > >> > >
>> > >> >
>> > >>
>> > >
>> > >
>> >
>>
>
>

--001a1133cbc847920404f68d502f--