Ok, if you guys don't mind I'll merge Lift 2.4 branch back into trunk (hopefully this evening).
Vladimir 2011/9/13 Richard Hirsch <[email protected]> > I agree - work in the trunk - I assume that Lift 2.4 should be > finished by the time we release 1.4. > > D > > > On Tue, Sep 13, 2011 at 6:00 PM, Ethan Jewett <[email protected]> wrote: > > Hi Vladimir and all, > > > > Hmmm, I guess it depends on whether or not upgrading is required for > > our 1.4 release. Personally, I think that because of the Textile fixes > > (at least) we should shoot to include it, in which case we could merge > > it back in without waiting. This would result in it getting more > > testing from me, since I at least am developing on trunk and not the > > Lift 2.4 branch. > > > > Thoughts? > > > > Ethan > > > > 2011/9/13 Vladimir Ivanov <[email protected]>: > >> Hi Ethan, > >> > >> Sorry for the late response. > >> > >> I've fixed сompile-time errors and commited changes, but I think we need > >> more time to test it. Moreover I had to remove some members from > MsgParser > >> class because they dublicated definitions in Textile's superclass (in > its > >> new version). Some of them were exactly the same, other were slightly > >> different. So it would be ideal if Vassil find time to review it. > >> > >> I would wait for at least RC, but if you guys decide that it's > neccessary > >> right now - why not? > >> > >> Vladimir > >> > >> 2011/9/11 Ethan Jewett <[email protected]> > >> > >>> Hi Vladimir, > >>> > >>> Seems like we should get the branch merged back in to trunk soon. Do > >>> you think it would be best to do that on the current 2.4M4, or would > >>> it be wiser to wait until later? > >>> > >>> Ethan > >>> > >>> On Sun, Sep 11, 2011 at 3:59 PM, Vladimir Ivanov < > [email protected]> > >>> wrote: > >>> > As Vassil noted, the most important one is Textile update. > >>> > > >>> > Among other fixes I would also mention fix for dublicated context > path > >>> > problem: > >>> > > >>> > http://www.assembla.com/spaces/liftweb/tickets/1082-redirectresponse-duplicates-contextpath > >>> > > >>> > There were many reports and question on StackOverflow regarding this > >>> issue. > >>> > > >>> > Vladimir > >>> > > >>> > 2011/9/10 Vassil Dichev <[email protected]> > >>> > > >>> >> It should fix the long-standing ESME-307 (Textile and punctuation) > >>> >> > >>> >> > >>> >> On Fri, Sep 9, 2011 at 6:03 PM, Richard Hirsch < > [email protected]> > >>> >> wrote: > >>> >> > Thanks for the update. > >>> >> > > >>> >> > Are there any new features in the release that might be of > interest to > >>> >> ESME? > >>> >> > > >>> >> > D. > >>> >> > > >>> >> > On Fri, Sep 9, 2011 at 4:12 PM, Vladimir Ivanov < > >>> [email protected]> > >>> >> wrote: > >>> >> >> Below is an small update on current status for Lift 2.4 branch. > >>> >> >> > >>> >> >> Lift-2.4M4 was announced yesterday: > >>> >> >> > >>> >> > >>> > http://groups.google.com/group/liftweb/browse_thread/thread/f695772c254219b2 > >>> >> >> > >>> >> >> I've updated project files to new version, but it doesn't compile > due > >>> to > >>> >> >> changes in Loc.Snippet class (it's neccessary to explicitly > specify _ > >>> in > >>> >> >> order to compiler recognize that Snippet's parameter is a > function). > >>> See > >>> >> >> following link for more details: > >>> >> >> > >>> >> >> > >>> >> > >>> > http://groups.google.com/group/liftweb/browse_thread/thread/1bf211102812615b/816299dd5a4dbff1?lnk=gst&q=Updated+object+Snippet+apply()+causing#816299dd5a4dbff1 > < > http://groups.google.com/group/liftweb/browse_thread/thread/1bf211102812615b/816299dd5a4dbff1?lnk=gst&q=Updated+object+Snippet+apply%28%29+causing#816299dd5a4dbff1 > > > >>> >> >> > >>> >> >> I suppose this issue is easy to fix, but it will take some time > for > >>> me > >>> >> due > >>> >> >> to extremly high workload (even on this weekend :s( ). > >>> >> >> > >>> >> >> M4 will also make possible moving to 2.9.1 Scala version (after > all > >>> >> >> dependencies are updated accordingly). > >>> >> >> > >>> >> >> Vladimir > >>> >> >> > >>> >> >> 2011/7/27 <[email protected]> > >>> >> >> > >>> >> >>> Going by the content of the discussion on this topic for the > last > >>> >> couple of > >>> >> >>> days, it should be possible for Vladimir to implement the > >>> replacement, > >>> >> if > >>> >> >>> not just port the code from the old Ostrich code base. > >>> >> >>> > >>> >> >>> Hope he finds the time and the inclination! > >>> >> >>> > >>> >> >>> Imtiaz > >>> >> >>> > >>> >> >>> ----- Original Message ----- From: "Richard Hirsch" < > >>> >> [email protected] > >>> >> >>> > > >>> >> >>> To: <[email protected]> > >>> >> >>> Sent: Wednesday, July 27, 2011 8:33 PM > >>> >> >>> Subject: Re: Lift 2.4 and Scala 2.9.x > >>> >> >>> > >>> >> >>> > >>> >> >>> > >>> >> >>> I think they (twitter) decided that the HTTP-based stats was > enough > >>> or > >>> >> >>>> the JMX scala code conflicted with the new Scala version. > >>> >> >>>> > >>> >> >>>> On Wed, Jul 27, 2011 at 4:17 PM, Vladimir Ivanov < > >>> >> [email protected]> > >>> >> >>>> wrote: > >>> >> >>>> > >>> >> >>>>> Yes, I wasn't able to find any replacement too. I also tried > to > >>> find > >>> >> any > >>> >> >>>>> forum or group to ask about this modification, but it seems > >>> there's > >>> >> no > >>> >> >>>>> other > >>> >> >>>>> resources dedicated to this project besides GitHub. > >>> >> >>>>> > >>> >> >>>>> Next step is to find the reason of failed tests. > >>> >> >>>>> > >>> >> >>>> > >>> >> >>>> OK - sounds good > >>> >> >>>> > >>> >> >>>> > >>> >> >>>>> Vladimir > >>> >> >>>>> > >>> >> >>>>> 2011/7/27 Richard Hirsch <[email protected]> > >>> >> >>>>> > >>> >> >>>>> Looked at the code last night and this morning. The JMX > support > >>> has > >>> >> >>>>>> been totally removed. > >>> >> >>>>>> > >>> >> >>>>>> I had a quick look and didn't find anything that might be a > >>> >> >>>>>> replacement. In the meantime, we'll have to just use the > >>> HTTP-based > >>> >> >>>>>> interface. > >>> >> >>>>>> > >>> >> >>>>>> D. > >>> >> >>>>>> > >>> >> >>>>>> On Tue, Jul 26, 2011 at 3:27 PM, Vladimir Ivanov < > >>> >> [email protected] > >>> >> >>>>>> > > >>> >> >>>>>> wrote: > >>> >> >>>>>> > Thank you, Dick! > >>> >> >>>>>> > > >>> >> >>>>>> > Here is the link to Ostrich's GitHub repo changeset with > >>> >> StatsMBean > > >>> >> >>>>>> class > >>> >> >>>>>> > removed: > >>> >> >>>>>> > > >>> >> >>>>>> https://github.com/twitter/**ostrich/commit/** > >>> >> >>>>>> > 5e6e557f0fab77282bc7ccd2113a78**309db908b0#src/main/scala/com/** > >>> >> >>>>>> twitter/ostrich< > >>> >> > >>> > https://github.com/twitter/ostrich/commit/5e6e557f0fab77282bc7ccd2113a78309db908b0#src/main/scala/com/twitter/ostrich > >>> >> > > >>> >> >>>>>> > > >>> >> >>>>>> > May be it helps somehow. > >>> >> >>>>>> > > >>> >> >>>>>> > Vladimir > >>> >> >>>>>> > > >>> >> >>>>>> > 2011/7/26 Richard Hirsch <[email protected]> > >>> >> >>>>>> > > >>> >> >>>>>> >> OK - I'll take a look at it tonight > >>> >> >>>>>> >> > >>> >> >>>>>> >> On Tue, Jul 26, 2011 at 3:12 PM, Vladimir Ivanov >> < > >>> >> >>>>>> [email protected] > >>> >> >>>>>> > > >>> >> >>>>>> >> wrote: > >>> >> >>>>>> >> > Yes, I had to comment it out. > >>> >> >>>>>> >> > > >>> >> >>>>>> >> > 2011/7/26 Richard Hirsch <[email protected]> > >>> >> >>>>>> >> > > >>> >> >>>>>> >> >> Great. > >>> >> >>>>>> >> >> > >>> >> >>>>>> >> >> Let me take a look for the StatsMBean replacement. Did > you > >>> >> comment > >>> >> >>>>>> out > >>> >> >>>>>> >> >> the usage of StatsMBean to get ESME to compile? > >>> >> >>>>>> >> >> > >>> >> >>>>>> >> >> D. > >>> >> >>>>>> >> >> > >>> >> >>>>>> >> >> On Tue, Jul 26, 2011 at 2:45 PM, Vladimir Ivanov < > >>> >> >>>>>> [email protected] > >>> >> >>>>>> >> > > >>> >> >>>>>> >> >> wrote: > >>> >> >>>>>> >> >> > Configgy is not used for configuration anymore > >>> >> >>>>>> >> >> > - Package and method names were changed > >>> >> >>>>>> >> >> > - There's no more StatsMBean class (and I have no > idea > >>> >> whether > >>> >> >>>>>> >> >> > any > >>> >> >>>>>> >> >> > replacement is available). > >>> >> >>>>>> >> >> > >>> >> >>>>>> >> > > >>> >> >>>>>> >> > > >>> >> >>>>>> >> > > >>> >> >>>>>> >> > -- > >>> >> >>>>>> >> > Best Regards, > >>> >> >>>>>> >> > Vladimir Ivanov > >>> >> >>>>>> >> > > >>> >> >>>>>> >> > >>> >> >>>>>> > > >>> >> >>>>>> > > >>> >> >>>>>> > > >>> >> >>>>>> > -- > >>> >> >>>>>> > Best Regards, > >>> >> >>>>>> > Vladimir Ivanov > >>> >> >>>>>> > > >>> >> >>>>>> > >>> >> >>>>>> > >>> >> >>>>> > >>> >> >>>>> > >>> >> >>>>> -- > >>> >> >>>>> Best Regards, > >>> >> >>>>> Vladimir Ivanov > >>> >> >>>>> > >>> >> >>>>> > >>> >> >>> > >>> >> >> > >>> >> >> > >>> >> >> -- > >>> >> >> Best Regards, > >>> >> >> Vladimir Ivanov > >>> >> >> > >>> >> > > >>> >> > >>> > > >>> > > >>> > > >>> > -- > >>> > Best Regards, > >>> > Vladimir Ivanov > >>> > > >>> > >> > >> > >> > >> -- > >> Best Regards, > >> Vladimir Ivanov > >> > > > -- Best Regards, Vladimir Ivanov
