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=6a45c8eba310afa9b8173c0522d77e7a574ddc52 Author: Konstantin M <[email protected]> Committer: Ovidiu Sas <[email protected]> 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. <[email protected]> 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. <[email protected]> > >> 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. _______________________________________________ sr-dev mailing list [email protected] http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
