Re: want to discuss - mvc extension

On Mon, Aug 10, 2009 at 11:51 PM, Alexey Zakhlestin <indeyets@gmail.com>wrote: > On Tue, Aug 11, 2009 at 3:15 AM, Nathan Nobbe<quickshiftin@gmail.com> > wrote: > > On Mon, Aug 10, 2009 at 4:03 PM, Kalle Sommer Nielsen <kalle@php.net> > wrote: > >> > >> Hi Nathan > >> > >> 2009/8/10 Nathan Nobbe <quickshiftin@gmail.com>: > >> > On Mon, Aug 10, 2009 at 12:31 PM, Alexey Zakhlestin > >> > <indeyets@gmail.com>wrote: > >> > > >> > ive uploaded the source to github, since thats what im using to > version > >> > the > >> > code myself. and before a flame war starts, im more than happy to > drop > >> > the > >> > code into svn to host here :D > >> > > >> > > >> > http://github.com/quickshiftin/PureMVC-ext/tree/master > >> > >> Is this extension 5.3 compatible? Because I get many compilation > >> errors when trying to build under 5.3.1-dev from SVN: > >> http://pastie.org/579173 > >> > >> Ps. Remember to update config.w32 so you can enable the extension :) > > > > the first email (from the original proposal) you can see where im at w/ > the > > extension and the future roadmap; here is the list again: > > > > goals for a promotion to beta would roughly include, > > . address current list of todos (small) > > . build & test on windows platforms > > . build on additional arch / os > > . create an additional branch for 5.3 and add namespaces to the classes > in > > that branch > > . generate/write documentation following > > http://wiki.php.net/doc/scratchpad/pecldocs > > . pureMVC multi-core support (pending further research) > > > > nothing much has changed here since my original proposal :( what i was > > working on for a little bit was a thin application framework around > pureMVC, > > w/ a class loader that supports modular class naming conventions such as > the > > one used by zend framework for php 5.2, and one that caters more to a 5.3 > > naming convention since namespaces are available. > > im glad to touch it up though to get it inducted into pecl. ill get it > to > > build under 5.3 and also, install windows in some sort of vm so that i > can > > test compilation there. my original hope was to do this post induction > to > > pecl (5.3 having not yet been release at the time of the original > proposal), > > but it does seem that it should be a pre-requisite now that 5.3 has > reached > > prime time. > > I guess 5.3 compatibility can be added "post factum" > But there are more problems: > 1) Code is, actually, not compatible with 5.2 in ZTS-mode also.. > Attaching a git-patch to fix this > 2) There are many memory-leak reports (these are reported when "debug" > mode is enabled) > > it is considered to be a good idea to develop extensions using > zst-debug mode of php. If it works this way, it will still work in > non-zts-non-debug mode too. > > 3) Code is not strictly following php's coding standards > > http://svn.php.net/viewvc/php/php-src/trunk/CODING_STANDARDS?view=markup > > > Apply patch for #1 and fix #3. > #2 can wait until in svn-tree thanks, alexey, ive added the patch and pushed it up to github. ill address the coding standard issues tonight and start working on the memory leak issues. -nathan