These are usually easy to fix by moving one of the require()s one nesting deeper.
That said, it might be worth just rolling the CL back for now and then rolling it forward after the tagging. On Fri, Aug 31, 2012 at 3:37 PM, Filip Maj <f...@adobe.com> wrote: > Gord is this a showstopper? > > On 8/31/12 11:58 AM, "Shazron" <shaz...@gmail.com> wrote: > > >So, what does this mean. Should all platforms hold? > > > >On Fri, Aug 31, 2012 at 11:44 AM, Filip Maj <f...@adobe.com> wrote: > >> Oh balls. Just tagged 2.1.0rc2 > >> > >> On 8/31/12 11:35 AM, "Gord Tanner" <gtan...@gmail.com> wrote: > >> > >>>This is currently breaking tizen and File API's > >>> > >>>Should we remove this and push to 2.2 to give people time to clean this > >>>up? > >>> > >>>On Mon, Aug 20, 2012 at 2:06 PM, Andrew Grieve <agri...@chromium.org> > >>>wrote: > >>>> Done and done. > >>>> > >>>> > >>>> > https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-js.git;a=co > >>>>mm > >>>>it;h=188232f42e60745c961363638560ad3c41b6590c > >>>> > >>>> > >>>> On Mon, Aug 20, 2012 at 1:24 PM, Patrick Mueller <pmue...@gmail.com> > >>>>wrote: > >>>> > >>>>> On Mon, Aug 20, 2012 at 1:12 PM, Andrew Grieve <agri...@google.com> > >>>>>wrote: > >>>>> > >>>>> > ... > >>>>> > I think these restrictions are too hard to get right, and that we > >>>>>should > >>>>> > just make require cycles an error. Objections? > >>>>> > > >>>>> > >>>>> +1, I think this should be containable for us. > >>>>> > >>>>> -- > >>>>> Patrick Mueller > >>>>> http://muellerware.org > >>>>> > >> > >
