GCCAVR & GCCARM Not supported under Linux.

[Slammer]

I notice that GCCAVR and GCCARM are not supported under linux.
It seems, in the file compilergcc.cpp that the definitions for these compilers are  used only under windows.
I changed some lines to include these compilers in Linux build and everything went OK.
Here is the patch:

Code Select Expand


@@ -59,6 +59,8 @@
// TODO (mandrav#1#): Find out which compilers exist for linux and adapt this
     #include "compilerGDC.h"
     #include "compilerDMD.h"
+    #include "compilerGNUARM.h"
+    #include "compilerGNUAVR.h"
#endif
#ifdef __WXMSW__
     #include "compilerMSVC.h"
@@ -328,14 +330,14 @@
     CompilerFactory::RegisterCompiler(new CompilerBCC);
     CompilerFactory::RegisterCompiler(new CompilerDMC);
     CompilerFactory::RegisterCompiler(new CompilerOW);
-    CompilerFactory::RegisterCompiler(new CompilerGNUARM);
-    CompilerFactory::RegisterCompiler(new CompilerGNUAVR);
     CompilerFactory::RegisterCompiler(new CompilerCYGWIN);
#endif
     CompilerFactory::RegisterCompiler(new CompilerICC);
     CompilerFactory::RegisterCompiler(new CompilerSDCC);
     CompilerFactory::RegisterCompiler(new CompilerTcc);
     CompilerFactory::RegisterCompiler(new CompilerGDC);
+    CompilerFactory::RegisterCompiler(new CompilerGNUARM);
+    CompilerFactory::RegisterCompiler(new CompilerGNUAVR);
#if defined(__WIN32__) || defined(__linux__)
     CompilerFactory::RegisterCompiler(new CompilerDMD);
#endif


Is there any reason to not include these compilers in linux build?
 

[rickg22]

Code Select Expand


// TODO (mandrav#1#): Find out which compilers exist for linux and adapt this


Perhaps that's a good answer... Anyway, mind officially posting your patch in the patch requests on berlios, so it won't get forgotten in the forum jungle?