rev 3967 does not build on linux (and maybe others)

[killerbot]

wxsgrid.cpp needs to be added, or it should be removed from the makefile ...

EDIT : it's added, but I think it is referred to in the wrong makefile --> one level deeper in defitems ??

[TDragon]

it's added, but I think it is referred to in the wrong makefile --> one level deeper in defitems ??

Moving wxsgrid.cpp and wxsgrid.h from wxSmith/wxwidgets/Makefile.am to wxSmith/wxwidgets/defitems/Makefile.am solved the problem for me. See my post in the Nightlies forum.

[juhas]

it's still wrong on my linux (wxWidgets 2.8.4) in rev. 3975:

 g++ -DHAVE_CONFIG_H -I. -I. -I../../src/include -I/usr/local/lib/wx/include/gtk2-ansi-release-2.8 -I/usr/local/include/wx-2.8 -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES -D__WXGTK__ -pthread -I../../src/include -I../../src/include/wxscintilla/include -I../../src/include/tinyxml -I../../src/include/scripting/include -I../../src/include/scripting/sqplus -I../../src/include/wxFlatNotebook/include -I../../src/include/propgrid/include -Ulinux -Uunix -O2 -ffast-math -g -O2 -DCB_PRECOMP -Winvalid-pch -fPIC -DPIC -fexceptions -MT editormanager.lo -MD -MP -MF .deps/editormanager.Tpo -c editormanager.cpp  -fPIC -DPIC -o .libs/editormanager.o
editormanager.cpp: In member function 'int EditorManager::Replace(cbStyledTextCtrl*, cbFindReplaceData*)':
editormanager.cpp:1455: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp:1457: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp: In member function 'int EditorManager::ReplaceInFiles(cbFindReplaceData*)':
editormanager.cpp:1739: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp:1741: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp: In member function 'int EditorManager::Find(cbStyledTextCtrl*, cbFindReplaceData*)':
editormanager.cpp:2014: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp:2016: error: 'wxRE_ADVANCED' was not declared in this scope

[stahta01]

it's still wrong on my linux (wxWidgets 2.8.4) in rev. 3975:

 g++ -DHAVE_CONFIG_H -I. -I. -I../../src/include -I/usr/local/lib/wx/include/gtk2-ansi-release-2.8 -I/usr/local/include/wx-2.8 -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES -D__WXGTK__ -pthread -I../../src/include -I../../src/include/wxscintilla/include -I../../src/include/tinyxml -I../../src/include/scripting/include -I../../src/include/scripting/sqplus -I../../src/include/wxFlatNotebook/include -I../../src/include/propgrid/include -Ulinux -Uunix -O2 -ffast-math -g -O2 -DCB_PRECOMP -Winvalid-pch -fPIC -DPIC -fexceptions -MT editormanager.lo -MD -MP -MF .deps/editormanager.Tpo -c editormanager.cpp  -fPIC -DPIC -o .libs/editormanager.o
editormanager.cpp: In member function 'int EditorManager::Replace(cbStyledTextCtrl*, cbFindReplaceData*)':
editormanager.cpp:1455: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp:1457: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp: In member function 'int EditorManager::ReplaceInFiles(cbFindReplaceData*)':
editormanager.cpp:1739: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp:1741: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp: In member function 'int EditorManager::Find(cbStyledTextCtrl*, cbFindReplaceData*)':
editormanager.cpp:2014: error: 'wxRE_ADVANCED' was not declared in this scope
editormanager.cpp:2016: error: 'wxRE_ADVANCED' was not declared in this scope

This is related to not using the builtin regular expression library, I posted a work around a month or two ago.
But, try using doing config using at least "--with-regex=builtin" below is the ones I used below in windows.

Note, some builtin library seem to require the use "--with-libjpeg=builtin" to work.

Tim S

Note, the below windows build is not for the Code::Blocks build. It show as example of using --with

Code

../configure --with-msw --enable-shared=no --enable-monolithic=no --enable-unicode=no \
  --with-libjpeg=builtin --with-libpng=builtin --with-regex=builtin --with-libtiff=builtin --with-zlib=builtin -with-expat=builtin

Found, my patch listed in thread below. I don't remember if it worked or not. My final patch did, but not sure this is my final patch.
http://forums.codeblocks.org/index.php/topic,5681.msg43657.html#msg43657

[juhas]

Found, my patch listed in thread below. I don't remember if it worked or not. My final patch did, but not sure this is my final patch.
http://forums.codeblocks.org/index.php/topic,5681.msg43657.html#msg43657

It works, thanks a lot!
Maybe developers could apply this patch in new rev?

[stahta01]

Found, my patch listed in thread below. I don't remember if it worked or not. My final patch did, but not sure this is my final patch.
http://forums.codeblocks.org/index.php/topic,5681.msg43657.html#msg43657

It works, thanks a lot!
Maybe developers could apply this patch in new rev?

I worked on the patch over a month ago, I just don't have time to confirm it's safe to use and keep updating it. Code::Blocks is changing too fast for me to maintain patches I don't care about.
To me it is easier to use the libraries as built-in, but I am a windows user it defaults to built-in normally, I saw the issue under cygwin testing.

It's in the bug filed on the problem.

http://developer.berlios.de/bugs/?func=detailbug&bug_id=10881&group_id=5358

Tim S

[Biplab]

As commented on the bug report, the patch will avoid compilation error. But it may create a False sense of an Advanced Feature.

IMHO, it would not be a good idea to keep that option enabled if wxRE_ADVANCED is not at all available during compilation. If an user tries to use advanced regex function with wxRE_ADVANCED set to 0, the function will not work appropriately and we may see another bug report reporting that the advanced regex function is not working properly.

This is my concern about the patch. I think, it'd be better if the advanced regex functions are #ifdef'ed.

Regards,

Biplab