Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of agrieve@google.com designates
 209.85.214.175 as permitted sender)
MIME-Version: 1.0
Sender: agrieve@google.com
In-Reply-To: <CC357C2B.AC46%fil@adobe.com>
References: 
 <CABiQX1Vjt782JMYn+kc_bwisChVduqchDsstmCO9_E5nqry7KA@mail.gmail.com>
 <CC357C2B.AC46%fil@adobe.com>
From: Andrew Grieve <agrieve@chromium.org>
Date: Fri, 27 Jul 2012 21:24:00 -0400
Message-ID: 
 <CABiQX1VWbcAmvVig31Ubj7dGX5aPMJpjBgDNQjh4kUUZdLYnVA@mail.gmail.com>
Subject: Re: Adding headers to FileTransfer.upload
To: callback-dev@incubator.apache.org
Content-Type: multipart/alternative; boundary=e89a8ff250ee2b75a604c5d9b082

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

Okay. Submitted. The new mobile-spec test fails on Android currently
because the test server it uses requires the Content-Length header to be
set.

I have a change to address that (
https://github.com/apache/incubator-cordova-android/pull/32), but I need to
ensure that it doesn't increase memory usage (the JavaDoc suggests that it
doesn't, but I'll double check via Runtime.getRuntime().freeMemory() when I
get a chance.


On Wed, Jul 25, 2012 at 1:34 PM, Filip Maj <fil@adobe.com> wrote:

> Go for it mang.
>
> Any issues with access or whatnot?
>
> On 7/25/12 8:48 AM, "Andrew Grieve" <agrieve@google.com> wrote:
>
> >Fil, do you want to go ahead with these now? Or just wait until I figure
> >out how to commit things directly and I can put them in?
> >
> >
> >On Mon, Jul 16, 2012 at 3:14 PM, Andrew Grieve <agrieve@google.com>
> wrote:
> >
> >> Great, sounds good.
> >>
> >>
> >> On Mon, Jul 16, 2012 at 3:00 PM, Simon MacDonald <
> >> simon.macdonald@gmail.com> wrote:
> >>
> >>> +1
> >>>
> >>> wait until 2.0.0 is out then push the change.
> >>>
> >>> Simon Mac Donald
> >>> http://hi.im/simonmacdonald
> >>>
> >>>
> >>> On Mon, Jul 16, 2012 at 3:01 PM, Filip Maj <fil@adobe.com> wrote:
> >>>
> >>> > Andrew,
> >>> >
> >>> > Thanks a lot for this! This is a lot of work and it's great that you
> >>> > tackled this across a couple of platforms, too.
> >>> >
> >>> > Agree with you that since the RC is out for 2.0, let's wait until we
> >>> land
> >>> > 2.0 proper end of this week before we merge your changes into the
> >>> master.
> >>> >
> >>> > Usually, a committer would push these sorts of pending changes to a
> >>> topic
> >>> > branch on the related repositories. We essentially have this already
> >>>in
> >>> > place, the branches simply exist on your GitHub fork :)
> >>> >
> >>> > I'll keep a reference to this mail in my inbox and we'll revisit /
> >>>merge
> >>> > next week. Sound good?
> >>> >
> >>> > On 7/16/12 11:26 AM, "Andrew Grieve" <agrieve@google.com> wrote:
> >>> >
> >>> > >I've got a set of pull requests to do so, and just wanted to make
> >>>sure
> >>> > >everyone knows that they are all related:
> >>> > >
> >>> > >JS: https://github.com/apache/incubator-cordova-js/pull/20
> >>> > >Android Impl:
> >>> https://github.com/apache/incubator-cordova-android/pull/33
> >>> > >iOS Impl: https://github.com/apache/incubator-cordova-ios/pull/35
> >>> > >Mobile-Spec Impl:
> >>> > >https://github.com/apache/incubator-cordova-mobile-spec/pull/9
> >>> > >Docs: https://github.com/apache/incubator-cordova-docs/pull/24
> >>> > >
> >>> > >I didn't update the cordova.js files in the Andoid/iOS changes
> >>>since I
> >>> > >thought maybe they'd change while these are pending, and wasn't sure
> >>> if it
> >>> > >was normal to update the JS on every platform every time
> >>> > >incubator-cordova-js changes.
> >>> > >
> >>> > >Also - I know a 2.0 candidate is now tagged so probably this change
> >>> will
> >>> > >wait until after the release is done? Or does development continue
> >>>on
> >>> and
> >>> > >changes are cherry-picked into a release branch?
> >>> > >
> >>> > >Andrew
> >>> >
> >>> >
> >>>
> >>
> >>
>
>

--e89a8ff250ee2b75a604c5d9b082--