Return-Path: X-Original-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id C8B379D34 for ; Tue, 1 May 2012 00:31:28 +0000 (UTC) Received: (qmail 45980 invoked by uid 500); 1 May 2012 00:31:28 -0000 Delivered-To: apmail-incubator-callback-dev-archive@incubator.apache.org Received: (qmail 45944 invoked by uid 500); 1 May 2012 00:31:28 -0000 Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: callback-dev@incubator.apache.org Delivered-To: mailing list callback-dev@incubator.apache.org Received: (qmail 45933 invoked by uid 99); 1 May 2012 00:31:28 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 May 2012 00:31:28 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of shazron@gmail.com designates 209.85.210.175 as permitted sender) Received: from [209.85.210.175] (HELO mail-iy0-f175.google.com) (209.85.210.175) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 May 2012 00:31:22 +0000 Received: by iakk32 with SMTP id k32so5199413iak.6 for ; Mon, 30 Apr 2012 17:31:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type:content-transfer-encoding; bh=yI98E4jd6fWMLl5n51v06+DlF3r+GcGHmcCNw1Yx0E8=; b=djg1GQEHf/fPs8H1FVloYx+BzQCvfKY6jl+jm7y745m3uvtNN/YfqYUDXvPtNe1+w7 mEHktrZyJFTCqNJXNDx1uhYhBviROQI5FeTMvbtFZ9kDI1vkFVVeZ3aD8MRU/qDfCAFK NKSH8dLMBiqgGCk9px0o0ur2T+XhIWjyruRFRS8xDubvDDmkoDreMWnBMhT5FYaoKs+R QgoP7A01bcjzSjeDE75NKvNjwQ8W4+WB/a5LdDfSxFTxkaid0uDyNWywoZzT+lC27Uvi oYANEsH6kheluySU4ug4D3iB8S5TReCPAExU8E/yhzyHdGjEilnftVxhy4XeJ5gtNAQ4 Ueog== Received: by 10.43.49.3 with SMTP id uy3mr3351576icb.2.1335832261556; Mon, 30 Apr 2012 17:31:01 -0700 (PDT) MIME-Version: 1.0 Received: by 10.42.141.201 with HTTP; Mon, 30 Apr 2012 17:30:21 -0700 (PDT) In-Reply-To: References: <5775614995031544824@unknownmsgid> From: Shazron Date: Tue, 1 May 2012 00:30:21 +0000 Message-ID: Subject: Re: Ready to Tag 1.7.0? To: callback-dev@incubator.apache.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Failing tests? WP7 specific ones? On Tue, May 1, 2012 at 12:28 AM, Jesse wrote: > I will still have changes ... > > On Mon, Apr 30, 2012 at 5:12 PM, Joe Bowser wrote: > >> OK >> >> On Mon, Apr 30, 2012 at 5:06 PM, Anis KADRI wrote= : >> >> > do it >> > >> > On Mon, Apr 30, 2012 at 5:00 PM, Shazron wrote: >> > >> > > The fix for this so we can get 1.7.0 out is: >> > > >> > > 1. Revert >> > > >> > >> https://github.com/apache/incubator-cordova-js/blob/master/lib/cordova.j= s#L280-285 >> > > to it's previous use (no deprecation function call, straight >> > > assignment to cordova object - I've already re-opened CB-298) >> > > 2. Commit the fix >> > > 3. Un-tag and re-tag 1.7.0 cordova-js >> > > 4. All platforms get the new cordova-js, then re-test >> > > >> > > What do you guys think? >> > > >> > > On Mon, Apr 30, 2012 at 11:39 PM, Shazron wrote: >> > > > Found the problem, it's a timing issue: >> > > > >> > > > Before this function is called: >> > > > >> > > >> > >> https://github.com/apache/incubator-cordova-js/blob/master/lib/cordova.j= s#L281 >> > > > >> > > > ... cordova.exec is not defined yet. >> > > > >> > > > On Mon, Apr 30, 2012 at 11:13 PM, Joe Bowser >> > wrote: >> > > >> I do see it on Android, but this looks to me like a janky test wh= en >> > you >> > > >> look at it. =A0I guess we should un-tag and re-tag this. :( >> > > >> >> > > >> On Mon, Apr 30, 2012 at 4:01 PM, Shazron wrot= e: >> > > >> >> > > >>> iOS is fine except for >> https://issues.apache.org/jira/browse/CB-560 >> > > >>> being broken by the fix for >> > > >>> https://issues.apache.org/jira/browse/CB-298 which would fail fo= r >> > your >> > > >>> platforms as well, I'm sure you've seen this. >> > > >>> >> > > >>> Fil can you take a look at this when you get the chance since it >> was >> > > >>> your commit? I'm taking a look at it now, but you probably could >> > > >>> pinpoint this faster. >> > > >>> >> > > >>> >> > > >>> >> > > >>> On Mon, Apr 30, 2012 at 10:40 PM, Shazron >> wrote: >> > > >>> > Don't know what's going on with the tests, I've verified that >> the: >> > > >>> > 1. Contact update works, and does return a success callback (i= n >> > > Obj-C) >> > > >>> > - might be a faulty test, dunno? >> > > >>> > >> > > >>> > >> > > >>> > This one really fails -- PhoneGap.exec is not defined - probab= ly >> > > >>> > related to this change: >> > > >>> > >> > > >>> >> > > >> > >> https://github.com/apache/incubator-cordova-js/commit/b80ff3407ee5de1736= b26fd2f82a96c2dd8d34c5 >> > > >>> > >> > > >>> > >> > > >>> > >> > > >>> > On Mon, Apr 30, 2012 at 8:49 PM, Shazron >> > wrote: >> > > >>> >> Investigating two failing tests that didn't fail before: >> > > >>> >> >> > > >>> >> 1. Contacts (navigator.contacts) save method updated a contac= t. >> > > >>> >> (saveSuccess timeout) >> > > >>> >> 2. Platform (cordova) Platform (PhoneGap) exec method should >> exist >> > > >>> >> >> > > >>> >> >> > > >>> >> On Mon, Apr 30, 2012 at 8:43 PM, Joe Bowser >> > > wrote: >> > > >>> >>> Android is tested and tagged. >> > > >>> >>> >> > > >>> >>> On Mon, Apr 30, 2012 at 1:29 PM, Joe Bowser > > >> > > wrote: >> > > >>> >>> >> > > >>> >>>> Agreed. I have cb7a4c7a4 on this end. >> > > >>> >>>> >> > > >>> >>>> On Mon, Apr 30, 2012 at 1:18 PM, Jesse < >> purplecabbage@gmail.com >> > > >> > > >>> wrote: >> > > >>> >>>> >> > > >>> >>>>> Ah, that is the merge commit to my own fork ... >> > > >>> >>>>> >> > > >>> >>>>> >> > > >>> >> > > >> > >> https://github.com/purplecabbage/incubator-cordova-js/commit/97d4ff51d1c= 63ce1017fd54e1493795c193722f6 >> > > >>> >>>>> >> > > >>> >>>>> >> > > >>> >>>>> Seems this commit output file logging does not function as >> > > expected. >> > > >>> >>>>> >> > > >>> >>>>> On Mon, Apr 30, 2012 at 1:11 PM, Shazron >> > > wrote: >> > > >>> >>>>> >> > > >>> >>>>> > I don't think it's 97d4. >> > > >>> >>>>> > The tag was: >> > > >>> >>>>> > >> > > >>> >>>>> > Updated Tags: =A0refs/tags/1.7.0 [created] cb7a4c7a4 >> > > >>> >>>>> > >> > > >>> >>>>> > >> > > >>> >>>>> > On Mon, Apr 30, 2012 at 12:29 PM, Jesse < >> > > purplecabbage@gmail.com> >> > > >>> >>>>> wrote: >> > > >>> >>>>> > > Thanks Shaz! >> > > >>> >>>>> > > >> > > >>> >>>>> > > So everyone should be working from a platform js file >> that >> > > has >> > > >>> the >> > > >>> >>>>> SHA: >> > > >>> >>>>> > > // commit 97d4ff51d1c63ce1017fd54e1493795c193722f6 >> > > >>> >>>>> > > >> > > >>> >>>>> > > On Mon, Apr 30, 2012 at 12:23 PM, Shazron < >> > shazron@gmail.com >> > > > >> > > >>> wrote: >> > > >>> >>>>> > > >> > > >>> >>>>> > >> I tagged cordova-js so we can move on with testing. >> > > >>> >>>>> > >> >> > > >>> >>>>> > >> On Mon, Apr 30, 2012 at 12:05 PM, Shazron < >> > > shazron@gmail.com> >> > > >>> wrote: >> > > >>> >>>>> > >> > I can tag - no one is making changes anymore I reck= on? >> > > >>> >>>>> > >> > >> > > >>> >>>>> > >> > On Mon, Apr 30, 2012 at 11:31 AM, Joe Bowser < >> > > >>> bowserj@gmail.com> >> > > >>> >>>>> > wrote: >> > > >>> >>>>> > >> >> Who is tagging cordova-js again? >> > > >>> >>>>> > >> >> >> > > >>> >>>>> > >> >> On Mon, Apr 30, 2012 at 11:27 AM, Brian LeRoux < >> > > b@brian.io> >> > > >>> >>>>> wrote: >> > > >>> >>>>> > >> >> >> > > >>> >>>>> > >> >>> tag it, and create a branche for 1.8 >> > > >>> >>>>> > >> >>> >> > > >>> >>>>> > >> >>> On Mon, Apr 30, 2012 at 11:18 AM, Jesse < >> > > >>> purplecabbage@gmail.com >> > > >>> >>>>> > >> > > >>> >>>>> > >> wrote: >> > > >>> >>>>> > >> >>> > +1 on readiness >> > > >>> >>>>> > >> >>> > >> > > >>> >>>>> > >> >>> > On Mon, Apr 30, 2012 at 11:13 AM, Anis KADRI < >> > > >>> >>>>> > anis.kadri@gmail.com> >> > > >>> >>>>> > >> >>> wrote: >> > > >>> >>>>> > >> >>> > >> > > >>> >>>>> > >> >>> >> I am ready to tag my repos. I actually I'd lik= e >> to >> > > do it >> > > >>> >>>>> today so >> > > >>> >>>>> > >> that I >> > > >>> >>>>> > >> >>> >> can tackle the 1.8.0 tasks. >> > > >>> >>>>> > >> >>> >> >> > > >>> >>>>> > >> >>> >> On Mon, Apr 30, 2012 at 10:24 AM, Shazron >> Abdullah >> > < >> > > >>> >>>>> > >> shazron@gmail.com >> > > >>> >>>>> > >> >>> >> >wrote: >> > > >>> >>>>> > >> >>> >> >> > > >>> >>>>> > >> >>> >> > I'm gonna punt that one to 1.8. All I have t= o >> do >> > is >> > > >>> update >> > > >>> >>>>> > >> guides, etc >> > > >>> >>>>> > >> >>> >> > - looking at the checklist. And perhaps new >> bugs >> > > filed >> > > >>> over >> > > >>> >>>>> the >> > > >>> >>>>> > >> >>> >> > weekend, seeing if they are critical. >> > > >>> >>>>> > >> >>> >> > >> > > >>> >>>>> > >> >>> >> > On 2012-04-30, at 10:01 AM, Joe Bowser < >> > > >>> bowserj@gmail.com> >> > > >>> >>>>> > wrote: >> > > >>> >>>>> > >> >>> >> > >> > > >>> >>>>> > >> >>> >> > > Hey >> > > >>> >>>>> > >> >>> >> > > >> > > >>> >>>>> > >> >>> >> > > How are people feeling about tagging 1.7.0 >> > > today? I >> > > >>> notice >> > > >>> >>>>> > that >> > > >>> >>>>> > >> the >> > > >>> >>>>> > >> >>> >> > tracker >> > > >>> >>>>> > >> >>> >> > > only has a single iOS bug. >> > > >>> >>>>> > >> >>> >> > > >> > > >>> >>>>> > >> >>> >> > > Joe >> > > >>> >>>>> > >> >>> >> > >> > > >>> >>>>> > >> >>> >> >> > > >>> >>>>> > >> >>> > >> > > >>> >>>>> > >> >>> > >> > > >>> >>>>> > >> >>> > >> > > >>> >>>>> > >> >>> > -- >> > > >>> >>>>> > >> >>> > @purplecabbage >> > > >>> >>>>> > >> >>> > risingj.com >> > > >>> >>>>> > >> >>> >> > > >>> >>>>> > >> >> > > >>> >>>>> > > >> > > >>> >>>>> > > >> > > >>> >>>>> > > >> > > >>> >>>>> > > -- >> > > >>> >>>>> > > @purplecabbage >> > > >>> >>>>> > > risingj.com >> > > >>> >>>>> > >> > > >>> >>>>> >> > > >>> >>>>> >> > > >>> >>>>> >> > > >>> >>>>> -- >> > > >>> >>>>> @purplecabbage >> > > >>> >>>>> risingj.com >> > > >>> >>>>> >> > > >>> >>>> >> > > >>> >>>> >> > > >>> >> > > >> > >> > > > > -- > @purplecabbage > risingj.com