Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of curtis.bryce@gmail.com
 designates 209.85.213.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CC09FB29.8B23%fil@adobe.com>
References: 
 <CAOBL_k6ibngSu5awm15qFoez-GWjNkwt3sGC-_7jotQhTWrS6Q@mail.gmail.com>
	<CC09FB29.8B23%fil@adobe.com>
Date: Fri, 22 Jun 2012 14:00:39 -0500
Message-ID: 
 <CAMT1KGPAh9ORkd-EW_mgjk6DLOYygnHgAAZFLVg83u7oPxSbMg@mail.gmail.com>
Subject: Re: Apache Cordova Android does not compile
From: Bryce Curtis <curtis.bryce@gmail.com>
To: callback-dev@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

We all need to be thorough when contributing code.  It should be
standard practice to verify that any code checked in compiles and
resolves the issue without regression.

On Fri, Jun 22, 2012 at 12:36 PM, Filip Maj <fil@adobe.com> wrote:
> Because I did not finish running the tests at that moment. I could have
> deleted all the code; that woulda compiled too ;)
>
> Anyways, passes the tests, I've pushed it up to the main repo.
>
> On 6/22/12 10:20 AM, "Joe Bowser" <bowserj@gmail.com> wrote:
>
>>If this fixes compilation, why not push it to master?
>>On Jun 22, 2012 10:08 AM, "Filip Maj" <fil@adobe.com> wrote:
>>
>>> Hye guys,
>>>
>>> I pushed a "buttonfix" branch up to apache. All I did was removed what
>>> looks like unused legacy code from App.java as well as DroidGap.java.
>>>The
>>> hardware key binding is now done in cordovawebview.
>>>
>>> Additionally, the android JS routes a call to override the back button
>>>to
>>> the app plugin, yes, but never actually invokes the "is the back button
>>> bound" method - so it is unneeded. Therefore I removed it.
>>>
>>> I am going through the testing motions now but if someone wants to pull
>>> down that branch and test from their end too that would be helpful.
>>>
>>> On 6/22/12 9:52 AM, "Joe Bowser" <bowserj@gmail.com> wrote:
>>>
>>> >Hey
>>> >
>>> >It should be using the webView.bindBackButton in app, and not the
>>> >other one. =A0I think a merge of the buttons work must have messed up.
>>> >I'll fix it when I get in the office this morning.
>>> >
>>> >Joe
>>> >
>>> >On Fri, Jun 22, 2012 at 9:47 AM, Filip Maj <fil@adobe.com> wrote:
>>> >> 1.8.1 is a branch since cordovawebview was merged into master after
>>> >>1.8.0.
>>> >>
>>> >> Simon I'm looking into the compilation issue.
>>> >>
>>> >> On 6/22/12 9:42 AM, "Brian LeRoux" <b@brian.io> wrote:
>>> >>
>>> >>>also: tags for 1.8.x not on the apache repo
>>> >>>
>>> >>>On Fri, Jun 22, 2012 at 11:38 AM, Simon MacDonald
>>> >>><simon.macdonald@gmail.com> wrote:
>>> >>>> Hey all,
>>> >>>>
>>> >>>> The recent changes to CordovaInterface where isBackButtonBound has
>>> >>>> been removed is causing compilation errors. The App class is still
>>> >>>> trying to call this method. Can someone who's been looking at the
>>> >>>> changes to button binding on Android look into this ASAP?
>>> >>>>
>>> >>>> Simon Mac Donald
>>> >>>> http://hi.im/simonmacdonald
>>> >>
>>>
>>>
>