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 D21479AA9 for ; Thu, 12 Apr 2012 23:13:09 +0000 (UTC) Received: (qmail 89352 invoked by uid 500); 12 Apr 2012 23:13:09 -0000 Delivered-To: apmail-incubator-callback-dev-archive@incubator.apache.org Received: (qmail 89322 invoked by uid 500); 12 Apr 2012 23:13:09 -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 89314 invoked by uid 99); 12 Apr 2012 23:13:09 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 12 Apr 2012 23:13:09 +0000 X-ASF-Spam-Status: No, hits=1.3 required=5.0 tests=FREEMAIL_REPLY,FRT_ADOBE2,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; Thu, 12 Apr 2012 23:13:02 +0000 Received: by iaag37 with SMTP id g37so3543862iaa.6 for ; Thu, 12 Apr 2012 16:12:41 -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=/4/91ZSAtD+mL14fcfjJdIX7ZF+X4Sk9rrMzbMKzGLs=; b=xrnueZWYopDxnl/0P73hwoCIZ/Gm09nyYwPGDw/39CJvXV7blKCWL+ttms68Tj2O/1 EyMuv3FCTdtQrfu/oIu6IqJr4K73nHp6Z7pGgFVg3hPQkVdjmWEotuQgw1TGt34kUDM/ wRQNwSv19DMv6eXMb1T362eakMq4ewYrxXqHTGrC2WavGdVWEKUSrZ43dSjVfoZXpkhi Ugj2QKGrASmNg5irnA8b+aKZzNhMuj/Vlnauiazm2OBFFHASGuMxAM9zKWd2AtvyWKHF uPZTY5Hd/jPtS35kwrWGFIgfwCxSPOefjSy+lHLv7juqbZoYRSLDv8uCNLVqUO3pvkpj 8Wwg== Received: by 10.50.153.132 with SMTP id vg4mr72219igb.2.1334272361549; Thu, 12 Apr 2012 16:12:41 -0700 (PDT) MIME-Version: 1.0 Received: by 10.42.146.131 with HTTP; Thu, 12 Apr 2012 16:12:00 -0700 (PDT) In-Reply-To: References: From: Shazron Date: Thu, 12 Apr 2012 16:12:00 -0700 Message-ID: Subject: Re: 1.6.1 To: callback-dev@incubator.apache.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable I'll remove it. *grabs the shovel* On Thu, Apr 12, 2012 at 4:10 PM, Joe Bowser wrote: > You know, let's not rebase it, because it does modify the history. =A0We > should remove and document why we did it instead, since that would be mor= e > transparent. Or we could just leave it in and say "don't do this next > time". =A0 I'm not that invested in it. > > On Thu, Apr 12, 2012 at 4:04 PM, Shazron wrote: > >> If we take out that iOS commit in cordova-js (somehow), it should be >> fine. cordova-ios 1.6.1 will then just be the same as 1.6.0. >> >> On Thu, Apr 12, 2012 at 4:02 PM, Filip Maj wrote: >> > >> >>Is it too much to drop the new commit that changed the common js API, = and >> >>reapply it after we tag 1.6.1, and figure this one thing out for 1.7? >> > >> > That sounds very good to me but I am under the impression that iOS >> > requires this commit? Like I said I'm quite confused. >> > >> >> >> >>I also have some cordova-js commits to add to 1.6.1? Although nothing >> that >> >>modifies any js outside of what is built for wp7. ( ie, all in my >> platform >> >>specific plugins folder ) >> >> >> > >> > O then it sounds like I tagged the JS 1.6.1 too early anyways.. >> > >> >> >> >> >> >> >> >>On Thu, Apr 12, 2012 at 3:51 PM, Shazron wrote: >> >> >> >>> No - it's about a user's contribution being noted (it exists), I wil= l >> >>> move it out of the common code post-1.6.1 (I acknowledge my mistake = in >> >>> allowing it in the first place). Can we branch off 1.6.0 then >> >>> cherry-pick Fil's latest fix for 1.6.1? >> >>> >> >>> On Thu, Apr 12, 2012 at 3:47 PM, Joe Bowser wrot= e: >> >>> > OK, so you're proposing that we leave the thing in the common JS, >> even >> >>> > though it's not supposed to be there. >> >>> > >> >>> > On Thu, Apr 12, 2012 at 3:45 PM, Shazron wrote= : >> >>> > >> >>> >> Will rebasing lose that commit? Then, no. >> >>> >> >> >>> >> On Thu, Apr 12, 2012 at 3:43 PM, Joe Bowser >> >>>wrote: >> >>> >> > So, can we just rebase the head, push this out and get this >> >>>rolling? >> >>> >> > >> >>> >> > On Thu, Apr 12, 2012 at 3:43 PM, Filip Maj wrot= e: >> >>> >> > >> >>> >> >> Correct, those iOS-specific parameters (like allow edit and th= ose >> >>> ones) >> >>> >> >> existed prior to cordova-js, so yeah, we left them in there as= we >> >>> didn't >> >>> >> >> want to remove without deprecation. >> >>> >> >> >> >>> >> >> If you can deal with the iOS issue without needing an API >> >>>extension: >> >>> >> then >> >>> >> >> do that. If not, leave it in there, but docs need to updated t= oo. >> >>> >> >> >> >>> >> >> Extending and changing the public API is a long and difficult >> >>>process >> >>> >> for >> >>> >> >> us and will only get tougher with every platform we add suppor= t >> >>>to. >> >>> >> >> >> >>> >> >> On 4/12/12 3:37 PM, "Shazron" wrote: >> >>> >> >> >> >>> >> >> >Then we have to scrap the other three(?) iOS parameters ones >> that >> >>> >> >> >already exist in common if you noticed. Don't know how we mer= ge >> >>> >> >> >(unmerge?) this one properly because taking those out will br= eak >> >>> >> >> >everyone that is using it, but a deprecation is the way to go= as >> >>>per >> >>> >> >> >http://wiki.apache.org/cordova/DeprecationPolicy . We can >> remove >> >>> this >> >>> >> >> >new commit and I'll find a way to get this in somehow. >> >>> >> >> > >> >>> >> >> > >> >>> >> >> > >> >>> >> >> >On Thu, Apr 12, 2012 at 3:32 PM, Joe Bowser >> >>> wrote: >> >>> >> >> >> The problem is that this parameter doesn't belong in the >> common >> >>> API. >> >>> >> >> >>It'd >> >>> >> >> >> be just ignored in Android, but the whole point of cordova-= js >> >>>is >> >>> so >> >>> >> that >> >>> >> >> >> it's common. =A0It'd look pretty screwed up to add this to = the >> >>>docs, >> >>> >> IMO. >> >>> >> >> >> >> >>> >> >> >> On Thu, Apr 12, 2012 at 3:26 PM, Shazron >> >>> wrote: >> >>> >> >> >> >> >>> >> >> >>> To fix two bugs for Camera.takePicture on iOS >> >>> >> >> >>> >> >>> >> >> >>> On Thu, Apr 12, 2012 at 3:24 PM, Filip Maj >> >>> wrote: >> >>> >> >> >>> > so what's the point of this new parameter? >> >>> >> >> >>> > >> >>> >> >> >>> > On 4/12/12 3:16 PM, "Shazron" wrote: >> >>> >> >> >>> > >> >>> >> >> >>> >>I did. Do we want to revert the cordova-js? I can branch >> >>>1.6.0 >> >>> and >> >>> >> >> >>> >>cherry-pick your latest commit for 1.6.1 >> >>> >> >> >>> >> >> >>> >> >> >>> >>On Thu, Apr 12, 2012 at 3:04 PM, Filip Maj >> >>> wrote: >> >>> >> >> >>> >>> I saw something in the iOS commit logs for this. >> >>> >> >> >>> >>> >> >>> >> >> >>> >>> Shaz can you comment? >> >>> >> >> >>> >>> >> >>> >> >> >>> >>> On 4/12/12 2:59 PM, "Joe Bowser" >> >>>wrote: >> >>> >> >> >>> >>> >> >>> >> >> >>> >>>>Can we revert this, since this looks like it's intende= d >> >>>for >> >>> both >> >>> >> >> >>> Android >> >>> >> >> >>> >>>>and iOS, but I don't think it should be in 1.6.1. >> >>> >> >> >>> >>>> >> >>> >> >> >>> >>>>On Thu, Apr 12, 2012 at 2:52 PM, Filip Maj < >> fil@adobe.com> >> >>> >> wrote: >> >>> >> >> >>> >>>> >> >>> >> >> >>> >>>>> I was actually just looking at that myself. >> >>> >> >> >>> >>>>> >> >>> >> >> >>> >>>>> Looks like it's an iOS-only param.. =A0sigh. We gott= a >> stop >> >>> doing >> >>> >> >> >>>that, >> >>> >> >> >>> >>>>> especially if we don't update documentation. >> >>> >> >> >>> >>>>> >> >>> >> >> >>> >>>>> On 4/12/12 2:49 PM, "Joe Bowser" >> >>> wrote: >> >>> >> >> >>> >>>>> >> >>> >> >> >>> >>>>> >Has anyone tested the cropToSize param for >> >>> Camera.getPicture >> >>> >> >> >>>yet? =A0I >> >>> >> >> >>> >>>>> >didn't >> >>> >> >> >>> >>>>> >notice that change until now. >> >>> >> >> >>> >>>>> > >> >>> >> >> >>> >>>>> >On Thu, Apr 12, 2012 at 2:46 PM, Filip Maj >> >>>> >>> > >> >>> >> >> >>>wrote: >> >>> >> >> >>> >>>>> > >> >>> >> >> >>> >>>>> >> JS tagged 1.6.1 - the build task in the JS should >> >>>work >> >>> >> >> >>>properly >> >>> >> >> >>> >>>>>now. >> >>> >> >> >>> >>>>> >> >> >>> >> >> >>> >>>>> >> FYI if anyone has to modify the jake tasks in the >> >>>future >> >>> >> and >> >>> >> >> >>>make >> >>> >> >> >>> >>>>>them >> >>> >> >> >>> >>>>> >> async, make sure you add the 4th "async" paramete= r >> to >> >>> the >> >>> >> >> >>>task() >> >>> >> >> >>> >>>>> >>function >> >>> >> >> >>> >>>>> >> and set it to true, and call complete() when the >> >>>task is >> >>> >> done. >> >>> >> >> >>> >>>>> >> >> >>> >> >> >>> >>>>> >> See >> >>> http://howtonode.org/intro-to-jake/async-jakefile.jsfor >> >>> >> >> >>>an >> >>> >> >> >>> >>>>> example. >> >>> >> >> >>> >>>>> >> >> >>> >> >> >>> >>>>> >> On 4/12/12 2:35 PM, "Joe Bowser" > > >> >>> >> wrote: >> >>> >> >> >>> >>>>> >> >> >>> >> >> >>> >>>>> >> >Ready to tag once cordova-js is tagged 1.6.1. >> >>> >> >> >>> >>>>> >> > >> >>> >> >> >>> >>>>> >> >On Thu, Apr 12, 2012 at 2:33 PM, Anis KADRI < >> >>> >> >> >>> anis.kadri@gmail.com> >> >>> >> >> >>> >>>>> >>wrote: >> >>> >> >> >>> >>>>> >> > >> >>> >> >> >>> >>>>> >> >> I will tag my repository to 1.6.1 before end o= f >> >>>day >> >>> >> today >> >>> >> >> >>>after >> >>> >> >> >>> >>>>>some >> >>> >> >> >>> >>>>> >> >> thorough testing. >> >>> >> >> >>> >>>>> >> >> >> >>> >> >> >>> >>>>> >> >> On Thu, Apr 12, 2012 at 2:30 PM, Shazron >> >>> >> >> >>> >> >>> >> >> >>> >>>>>wrote: >> >>> >> >> >>> >>>>> >> >> >> >>> >> >> >>> >>>>> >> >> > I also created a 1.6.1 version tag as well i= n >> >>> jira... >> >>> >> >> >>> >>>>> >> >> > >> >>> >> >> >>> >>>>> >> >> > On Thu, Apr 12, 2012 at 2:29 PM, Shazron >> >>> >> >> >>> >> >>> >> >> >>> >>>>> wrote: >> >>> >> >> >>> >>>>> >> >> > > Alright guys since 1.6.0 was released, I'm >> >>>going >> >>> to >> >>> >> >> >>> >>>>>"Release" >> >>> >> >> >>> >>>>>the >> >>> >> >> >>> >>>>> >> >> > > 1.6.0 version in jira and move outstanding >> >>> issues to >> >>> >> >> >>>1.7.0 >> >>> >> >> >>> >>>>> >>version >> >>> >> >> >>> >>>>> >> >> > > tag. >> >>> >> >> >>> >>>>> >> >> > > >> >>> >> >> >>> >>>>> >> >> > > >> >>> >> >> >>> >>>>> >> >> > > On Thu, Apr 12, 2012 at 2:26 PM, Filip Maj >> >>> >> >> >>> >> >>> >> >> >>> >>>>> wrote: >> >>> >> >> >>> >>>>> >> >> > >> The JS needs patching too. Since adding t= he >> >>> commit >> >>> >> ID >> >>> >> >> >>>into >> >>> >> >> >>> >>>>>the >> >>> >> >> >>> >>>>> >> >> packaging >> >>> >> >> >>> >>>>> >> >> > >> in the jakefile, the default task isn't >> >>>running >> >>> >> >> >>>properly. >> >>> >> >> >>> >>>>> >> >> > >> >> >>> >> >> >>> >>>>> >> >> > >> You can manually run "jake build" and "ja= ke >> >>> test" >> >>> >> but >> >>> >> >> >>>not >> >>> >> >> >>> >>>>> >>"jake" on >> >>> >> >> >>> >>>>> >> >> its >> >>> >> >> >>> >>>>> >> >> > >> own - gonna try to figure it out. >> >>> >> >> >>> >>>>> >> >> > >> >> >>> >> >> >>> >>>>> >> >> > >> On 4/12/12 2:21 PM, "Shazron" < >> >>> shazron@gmail.com> >> >>> >> >> >>>wrote: >> >>> >> >> >>> >>>>> >> >> > >> >> >>> >> >> >>> >>>>> >> >> > >>>I'm good with tagging. =A0iOS has a few >> commits >> >>> (one >> >>> >> >> >>>commit >> >>> >> >> >>> >>>>>in >> >>> >> >> >>> >>>>> >> >> > >>>cordova-js) as well. Aim for first thing >> >>> tomorrow >> >>> >> for >> >>> >> >> >>>you >> >>> >> >> >>> >>>>>to >> >>> >> >> >>> >>>>> >> >> > >>>repackage? >> >>> >> >> >>> >>>>> >> >> > >>> >> >>> >> >> >>> >>>>> >> >> > >>>On Thu, Apr 12, 2012 at 2:17 PM, Steven G= ill >> >>> >> >> >>> >>>>> >> >> >> >>> >> >> >>> >>>>> >> >> > >>>wrote: >> >>> >> >> >>> >>>>> >> >> > >>>> So a bug has been discovered in Cordova >> >>> Android >> >>> >> that >> >>> >> >> >>> >>>>>requires >> >>> >> >> >>> >>>>> >>a >> >>> >> >> >>> >>>>> >> >>new >> >>> >> >> >>> >>>>> >> >> > tag >> >>> >> >> >>> >>>>> >> >> > >>>>and >> >>> >> >> >>> >>>>> >> >> > >>>> release. You can check out the bug on t= he >> >>> issue >> >>> >> >> >>>tracker >> >>> >> >> >>> >>>>>at >> >>> >> >> >>> >>>>> >> >> > >>>> >> >>> >> >> >>> >>>>> >> >> > >>>> >> >>> >> >> >>> >>>>> >> >> > >> >>> >> >> >>> >>>>> >> >> >> >>> >> >> >>> >>>>> >> >> >> >>> >> >> >>> >>>>> >> >> >>> >> >> >>> >>>>> >> >> >>> >> >> >>> >>>>> >> >>> >> >> >>> >>>>> >> >>> >> >> >>> >> >>> >> >> >> >>> >> https://issues.apache.org/jira/browse/CB-490?focusedCommentId=3D13252809= # >> >>> >> >> >>> >>>>>co >> >>> >> >> >>> >>>>> >> >> > >>>>mment-13252809 >> >>> >> >> >>> >>>>> >> >> > >>>> >> >>> >> >> >>> >>>>> >> >> > >>>> It doesn't make sense to just repackage >> >>>1.6.0 >> >>> >> with >> >>> >> >> >>>this >> >>> >> >> >>> >>>>>bug >> >>> >> >> >>> >>>>> >> >>fixed as >> >>> >> >> >>> >>>>> >> >> > we >> >>> >> >> >>> >>>>> >> >> > >>>> will have no way to know if people are >> >>> >> experiencing >> >>> >> >> >>>this >> >>> >> >> >>> >>>>>or >> >>> >> >> >>> >>>>> >>not. >> >>> >> >> >>> >>>>> >> >> > >>>> >> >>> >> >> >>> >>>>> >> >> > >>>> Anis mentioned Bada also had some major >> >>>issues >> >>> >> >> >>>which he >> >>> >> >> >>> >>>>>has >> >>> >> >> >>> >>>>> >>fixed >> >>> >> >> >>> >>>>> >> >> now >> >>> >> >> >>> >>>>> >> >> > >>>>and >> >>> >> >> >>> >>>>> >> >> > >>>> would like in a 1.6.1 release. >> >>> >> >> >>> >>>>> >> >> > >>>> >> >>> >> >> >>> >>>>> >> >> > >>>> Thoughts? >> >>> >> >> >>> >>>>> >> >> > >>>> >> >>> >> >> >>> >>>>> >> >> > >>>> -Steve >> >>> >> >> >>> >>>>> >> >> > >> >> >>> >> >> >>> >>>>> >> >> > >> >>> >> >> >>> >>>>> >> >> >> >>> >> >> >>> >>>>> >> >> >>> >> >> >>> >>>>> >> >> >>> >> >> >>> >>>>> >> >>> >> >> >>> >>>>> >> >>> >> >> >>> >>> >> >>> >> >> >>> > >> >>> >> >> >>> >> >>> >> >> >> >>> >> >> >> >>> >> >> >>> >> > >>