14 Jul
2011
14 Jul
'11
10:08 p.m.
Hi All,
I've seen someone else had a similar problem but there were no responses. On
Solaris, compiling db_mysql results in the following:
CC (gcc) [M db_mysql.so] my_cmd.o
In file included from ../../globals.h:37:0,
from ../../pt.h:42,
from ../../counters.h:43,
from mysql_mod.h:39,
from my_cmd.c:40:
../../ip_addr.h:85:23: error: field 'sin6' has incomplete type
../../ip_addr.h: In function 'ip_addr_loopback':
../../ip_addr.h:281:3: warning: implicit declaration of function 'htonl'
../../ip_addr.h:284:3: warning: implicit declaration of function
'IN6_IS_ADDR_LOOPBACK'
../../ip_addr.h: In function 'sockaddr2ip_addr':
../../ip_addr.h:351:50: error: dereferencing pointer to incomplete type
../../ip_addr.h: In function 'su_getport':
../../ip_addr.h:397:4: warning: implicit declaration of function 'ntohs'
../../ip_addr.h: In function 'su_setport':
../../ip_addr.h:416:4: warning: implicit declaration of function 'htons'
whats puzzling me is that ip_addr.h is included in other modules which
compile perfectly, however in db_mysql, not
feel like I'm missing something really silly? any ideas?
14 Jul
14 Jul
11:56 p.m.
ok, removing the following macro from my_cmd.c fixes the issue:
define _XOPEN_SOURCE 4 /* BSD */
im not familiar with this macro though
anyone?
On Thu, Jul 14, 2011 at 9:08 PM, Jason Penton <jason.penton(a)gmail.com>wrote;wrote:
Hi All,
I've seen someone else had a similar problem but there were no responses.
On Solaris, compiling db_mysql results in the following:
CC (gcc) [M db_mysql.so] my_cmd.o
In file included from ../../globals.h:37:0,
from ../../pt.h:42,
from ../../counters.h:43,
from mysql_mod.h:39,
from my_cmd.c:40:
../../ip_addr.h:85:23: error: field 'sin6' has incomplete type
../../ip_addr.h: In function 'ip_addr_loopback':
../../ip_addr.h:281:3: warning: implicit declaration of function 'htonl'
../../ip_addr.h:284:3: warning: implicit declaration of function
'IN6_IS_ADDR_LOOPBACK'
../../ip_addr.h: In function 'sockaddr2ip_addr':
../../ip_addr.h:351:50: error: dereferencing pointer to incomplete type
../../ip_addr.h: In function 'su_getport':
../../ip_addr.h:397:4: warning: implicit declaration of function 'ntohs'
../../ip_addr.h: In function 'su_setport':
../../ip_addr.h:416:4: warning: implicit declaration of function 'htons'
whats puzzling me is that ip_addr.h is included in other modules which
compile perfectly, however in db_mysql, not
feel like I'm missing something really silly? any ideas?
15 Jul
15 Jul
5:54 p.m.
On Thursday 14 July 2011, Jason Penton wrote:
ok, removing the following macro from my_cmd.c fixes
the issue:
define _XOPEN_SOURCE 4 /* BSD */
im not familiar with this macro though
anyone?
Hi Jason,
ah, good catch. It seems that this declaration is necessary for the strptime
function in time.h, at least on some systems. Maybe we can include this in a
#ifndef that evaluates on the solaris plattform?
Best regards,
Henning
6:07 p.m.
Thanks Henning,
Should I put a patch into git for Solaris then? Daniel?
Cheers
Jason
On Fri, Jul 15, 2011 at 4:54 PM, Henning Westerholt <
henning.westerholt(a)1und1.de> wrote:
On Thursday 14 July 2011, Jason Penton wrote:
ok, removing the following macro from my_cmd.c
fixes the issue:
define _XOPEN_SOURCE 4 /* BSD */
im not familiar with this macro though
anyone?
Hi Jason,
ah, good catch. It seems that this declaration is necessary for the
strptime
function in time.h, at least on some systems. Maybe we can include this in
a
#ifndef that evaluates on the solaris plattform?
Best regards,
Henning
6:09 p.m.
Hello,
On 7/15/11 5:07 PM, Jason Penton wrote:
Thanks Henning,
Should I put a patch into git for Solaris then? Daniel?
sure, make it to master
branch and if proved to work, then it can be
backported.
Thanks,
Daniel
Cheers
Jason
On Fri, Jul 15, 2011 at 4:54 PM, Henning Westerholt
<henning.westerholt(a)1und1.de <mailto:henning.westerholt@1und1.de>> wrote:
On Thursday 14 July 2011, Jason Penton wrote:
ok, removing the following macro from my_cmd.c
fixes the issue:
define _XOPEN_SOURCE 4 /* BSD */
im not familiar with this macro though
anyone?
Hi Jason,
ah, good catch. It seems that this declaration is necessary for
the strptime
function in time.h, at least on some systems. Maybe we can include
this in a
#ifndef that evaluates on the solaris plattform?
Best regards,
Henning
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
16 Jul
16 Jul
11:18 a.m.
Hi Daniel,
I have committed the change. As this is my first commit using GIT - pls
confirm I have done it correctly and according to the siprouter specs
Cheers
Jason
On Fri, Jul 15, 2011 at 5:09 PM, Daniel-Constantin Mierla <
daniel(a)kamailio.org> wrote:
Hello,
On 7/15/11 5:07 PM, Jason Penton wrote:
Thanks Henning,
Should I put a patch into git for Solaris then? Daniel?
sure, make it to master branch and if proved to work, then it can be
backported.
Thanks,
Daniel
Cheers
Jason
On Fri, Jul 15, 2011 at 4:54 PM, Henning Westerholt <
henning.westerholt(a)1und1.de> wrote:
On Thursday 14 July 2011, Jason Penton wrote:
_______________________________________________
sr-dev mailing
listsr-dev@lists.sip-router.orghttp://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
ok, removing the following macro from my_cmd.c
fixes the issue:
define _XOPEN_SOURCE 4 /* BSD */
im not familiar with this macro though
anyone?
Hi Jason,
ah, good catch. It seems that this declaration is necessary for the
strptime
function in time.h, at least on some systems. Maybe we can include this in
a
#ifndef that evaluates on the solaris plattform?
Best regards,
Henning
12:01 p.m.
Hello,
I don't see the commit on sip-router.org GIT, have you push it to remote
repo, like:
git push origin master:master
Cheers,
Daniel
On 7/16/11 10:18 AM, Jason Penton wrote:
Hi Daniel,
I have committed the change. As this is my first commit using GIT -
pls confirm I have done it correctly and according to the siprouter specs
Cheers
Jason
On Fri, Jul 15, 2011 at 5:09 PM, Daniel-Constantin Mierla
<daniel(a)kamailio.org <mailto:daniel@kamailio.org>> wrote:
Hello,
On 7/15/11 5:07 PM, Jason Penton wrote:
--
Daniel-Constantin Mierla -- http://www.asipto.com
Kamailio Advanced Training, Oct 10-13, Berlin: http://asipto.com/u/kat
http://linkedin.com/in/miconda -- http://twitter.com/miconda
Thanks Henning,
Should I put a patch into git for Solaris then? Daniel?
sure, make it to
master branch and if proved to work, then it can
be backported.
Thanks,
Daniel
Cheers
Jason
On Fri, Jul 15, 2011 at 4:54 PM, Henning Westerholt
<henning.westerholt(a)1und1.de
<mailto:henning.westerholt@1und1.de>> wrote:
On Thursday 14 July 2011, Jason Penton wrote:
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev ok, removing the following macro from my_cmd.c
fixes the issue:
define _XOPEN_SOURCE 4 /* BSD */
im not familiar with this macro though
anyone?
Hi Jason,
ah, good catch. It seems that this declaration is necessary
for the strptime
function in time.h, at least on some systems. Maybe we can
include this in a
#ifndef that evaluates on the solaris plattform?
Best regards,
Henning
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org <mailto:sr-dev@lists.sip-router.org>
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
12:14 p.m.
ahh, thanks Daniel, I had only comitted locally
Cheers
Jason
On Sat, Jul 16, 2011 at 11:01 AM, Daniel-Constantin Mierla <
miconda(a)gmail.com> wrote:
Hello,
I don't see the commit on sip-router.org GIT, have you push it to remote
repo, like:
git push origin master:master
Cheers,
Daniel
On 7/16/11 10:18 AM, Jason Penton wrote:
Hi Daniel,
I have committed the change. As this is my first commit using GIT - pls
confirm I have done it correctly and according to the siprouter specs
Cheers
Jason
On Fri, Jul 15, 2011 at 5:09 PM, Daniel-Constantin Mierla <
daniel(a)kamailio.org> wrote:
Hello,
On 7/15/11 5:07 PM, Jason Penton wrote:
Thanks Henning,
Should I put a patch into git for Solaris then? Daniel?
sure, make it to master branch and if proved to work, then it can be
backported.
Thanks,
Daniel
Cheers
Jason
On Fri, Jul 15, 2011 at 4:54 PM, Henning Westerholt <
henning.westerholt(a)1und1.de> wrote:
_______________________________________________
sr-dev mailing
listsr-dev@lists.sip-router.orghttp://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
--
Daniel-Constantin Mierla -- http://www.asipto.com
Kamailio Advanced Training, Oct 10-13, Berlin:
http://asipto.com/u/kathttp://linkedin.com/in/miconda -- http://twitter.com/miconda
On Thursday 14 July 2011, Jason Penton wrote:
_______________________________________________
sr-dev mailing
listsr-dev@lists.sip-router.orghttp://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
ok, removing the following macro from my_cmd.c
fixes the issue:
define _XOPEN_SOURCE 4 /* BSD */
im not familiar with this macro though
anyone?
Hi Jason,
ah, good catch. It seems that this declaration is necessary for the
strptime
function in time.h, at least on some systems. Maybe we can include this
in a
#ifndef that evaluates on the solaris plattform?
Best regards,
Henning
12:41 p.m.
On 7/16/11 11:14 AM, Jason Penton wrote:
ahh, thanks Daniel, I had only comitted locally
np, all looks fine now, thanks.
Cheers,
Daniel
Cheers
Jason
On Sat, Jul 16, 2011 at 11:01 AM, Daniel-Constantin Mierla
<miconda(a)gmail.com <mailto:miconda@gmail.com>> wrote:
Hello,
I don't see the commit on sip-router.org <http://sip-router.org>
GIT, have you push it to remote repo, like:
git push origin master:master
Cheers,
Daniel
On 7/16/11 10:18 AM, Jason Penton wrote:
--
Daniel-Constantin Mierla -- http://www.asipto.com
Kamailio Advanced Training, Oct 10-13, Berlin: http://asipto.com/u/kat
http://linkedin.com/in/miconda -- http://twitter.com/miconda
Hi Daniel,
I have committed the change. As this is my first commit using GIT
- pls confirm I have done it correctly and according to the
siprouter specs
Cheers
Jason
On Fri, Jul 15, 2011 at 5:09 PM, Daniel-Constantin Mierla
<daniel(a)kamailio.org <mailto:daniel@kamailio.org>> wrote:
Hello,
On 7/15/11 5:07 PM, Jason Penton wrote:
--
Daniel-Constantin Mierla --http://www.asipto.com
Kamailio Advanced Training, Oct 10-13, Berlin:http://asipto.com/u/kat
http://linkedin.com/in/miconda -- http://twitter.com/miconda
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev Thanks Henning,
Should I put a patch into git for Solaris then? Daniel?
sure, make
it to master branch and if proved to work, then it
can be backported.
Thanks,
Daniel
Cheers
Jason
On Fri, Jul 15, 2011 at 4:54 PM, Henning Westerholt
<henning.westerholt(a)1und1.de
<mailto:henning.westerholt@1und1.de>> wrote:
On Thursday 14 July 2011, Jason Penton wrote:
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org <mailto:sr-dev@lists.sip-router.org>
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev ok, removing the following macro from my_cmd.c
fixes
the issue:
define _XOPEN_SOURCE 4 /* BSD */
im not familiar with this macro though
anyone?
Hi Jason,
ah, good catch. It seems that this declaration is
necessary for the strptime
function in time.h, at least on some systems. Maybe we
can include this in a
#ifndef that evaluates on the solaris plattform?
Best regards,
Henning
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org <mailto:sr-dev@lists.sip-router.org>
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
6:21 p.m.
On Fri, Jul 15, 2011 at 5:10 PM, Henning Westerholt <
henning.westerholt(a)1und1.de> wrote:
On Friday 15 July 2011, Jason Penton wrote:
absolutely, and was a major red herring as I thought it was an ipv6 issue at
first!!!! ;(
Thanks Henning,
Should I put a patch into git for Solaris then? Daniel?
Hi Jason,
in my opinion it should be fine - in the end the module seems not to
compile
for solaris at the moment.
Best regards,
Henning
4882
days inactive
4884
days old
10 comments
4 participants
participants (4)
-
Daniel-Constantin Mierla -
Daniel-Constantin Mierla -
Henning Westerholt -
Jason Penton