Developer forums (C::B DEVELOPMENT STRICTLY!) > Development
Today's updates: 30 Nov 2005
Urxae:
--- Quote from: mandrav on December 02, 2005, 02:29:06 pm ---
--- Code: ---codeblocks.exe --personality=Build --hidden --rebuild --target="Release" ogre_sample.cbp > build.log
--- End code ---
--- End quote ---
Judging by the redirection, I presume it now also sends the build outpout to stdout? If so, you forgot to mention this.
Der Meister:
It seems that the last changes broke the linux-version of the compilergcc-plugin. While trying to compile the recent svn-version I got this error:
--- Code: ---compilergcc.cpp: In member function `int CompilerGCC::DoRunQueue()':
compilergcc.cpp:767: error: ambiguous overload for 'operator+' in 'operator+(const wxString&, const wxChar*)(((const char*)" \'")) + cmd'
/usr/include/wx-2.6/wx/string.h:1466: note: candidates are: wxString operator+(const wxString&, const wxString&) <near match>
/usr/include/wx-2.6/wx/string.h:1467: note: wxString operator+(const wxString&, wxChar) <near match>
/usr/include/wx-2.6/wx/string.h:1468: note: wxString operator+(wxChar, const wxString&) <near match>
/usr/include/wx-2.6/wx/string.h:1470: note: wxString operator+(const wxChar*, const wxString&) <near match>
/usr/include/wx-2.6/wx/string.h:1479: note: wxString operator+(const wxString&, const wxCharBuffer&) <near match>
/usr/include/wx-2.6/wx/longlong.h:909: note: wxLongLong operator+(long int, const wxLongLong&) <near match>
/usr/include/wx-2.6/wx/longlong.h:922: note: wxULongLong operator+(long unsigned int, const wxULongLong&) <near match>
make[4]: *** [compilergcc.lo] Error 1
--- End code ---
This is the problematic line:
--- Code: ---cmd = GetConsoleShell() + _T(" '") + cmd + _T("'");
--- End code ---
I assume that it should look like this:
--- Code: ---cmd->command = GetConsoleShell() + _T(" '") + cmd->command + _T("'");
--- End code ---
I don't know if it's correct and I could not test it yet but (at least in my opinion) it makes more sense than before and it compiles without any problems.
mandrav:
--- Quote from: Urxae on December 02, 2005, 03:31:25 pm ---Judging by the redirection, I presume it now also sends the build outpout to stdout? If so, you forgot to mention this.
--- End quote ---
Yes, I thought it was evident by the example :)
--- Quote from: Der Meister on December 02, 2005, 03:33:01 pm ---I assume that it should look like this:
--- Code: ---cmd->command = GetConsoleShell() + _T(" '") + cmd->command + _T("'");
--- End code ---
I don't know if it's correct and I could not test it yet but (at least in my opinion) it makes more sense than before and it compiles without any problems.
--- End quote ---
Yes, you 're right. I haven't built it under linux yet so I missed it. Fixed and commited now.
Der Meister:
I just discovered another issue with that line. It encloses the command (that will be passed to the shell) with backticks ('). This makes sense but collides with the title for the shell which is also enclosed in backticks:
--- Code: ---shell.Replace(_T("$TITLE"), _T("'") + m_Project->GetTitle() + _T("'"));
--- End code ---
(This should be line 1134 in 'compilergcc.cpp'.)
Thus I think we should use double quotes (") to enclose the title instead of backticks. This seems to work (at least in my little test).
rickg22:
--- Quote from: mandrav on December 02, 2005, 02:29:06 pm ---Also revamped the compiler plugin and it's now using a CommandQueue. This has simplified the plugin code a lot.
--- End quote ---
O.O You mean *BESIDES* the cmdlinegenerator stuff you had added?
Wow! I'd love to have a look at that!
--- Quote from: Der Meister on December 02, 2005, 04:12:57 pm ---I just discovered another issue with that line. It encloses the command (that will be passed to the shell) with backticks ('). This makes sense but collides with the title for the shell which is also enclosed in backticks:
--- Code: ---shell.Replace(_T("$TITLE"), _T("'") + m_Project->GetTitle() + _T("'"));
--- End code ---
(This should be line 1134 in 'compilergcc.cpp'.)
Thus I think we should use double quotes (") to enclose the title instead of backticks. This seems to work (at least in my little test).
--- End quote ---
No, backticks ` are different from single quotes ' . Notice the backticks are a little skewed, like the Tower of Pisa (lol).
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version