Hi,

I have recently been working on wxQT which allows wxWidgets to sit on top
of Qt (and thus target otherwise unsupported platforms like android and ios)

I found the FindwxWidgets module had a slight bug, but was not revealed
until I attempted to build applications which use wxQT.  I have corrected
it with the attached patch.

I was informed of the recent discussion:
http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/10679

As I use Modules/FindwxWidgets.cmake but not Modules/UsewxWidgets.cmake, I
don't think the recent change will work, further, my change corrects more
problems.   I believe the commit "e6fa6e60f6330ddf60294a0d9a6ed4cb3f27d4c4"
is probably not needed after my patch  applied.

Thanks,
Sean
--- FindwxWidgets.cmake 2014-06-25 14:41:27.490159713 +0800
+++ /usr/local/share/cmake-3.0/Modules/FindwxWidgets.cmake      2014-08-22 
11:53:11.928480401 +0800
@@ -788,11 +789,16 @@
         # parse include dirs from cxxflags; drop -I prefix
         string(REGEX MATCHALL "-I[^;]+"
           wxWidgets_INCLUDE_DIRS "${wxWidgets_CXX_FLAGS}")
-        string(REGEX REPLACE "-I[^;]+;" ""
+        string(REGEX REPLACE "-I[^;]+(;|$)" ""
+          wxWidgets_CXX_FLAGS "${wxWidgets_CXX_FLAGS}")
+        string(REGEX REPLACE ";$" ""
           wxWidgets_CXX_FLAGS "${wxWidgets_CXX_FLAGS}")
         string(REPLACE "-I" ""
           wxWidgets_INCLUDE_DIRS "${wxWidgets_INCLUDE_DIRS}")
 
+        string(REGEX REPLACE ";" " "
+          wxWidgets_CXX_FLAGS "${wxWidgets_CXX_FLAGS}")
+
         DBG_MSG_V("wxWidgets_DEFINITIONS=${wxWidgets_DEFINITIONS}")
         DBG_MSG_V("wxWidgets_INCLUDE_DIRS=${wxWidgets_INCLUDE_DIRS}")
         DBG_MSG_V("wxWidgets_CXX_FLAGS=${wxWidgets_CXX_FLAGS}")
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Reply via email to