[Plplot-cvs] SF.net SVN: plplot:[10424] trunk/bindings/python/CMakeLists.txt

[<ai...@us...>]

Revision: 10424
          http://plplot.svn.sourceforge.net/plplot/?rev=10424&view=rev
Author:   airwin
Date:     2009-09-18 17:34:10 +0000 (Fri, 18 Sep 2009)

Log Message:
-----------
Remove FIXME comments since there is now an EXCLUDE_PYTHON_LIBRARIES option
the user can set to ON if they prefer not to link in the python libraries
for some reason.

Modified Paths:
--------------
    trunk/bindings/python/CMakeLists.txt

Modified: trunk/bindings/python/CMakeLists.txt
===================================================================
--- trunk/bindings/python/CMakeLists.txt	2009-09-18 17:27:21 UTC (rev 10423)
+++ trunk/bindings/python/CMakeLists.txt	2009-09-18 17:34:10 UTC (rev 10424)
@@ -76,9 +76,6 @@
   swig_add_module(plplotcmodule python plplotcmodule.i)
 
   swig_link_libraries(plplotcmodule plplot${LIB_TAG} ${PYTHON_LIBRARIES})
-  # FIXME: Previously I found this should be retwritten as:
-  #swig_link_libraries(plplotcmodule plplot${LIB_TAG})
-  # Investigate before changing.
 
   if(USE_RPATH)
     get_target_property(LIB_INSTALL_RPATH plplot${LIB_TAG} INSTALL_RPATH)
@@ -97,9 +94,6 @@
     "-DUSINGDLL"
     )
 
-  # FIXME: It remains unclear why PYTHON_LIBRARIES is in this list.  This
-  # might be system dependent.  Doesn't seem to be needed for Linux.  Is it
-  # really needed for other platforms?  Further investigation required.
   if(ENABLE_tk)
     target_link_libraries( plplot_widgetmodule
       plplot${LIB_TAG}


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.