Hello Konstantin,
I tested your patch and I pushed your patches:
Commit: 6a45c8eba310afa9b8173c0522d77e7a574ddc52
URL:
http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6a45c8e…
Author: Konstantin M <evilzluk(a)gmail.com>
Committer: Ovidiu Sas <osas(a)voipembedded.com>
Date: Sun Dec 2 22:21:05 2012 -0500
Thanks for the contribution,
Ovidiu Sas
On Sun, Dec 2, 2012 at 6:56 PM, Konstantin M. <evilzluk(a)gmail.com> wrote:
Next patch for fixing:
CC (gcc) [M app_python.so] python_msgobj.o
python_msgobj.c: In function ‘python_msgobj_init’:
python_msgobj.c:530:5: warning: dereferencing type-punned pointer will break
strict-aliasing rules [-Wstrict-aliasing]
2012/12/3 Konstantin M. <evilzluk(a)gmail.com>
> Hello dev team,
>
> I'm trying to compile app_python under kamailio 3.3.2 and gcc (Debian
> 4.7.2-4) 4.7.2 and I'm getting a bunch of warnings like:
> CC (gcc) [M app_python.so] python_exec.o
> CC (gcc) [M app_python.so] python_iface.o
> In file included from /usr/include/python2.7/Python.h:8:0,
> from python_iface.c:28:
> /usr/include/python2.7/pyconfig.h:1161:0: warning: "_POSIX_C_SOURCE"
> redefined [enabled by default]
> In file included from /usr/include/i386-linux-gnu/sys/types.h:26:0,
> from ../../route_struct.h:45,
> from ../../action.h:42,
> from python_iface.c:23:
> /usr/include/features.h:214:0: note: this is the location of the previous
> definition
> CC (gcc) [M app_python.so] python_mod.o
> In file included from /usr/include/python2.7/Python.h:8:0,
> from python_iface.h:26,
> from python_mod.c:27:
> /usr/include/python2.7/pyconfig.h:1161:0: warning: "_POSIX_C_SOURCE"
> redefined [enabled by default]
> In file included from /usr/include/dlfcn.h:23:0,
> from ../../sr_module.h:60,
> from python_mod.c:24:
> /usr/include/features.h:214:0: note: this is the location of the previous
> definition
>
> ...
>
>
> According to the Python documentation
> (
http://docs.python.org/c-api/intro.html#includes), the include directive
> for “Python.h” should appear first in the C/C++ file.
> It doesn’t sound like a good idea to require this, but if you don’t put
> “Python.h” first, you might get compiler warnings like I mentioned above.
>
> I've made a diff (against kamailio-3.3.2) for fixing that.
> See attached.