Hey everyone, Can folk hold off on committing to cordova-cli until Braden sorts out the branch merging?
IM LOOKING AT YOU, BENN! ;) On 5/23/13 11:15 AM, "Jesse" <purplecabb...@gmail.com> wrote: >I agree. > >@purplecabbage >risingj.com > > >On Thu, May 23, 2013 at 11:10 AM, Andrew Grieve ><agri...@chromium.org>wrote: > >> Given that our goal was to cut Tuesday, I think we should just stick >>with >> the current code for now. I don't think there's any real reason to >>change >> this code at the last second. >> >> >> On Thu, May 23, 2013 at 1:40 PM, Braden Shepherdson <bra...@chromium.org >> >wrote: >> >> > I just merged the git_refs changes. They shouldn't affect anything >>else. >> > >> > >> > On Thu, May 23, 2013 at 1:35 PM, Filip Maj <f...@adobe.com> wrote: >> > >> > > SORRY false alarm: not true. Not merged into plugman yet. On a >>branch. >> > > >> > > On 5/23/13 10:28 AM, "Jesse" <purplecabb...@gmail.com> wrote: >> > > >> > > >So are we going with the tagged version? >> > > >Should I push off the plugin-loader changes until 2.9.0? >> > > > >> > > >@purplecabbage >> > > >risingj.com >> > > > >> > > > >> > > >On Thu, May 23, 2013 at 10:11 AM, Steven Gill >> > > ><stevengil...@gmail.com>wrote: >> > > > >> > > >> Sweet. I will have to take a look at these new changes to coho >>later >> > > >>today. >> > > >> >> > > >> >> > > >> On Wed, May 22, 2013 at 9:40 PM, Andrew Grieve < >> agri...@chromium.org> >> > > >> wrote: >> > > >> >> > > >> > Okay, had to fix some things in the coho script (which I will >>push >> > > >> > tomorrow). >> > > >> > >> > > >> > Parent bug is here: >>https://issues.apache.org/jira/browse/CB-3464 >> > > >> > >> > > >> > >> > > >> > On Wed, May 22, 2013 at 9:53 PM, Andrew Grieve < >> > agri...@chromium.org> >> > > >> > wrote: >> > > >> > >> > > >> > > Awesome stuff! Getting cutting now. >> > > >> > > >> > > >> > > >> > > >> > > On Wed, May 22, 2013 at 5:09 PM, Jesse >><purplecabb...@gmail.com >> > >> > > >> wrote: >> > > >> > > >> > > >> > >> Fixed CB-3307 for WP7, WP8, and Windows 8; >> > > >> > >> >> > > >> > >> @purplecabbage >> > > >> > >> risingj.com >> > > >> > >> >> > > >> > >> >> > > >> > >> On Wed, May 22, 2013 at 1:46 PM, Shazron <shaz...@gmail.com> >> > > wrote: >> > > >> > >> >> > > >> > >> > Fixed the OS X issue (CB-3443). >> > > >> > >> > >> > > >> > >> > >> > > >> > >> > On Wed, May 22, 2013 at 1:13 PM, Andrew Grieve < >> > > >> agri...@chromium.org> >> > > >> > >> > wrote: >> > > >> > >> > >> > > >> > >> > > LGTM so long as you've tested that the error even >>actually >> > > >>fires >> > > >> (I >> > > >> > >> know >> > > >> > >> > it >> > > >> > >> > > didn't use to on some older browsers). >> > > >> > >> > > >> > > >> > >> > > The other thing still outstanding is >> > > >> > >> > > https://issues.apache.org/jira/browse/CB-3307 for WP, >>BB, >> > OSX. >> > > >> > >> > > >> > > >> > >> > > I suppose we could branch without that done, but it's >>such >> a >> > > >>small >> > > >> > >> > change, >> > > >> > >> > > it'd be great to get that fixed before tagging. >> > > >> > >> > > >> > > >> > >> > > >> > > >> > >> > > This time around I'd like to try without using the >> automated >> > > >> script >> > > >> > >> that >> > > >> > >> > > creates issues since I think they capture only a small >> number >> > > >>of >> > > >> the >> > > >> > >> > steps, >> > > >> > >> > > and instead I'll create a master bug and one sub-bug per >> > repo. >> > > >> Then, >> > > >> > >> > let's >> > > >> > >> > > follow the updated wiki steps as best as possible: >> > > >> > >> > > >> > > >> > >> > > http://wiki.apache.org/cordova/CuttingReleases >> > > >> > >> > > >> > > >> > >> > > and just keep adding comments to the JIRA bugs whenever >> > someone >> > > >> does >> > > >> > >> any >> > > >> > >> > of >> > > >> > >> > > the steps. >> > > >> > >> > > >> > > >> > >> > > I'm out to a movie tonight, but will be back 9ish EST. >>I'll >> > > >>look >> > > >> to >> > > >> > >> > branch >> > > >> > >> > > & tag then. >> > > >> > >> > > >> > > >> > >> > > >> > > >> > >> > > On Wed, May 22, 2013 at 2:22 PM, Jesse >> > > >><purplecabb...@gmail.com> >> > > >> > >> wrote: >> > > >> > >> > > >> > > >> > >> > > > I am satisfied with the added error checking. >> > > >> > >> > > > Please review the changes to plugin loading [1] >> > > >> > >> > > > >> > > >> > >> > > > If possible I would like this in 2.8.0rc1, but perhaps >> more >> > > >> > >> > importantly, >> > > >> > >> > > we >> > > >> > >> > > > should do 2.8.0rc1 today. >> > > >> > >> > > > >> > > >> > >> > > > Cheers, >> > > >> > >> > > > Jesse >> > > >> > >> > > > >> > > >> > >> > > > [1] >>https://github.com/purplecabbage/cordova-js/tree/PL >> > > >> > >> > > > >> > > >> > >> > > > >> > > >> > >> > > > @purplecabbage >> > > >> > >> > > > risingj.com >> > > >> > >> > > > >> > > >> > >> > > > >> > > >> > >> > > > On Tue, May 21, 2013 at 5:34 PM, Jesse >> > > >><purplecabb...@gmail.com >> > > >> > >> > > >> > >> > wrote: >> > > >> > >> > > > >> > > >> > >> > > > > the error checking I am adding/testing right now is >> > > >>handling >> > > >> > >> onerror >> > > >> > >> > > > which >> > > >> > >> > > > > does fire when the file is missing. >> > > >> > >> > > > > >> > > >> > >> > > > > @purplecabbage >> > > >> > >> > > > > risingj.com >> > > >> > >> > > > > >> > > >> > >> > > > > >> > > >> > >> > > > > On Tue, May 21, 2013 at 5:25 PM, Andrew Grieve < >> > > >> > >> agri...@chromium.org >> > > >> > >> > > > >wrote: >> > > >> > >> > > > > >> > > >> > >> > > > >> I like the change I think, but I think it will >>fail if >> > > >>users >> > > >> > are >> > > >> > >> > > missing >> > > >> > >> > > > >> the cordova_plugins.js file (onload will never >>fire). >> On >> > > >>the >> > > >> > plus >> > > >> > >> > side >> > > >> > >> > > > >> though, this change makes it easier to combine >> > cordova.js >> > > >>and >> > > >> > >> > > > >> cordova_plugins.js by concatenating them together. >> > > >> > >> > > > >> >> > > >> > >> > > > >> Maybe we should just go all the way with this and >> > combine >> > > >>the >> > > >> > two >> > > >> > >> > .js >> > > >> > >> > > > >> files >> > > >> > >> > > > >> into one right from the start? Then we won't have >>to >> > worry >> > > >> > about >> > > >> > >> > > loading >> > > >> > >> > > > >> at >> > > >> > >> > > > >> all. >> > > >> > >> > > > >> >> > > >> > >> > > > >> >> > > >> > >> > > > >> On Tue, May 21, 2013 at 8:03 PM, Jesse < >> > > >> > purplecabb...@gmail.com> >> > > >> > >> > > wrote: >> > > >> > >> > > > >> >> > > >> > >> > > > >> > Sorry for jumping in late ... >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > On Friday Fil and I tested a workaround for >> > > >>plugin_loader's >> > > >> > >> use of >> > > >> > >> > > XHR >> > > >> > >> > > > >> to >> > > >> > >> > > > >> > load plugins. >> > > >> > >> > > > >> > The gist is that instead of XHR load, it would >>first >> > > >>write >> > > >> a >> > > >> > >> > script >> > > >> > >> > > > tag >> > > >> > >> > > > >> for >> > > >> > >> > > > >> > the cordova_plugins.js file, which defines the >>list >> of >> > > >> > plugins >> > > >> > >> ( >> > > >> > >> > > much >> > > >> > >> > > > >> the >> > > >> > >> > > > >> > same way that cordova_plugins.json was ) then the >> > plugin >> > > >> list >> > > >> > >> was >> > > >> > >> > > > >> processed >> > > >> > >> > > > >> > in exactly the same way. >> > > >> > >> > > > >> > This change affects all cordova-js platforms as >>well >> > as >> > > >> > >> > > > cordova-plugman, >> > > >> > >> > > > >> > and I would like to get it into 2.8.0rc1 if I can >> have >> > > >>the >> > > >> > next >> > > >> > >> > > couple >> > > >> > >> > > > >> > hours to do so. >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > On Friday we verified that it worked as expected >>for >> > > >>iOS, >> > > >> > >> Android, >> > > >> > >> > > > >> > BlackBerry, WP7 and WP8. >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > The small change to cordova-js can be seen here >>[1] >> > > >>minus >> > > >> > some >> > > >> > >> > extra >> > > >> > >> > > > >> error >> > > >> > >> > > > >> > checking. >> > > >> > >> > > > >> > And the change to plugman [2] >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > [1] >> > https://github.com/purplecabbage/cordova-js/tree/PL >> > > >> > >> > > > >> > [2] >> https://github.com/apache/cordova-plugman/tree/pl >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > This fix puts in a better position for >>dynamically >> > > >>loading >> > > >> > >> plugins >> > > >> > >> > > > going >> > > >> > >> > > > >> > forward, although that work is ongoing in the >>3.0.0 >> > > >>branch. >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > Just a little background on the issue: >> > > >> > >> > > > >> > Currently Windows Phone 7 and 8 immediately fail >>to >> > load >> > > >> > >> anything >> > > >> > >> > > > local >> > > >> > >> > > > >> via >> > > >> > >> > > > >> > XHR because of a misinformed security policy that >> > > >>prevents >> > > >> > it. >> > > >> > >> > > After >> > > >> > >> > > > >> > deviceready any local XHR requests are serviced >>by >> the >> > > >>File >> > > >> > API >> > > >> > >> > and >> > > >> > >> > > an >> > > >> > >> > > > >> XHR >> > > >> > >> > > > >> > shim. >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > @purplecabbage >> > > >> > >> > > > >> > risingj.com >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > On Tue, May 21, 2013 at 4:47 PM, Bryan Higgins < >> > > >> > >> > > > bhigg...@blackberry.com >> > > >> > >> > > > >> > >wrote: >> > > >> > >> > > > >> > >> > > >> > >> > > > >> > > Both of those BB10 issues have now been >>resolved. >> > > >> > >> > > > >> > > >> > > >> > >> > > > >> > > >> > > >> > >> > > > >> > > On Tue, May 21, 2013 at 1:09 PM, Bryan Higgins >>< >> > > >> > >> > > > >> br...@bryanhiggins.net >> > > >> > >> > > > >> > > >wrote: >> > > >> > >> > > > >> > > >> > > >> > >> > > > >> > > > There are a couple of BB10 related issues I'd >> like >> > > >>to >> > > >> get >> > > >> > >> in: >> > > >> > >> > > > >> > > > https://issues.apache.org/jira/browse/CB-3434 >> > > >> > >> > > > >> > > > https://issues.apache.org/jira/browse/CB-3415 >> > > >> > >> > > > >> > > > >> > > >> > >> > > > >> > > > I should be able to complete both by EOD. >> > > >> > >> > > > >> > > > >> > > >> > >> > > > >> > > > >> > > >> > >> > > > >> > > > On Tue, May 21, 2013 at 1:01 PM, Michael >>Brooks >> < >> > > >> > >> > > > >> > > mich...@michaelbrooks.ca>wrote: >> > > >> > >> > > > >> > > > >> > > >> > >> > > > >> > > >> > >> > > >> > >> > > > >> > > >> > The one thing that I'd like to see >>finished >> up >> > is >> > > >> > >> > > > >> > > >> > >> > > >> https://issues.apache.org/jira/browse/CB-3307(Renaming >> > > >> > >> > > > >> > > >> cordova.VERSION.js >> > > >> > >> > > > >> > > >> > -> cordova.js). It's done for iOS & >>Android, >> > but >> > > >> still >> > > >> > >> > needs >> > > >> > >> > > to >> > > >> > >> > > > >> be >> > > >> > >> > > > >> > > done >> > > >> > >> > > > >> > > >> for >> > > >> > >> > > > >> > > >> > other platforms. I'll create sub-tasks for >> them >> > > >>now. >> > > >> > >> > > > >> > > >> >> > > >> > >> > > > >> > > >> >> > > >> > >> > > > >> > > >> Good catch Andrew. >> > > >> > >> > > > >> > > >> >> > > >> > >> > > > >> > > >> I've also created tasks for updating the >>Hello >> > > >>World >> > > >> and >> > > >> > >> CLI >> > > >> > >> > as >> > > >> > >> > > > >> well. >> > > >> > >> > > > >> > > I'll >> > > >> > >> > > > >> > > >> try to have these finished by EOD. >> > > >> > >> > > > >> > > >> >> > > >> > >> > > > >> > > >> >> > > >> > >> > > > >> > > >> On Tue, May 21, 2013 at 9:13 AM, Filip Maj < >> > > >> > f...@adobe.com >> > > >> > >> > >> > > >> > >> > > > wrote: >> > > >> > >> > > > >> > > >> >> > > >> > >> > > > >> > > >> > Yes good call Andrew definitely need that >> done >> > > >> across >> > > >> > >> > > platforms >> > > >> > >> > > > >> > before >> > > >> > >> > > > >> > > >> we >> > > >> > >> > > > >> > > >> > ship another release. >> > > >> > >> > > > >> > > >> > >> > > >> > >> > > > >> > > >> > On 5/21/13 9:11 AM, "Andrew Grieve" < >> > > >> > >> agri...@chromium.org> >> > > >> > >> > > > >> wrote: >> > > >> > >> > > > >> > > >> > >> > > >> > >> > > > >> > > >> > >Checking back in to see if anyone has >> anything >> > > >> > they're >> > > >> > >> > > trying >> > > >> > >> > > > to >> > > >> > >> > > > >> > get >> > > >> > >> > > > >> > > in >> > > >> > >> > > > >> > > >> > >for >> > > >> > >> > > > >> > > >> > >2.8. >> > > >> > >> > > > >> > > >> > > >> > > >> > >> > > > >> > > >> > >If no one has anything by the EOD, then >>I'll >> > > >>take >> > > >> > care >> > > >> > >> of >> > > >> > >> > > > >> > branching & >> > > >> > >> > > > >> > > >> > >tagging & JIRA'ing this time around. I've >> been >> > > >> adding >> > > >> > >> > > > branching >> > > >> > >> > > > >> & >> > > >> > >> > > > >> > > >> tagging >> > > >> > >> > > > >> > > >> > >powers to coho, so want to test out the >> > changes >> > > >> this >> > > >> > >> > release >> > > >> > >> > > > :). >> > > >> > >> > > > >> > > >> > > >> > > >> > >> > > > >> > > >> > >The one thing that I'd like to see >>finished >> up >> > > >>is >> > > >> > >> > > > >> > > >> > > >> > > >> > https://issues.apache.org/jira/browse/CB-3307(Renaming >> > > >> > >> > > > >> > > >> > cordova.VERSION.js >> > > >> > >> > > > >> > > >> > >-> cordova.js). It's done for iOS & >>Android, >> > but >> > > >> > still >> > > >> > >> > needs >> > > >> > >> > > > to >> > > >> > >> > > > >> be >> > > >> > >> > > > >> > > done >> > > >> > >> > > > >> > > >> > >for >> > > >> > >> > > > >> > > >> > >other platforms. I'll create sub-tasks >>for >> > them >> > > >> now. >> > > >> > >> > > > >> > > >> > > >> > > >> > >> > > > >> > > >> > > >> > > >> > >> > > > >> > > >> > > >> > > >> > >> > > > >> > > >> > >On Fri, May 17, 2013 at 7:29 PM, Steven >> Gill < >> > > >> > >> > > > >> > stevengil...@gmail.com >> > > >> > >> > > > >> > > > >> > > >> > >> > > > >> > > >> > >wrote: >> > > >> > >> > > > >> > > >> > > >> > > >> > >> > > > >> > > >> > >> Sounds good to me. >> > > >> > >> > > > >> > > >> > >> >> > > >> > >> > > > >> > > >> > >> >> > > >> > >> > > > >> > > >> > >> On Fri, May 17, 2013 at 2:48 PM, Filip >> Maj < >> > > >> > >> > f...@adobe.com >> > > >> > >> > > > >> > > >> > >> > > > >> > wrote: >> > > >> > >> > > > >> > > >> > >> >> > > >> > >> > > > >> > > >> > >> > +1! >> > > >> > >> > > > >> > > >> > >> > >> > > >> > >> > > > >> > > >> > >> > On 5/17/13 2:44 PM, "Jesse" < >> > > >> > >> purplecabb...@gmail.com> >> > > >> > >> > > > >> wrote: >> > > >> > >> > > > >> > > >> > >> > >> > > >> > >> > > > >> > > >> > >> > >Monday is a holiday in Canada, so I >> > > >>propose we >> > > >> > >> aim to >> > > >> > >> > > do >> > > >> > >> > > > >> our >> > > >> > >> > > > >> > rc1 >> > > >> > >> > > > >> > > >> > >>ritual >> > > >> > >> > > > >> > > >> > >> on >> > > >> > >> > > > >> > > >> > >> > >Tuesday, May 14th >> > > >> > >> > > > >> > > >> > >> > >This still gives us a little time to >> > close >> > > >>the >> > > >> > >> loop >> > > >> > >> > on >> > > >> > >> > > > any >> > > >> > >> > > > >> > > >> > >>outstanding >> > > >> > >> > > > >> > > >> > >> > >items, and lots of time to test. >> > > >> > >> > > > >> > > >> > >> > >Thoughts? >> > > >> > >> > > > >> > > >> > >> > > >> > > >> > >> > > > >> > > >> > >> > >@purplecabbage >> > > >> > >> > > > >> > > >> > >> > >risingj.com >> > > >> > >> > > > >> > > >> > >> > >> > > >> > >> > > > >> > > >> > >> > >> > > >> > >> > > > >> > > >> > >> >> > > >> > >> > > > >> > > >> > >> > > >> > >> > > > >> > > >> > >> > > >> > >> > > > >> > > >> >> > > >> > >> > > > >> > > > >> > > >> > >> > > > >> > > > >> > > >> > >> > > > >> > > >> > > >> > >> > > > >> > >> > > >> > >> > > > >> >> > > >> > >> > > > > >> > > >> > >> > > > > >> > > >> > >> > > > >> > > >> > >> > > >> > > >> > >> > >> > > >> > >> >> > > >> > > >> > > >> > > >> > > >> > >> > > >> >> > > >> > > >> > >>