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 bowserj@gmail.com designates
 74.125.82.175 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAMT1KGPAh9ORkd-EW_mgjk6DLOYygnHgAAZFLVg83u7oPxSbMg@mail.gmail.com>
References: 
 <CAOBL_k6ibngSu5awm15qFoez-GWjNkwt3sGC-_7jotQhTWrS6Q@mail.gmail.com>
	<CC09FB29.8B23%fil@adobe.com>
	<CAMT1KGPAh9ORkd-EW_mgjk6DLOYygnHgAAZFLVg83u7oPxSbMg@mail.gmail.com>
Date: Fri, 22 Jun 2012 12:35:13 -0700
Message-ID: 
 <CAOBL_k4NWW=9GVCZ47RBL2t2xN_s2U0rXj5-C7pDt4aA9kX6Jg@mail.gmail.com>
Subject: Re: Apache Cordova Android does not compile
From: Joe Bowser <bowserj@gmail.com>
To: callback-dev@incubator.apache.org
Content-Type: multipart/alternative; boundary=f46d04428b5028a42a04c314bbe6

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

The thing is that it worked yesterday on my machine.  I do think that
mistakes will still happen and I'd rather it happen on edge or preferably
on a tag.

Honestly, I admitted that I screwed up Bryce, there is no need to be so
passive-aggressive.
On Jun 22, 2012 12:01 PM, "Bryce Curtis" <curtis.bryce@gmail.com> wrote:

> 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.  I 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
> >>> >>
> >>>
> >>>
> >
>

--f46d04428b5028a42a04c314bbe6--