Code::Blocks Forums

Developer forums (C::B DEVELOPMENT STRICTLY!) => Development => Topic started by: killerbot on May 16, 2007, 12:11:55 pm

Title: rev 3967 does not build on linux (and maybe others)
Post by: killerbot on May 16, 2007, 12:11:55 pm
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 ??
Title: Re: rev 3967 does not build on linux (and maybe others)
Post by: TDragon on May 16, 2007, 05:55:13 pm
Quote from: killerbot on May 16, 2007, 12:11:55 pm
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 (http://forums.codeblocks.org/index.php/topic,5915.msg45278.html#msg45278).
Title: Re: rev 3967 does not build on linux (and maybe others)
Post by: juhas on May 20, 2007, 08:38:29 pm
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
Title: Re: rev 3967 does not build on linux (and maybe others)
Post by: stahta01 on May 20, 2007, 08:54:49 pm
Quote from: juhas on May 20, 2007, 08:38:29 pm
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
Title: Re: rev 3967 does not build on linux (and maybe others)
Post by: juhas on May 20, 2007, 09:36:54 pm
Quote
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?
Title: Re: rev 3967 does not build on linux (and maybe others)
Post by: stahta01 on May 20, 2007, 09:53:50 pm
Quote from: juhas on May 20, 2007, 09:36:54 pm
Quote
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

Title: Re: rev 3967 does not build on linux (and maybe others)
Post by: Biplab on May 21, 2007, 08:59:19 pm
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