Changeset 4639 for pjproject/branches/projects/pjsua2/pjsip-apps/src/swig/Makefile

Timestamp:

Nov 1, 2013 7:11:48 AM (11 years ago)

Author:

bennylp

Message:

Re #1519: Major change in pjsua2.i and etc after actual test in Python:

• Major changes in SWIG interface file (pjsua2.i).
  • The "ignore" and "unignore" approach caused link error with vector (duplicate symbols because SWIG generates two identical vector functions in the wrapper) so it cannot be used (tried with many combinations and still doesn't work).
  • So scrap that, and now we use "importing" approach, where needed symbols must be listed in symbols.lst file, then use importsym.py to import the symbol declarations to symbols.i. Then include symbols.i in pjsua.i
  • Due to the way importsym.py work, some macros and naked constants need to be placed into a named enumeration, for example:
    • #define PJ_SUCCESS 0 ==> enum pj_constants_ { PJ_SUCCESS=0 }; [types.h]
    • enum { PJMEDIA_VID_DEFAULT_CAPTURE_DEV = -1 } ==> enum pjmedia_vid_dev_std_index { PJMEDIA_VID_DEFAULT_CAPTURE_DEV = -1 }; [videodev.h]
  • Makefile was changed so that symbols.i would be generated if symbols.lst has changed
• Added "make install" and "make uninstall" targets for swig. Only implemented on Python. This will install to user's lib dir so doesn't need sudo.
• Deleted approachX.hpp files
• Added libVersion() method to Endpoint along with Version struct.
• Fix pjsua2.i to make exception (or redirection?) works in Python
• Add polling and worker thread related API:
  • PJSUA-LIB: pjsua_register_worker_thread(), pjsua_stop_worker_threads()
  • Endpoint: libRegisterWorkerThread(), libStopWorkerThreads(), libHandleEvents();

File:

• pjproject/branches/projects/pjsua2/pjsip-apps/src/swig/Makefile (modified) (2 diffs)

pjproject/branches/projects/pjsua2/pjsip-apps/src/swig/Makefile

@@ -9 +9 @@
 export SRCS=$(SRC_DIR)/pjsua2/endpoint.hpp $(SRC_DIR)/pjsua2/types.hpp
 
-all clean dep depend distclean print realclean:
+.PHONY: all clean dep depend distclean print realclean install uninstall
+
+all: symbols.i
+
+all clean dep depend distclean print realclean install uninstall:
         for dir in $(DIRS); do \
                 if $(MAKE) $(MAKE_FLAGS) -C $$dir $@; then \
@@ -17 +21 @@
                 fi; \
         done
+
+symbols.i: symbols.lst
+        python importsym.py