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?
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@gmail.comwrote:
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?
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
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@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
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@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@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
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@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@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 listsr-dev@lists.sip-router.orghttp://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
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@kamailio.org mailto:daniel@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, DanielCheers Jason On Fri, Jul 15, 2011 at 4:54 PM, Henning Westerholt <henning.westerholt@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@lists.sip-router.org <mailto:sr-dev@lists.sip-router.org> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
ahh, thanks Daniel, I had only comitted locally
Cheers Jason
On Sat, Jul 16, 2011 at 11:01 AM, Daniel-Constantin Mierla < miconda@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@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@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 listsr-dev@lists.sip-router.orghttp://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
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 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@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: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@kamailio.org <mailto:daniel@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, DanielCheers Jason On Fri, Jul 15, 2011 at 4:54 PM, Henning Westerholt <henning.westerholt@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@lists.sip-router.org <mailto:sr-dev@lists.sip-router.org> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev_______________________________________________ sr-dev mailing list sr-dev@lists.sip-router.org <mailto:sr-dev@lists.sip-router.org> http://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/kat http://linkedin.com/in/miconda -- http://twitter.com/miconda
sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
On Fri, Jul 15, 2011 at 5:10 PM, Henning Westerholt < henning.westerholt@1und1.de> wrote:
On Friday 15 July 2011, Jason Penton wrote:
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.
absolutely, and was a major red herring as I thought it was an ipv6 issue at first!!!! ;(
Best regards,
Henning
-
Daniel-Constantin Mierla -
Daniel-Constantin Mierla -
Henning Westerholt -
Jason Penton