Re: [PATCH] SQLBox against latest CVS

"Alejandro Guerrieri" <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Message-ID <[email protected]>
Martín,

I've tried it, great work! In fact I'm using the standalone version on
all many of our production servers with no issues whatsoever, but I've
noticed that there are many things that differ between the standalone
and the patch. Since the test I've been running didn't succeed with
the standalone, I've took a chance and tried to compile the patch.
There I've noticed the problem with CVS.

BTW, the differences between the standalone and the patch, besides the
obviously related to the fact that it's an independent beast, are
because the code on the standalone is older? Any chances of updating
it to the latest codebase?

Better yet, any chances of making the standalone the "defacto" sqlbox
and merging it on the main tree? It's quite more practical than
patching the code again and again.

Regards,

Alejandro

On 10/5/07, Reflejo <[email protected]> wrote:
> Maybe you should use stand-alone version. ;)
>
> http://www.kannel.org/~mconte/sqlbox/
>
> Martin Conte
>
>
> On 10/3/07, Alejandro Guerrieri < [email protected]> wrote:
> > I've been trying yo compile sqlbox against latest CVS, and the patches
> > doesn't apply.
> >
> > I've fixed it, here is the patch:
> >
> > Index: sqlbox.patch
> >
> ===================================================================
> > RCS file: /home/cvs/sqlbox/sqlbox.patch,v
> > retrieving revision 1.5
> > diff -u -r1.5 sqlbox.patch
> > --- sqlbox.patch        13 Jun 2006 22:52:45 -0000       1.5
> > +++ sqlbox.patch        3 Oct 2007 13:27:23 -0000
> > @@ -412,15 +412,15 @@
> > diff -u -r1.266 smsbox.c
> > --- gw/smsbox.c 15 May 2006 18:24:40 -0000  1.266
> > +++ gw/smsbox.c 30 May 2006 17:44:18 -0000
> > -@@ -3319,7 +3319,7 @@
> > +@@ -3308,7 +3308,7 @@
> >       Octstr *http_proxy_password = NULL;
> >       Octstr *http_proxy_exceptions_regex = NULL;
> >       int ssl = 0;
> > -    int lf, m;
> > +    int lf, m, bearerbox_is_sqlbox = 0;
> > +     long max_req;
> >
> > -     bb_port = BB_DEFAULT_SMSBOX_PORT;
> > -     bb_ssl = 0;
> > +    bb_port = BB_DEFAULT_SMSBOX_PORT;
> > @@ -3365,6 +3365,18 @@
> >       if (grp == NULL)
> >      panic(0, "No 'smsbox' group in configuration");
> > @@ -452,9 +452,9 @@
> >       OCTSTR(smsbox-id)
> >       OCTSTR(bearerbox-host)
> > +    OCTSTR(bearerbox-is-sqlbox)
> > +     OCTSTR(bearerbox-port)
> > +     OCTSTR(bearerbox-port-ssl)
> >       OCTSTR(sendsms-port)
> > -     OCTSTR(sendsms-port-ssl)
> > -     OCTSTR(sendsms-interface)
> > @@ -556,6 +557,27 @@
> >       OCTSTR(unified-prefix)
> >   )
> >
> > Regards,
> >
> > Alejandro
> >
> > --
> > Alejandro Guerrieri
> > Magicom
> > http://www.magicom-bcn.net/
> > LinkedIn: http://www.linkedin.com/in/aguerrieri
> >
> >
>
>


-- 
Alejandro Guerrieri
Magicom
http://www.magicom-bcn.net/
LinkedIn: http://www.linkedin.com/in/aguerrieri
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.