Re: want to discuss - mvc extension

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. > -nathan 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.. git-patch is sent off-list 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 -- Alexey Zakhlestin http://www.milkfarmsoft.com/ Sent from St Petersburg, Russian Federation