Other than the trademark registered name of Asterisk, I understand when we forked we went to OGI, but is it really needed? Can't we just improve AGI? I'm sure citats wouldn't mind changes and improvements to the Asterisk::AGI code?
-wasim On Tue, Mar 4, 2008 at 5:06 PM, Waba <[EMAIL PROTECTED]> wrote: > Hello, > > First of all, I upgraded callweaver-perl using asterisk-perl 0.10 (it > was based on 0.09). I'd appreciate some feedback on this (ticket #343). > > Now I'm working on a partial rewrite of CallWeaver::OGI - using CPAN > modules and automethods, I've cut the code size by 50%, improved its > readability and grown the documentation to 200% for good balance :-) > > I've still got to write a real testsuite and I'll be ready to send a > patch. However, I'm wondering if this is really the best course of > action. This will definitively fork the Asterisk::AGI module, and > without contacting the original author first I'm not sure it is even > needed. > > Indeed, as far as I can tell the only difference between Asterisk::AGI > and CallWeaver::OGI is the prefix used to pass variables around > (agi_$name: $value vs. ogi_$name: $value). This could easily be > configured at runtime rather than through a complete fork. > > Has someone tried to contact James Golovich about it? Should I? While > waiting for your answers I'll get that testsuite done - my work would > fit just as well in a unified Asterisk::AGI than in CallWeaver::OGI. > > -Waba. > > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1.4.6 (GNU/Linux) > > iD8DBQFHzTss3FzfxAeN+kERAp6LAJ99Zx23+/BRivXvlw3kLf1WFOgyXgCfd2bA > YDNtBIeh/IdQQIl1ZtsIbpM= > =oT5T > -----END PGP SIGNATURE----- > > _______________________________________________ > Callweaver-users mailing list > [email protected] > http://lists.callweaver.org/mailman/listinfo/callweaver-users > > -- wasim h. baig | principal consultant | convergence pk | +92 300 8508070
_______________________________________________ Callweaver-users mailing list [email protected] http://lists.callweaver.org/mailman/listinfo/callweaver-users
