Re: [PATCH] (sqlbox) sqlbox.patch

Stipe Tolj <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Organization tolj.org system architecture
Message-ID <[email protected]>
Hi Martin,

Mi Reflejo wrote:

> yes, but i don't think that sqlbox can be moved out of Kannel since
> some modification to smsbox need to be made.
> ***

that's a problem indeed. AFAIU the smsbox connects to the sqlbox rather then the 
bearerbox port. So sqlbox acts (from the view of smsbox) as bearerbox.

Ok, that's why you guys need to seperate and add the config. But I think you 
should use a different .conf file for sqlbox, where the "normal bearerbox port" 
is set addequate to reflect the sqlbox(bearerbox)-port. This is a benefit, since 
it doesn't require to patch smsbox at all. Right?

So, using 2 configs with core groups different for bearerbox port would solve 
the smsbox patching need.

> I have seen some very old mysql_config returning '-L/path
> -lmysqlclient -letc' (With single quotes).
> But i think that isn't necessary anymore.

yep, seems older versions of mysql did this nasty replying output for 
mysql-config --libs

> well.
> Actually yes, sqlbox acts towards smsbox like a bearerbox.
> 
> if bearerbox-is-sqlbox flag is true, smsbox-port is fetched from "sqlbox 
> group".
> (sqlbox listen on that port like bearerbox does and connect to
> bearerbox [smsbox-port from "core group"])

see comments and suggestion above.

>> * sqlbox should have it's own module gwlib/sqlbox-cfg.def and call the
>> cfg_add_hooks() routine I proposed to "load" the semantic checks for 
>> the new
>> config group.
>>
> Yes, but sqlbox need to be changed. I'll think how to do it.

Should be pretty simple thou.

Stipe

-------------------------------------------------------------------
Kölner Landstrasse 419
40589 Düsseldorf, NRW, Germany

tolj.org system architecture      Kannel Software Foundation (KSF)
http://www.tolj.org/              http://www.kannel.org/

mailto:st_{at}_tolj.org           mailto:stolj_{at}_kannel.org
-------------------------------------------------------------------
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.