buildsystem,epj2make,ide; (#973) use -L rather than -Wl,--library-path in makefile...
[sdk] / ide / src / project / Project.ec
index b83c9ef..1ab0083 100644 (file)
@@ -415,6 +415,7 @@ define stringInFileIncludedFrom = "In file included from ";
 define stringFrom =               "                 from ";
 
 // This function cannot accept same pointer for source and output
+// todo: rename ReplaceSpaces to EscapeSpaceAndSpecialChars or something
 void ReplaceSpaces(char * output, char * source)
 {
    int c, dc;
@@ -423,6 +424,8 @@ void ReplaceSpaces(char * output, char * source)
    for(c = 0, dc = 0; (ch = source[c]); c++, dc++)
    {
       if(ch == ' ') output[dc++] = '\\';
+      if(ch == '\"') output[dc++] = '\\';
+      if(ch == '&') output[dc++] = '\\';
       if(pch != '$')
       {
          if(ch == '(' || ch == ')') output[dc++] = '\\';
@@ -435,38 +438,81 @@ void ReplaceSpaces(char * output, char * source)
    output[dc] = '\0';
 }
 
-// This function cannot accept same pointer for source and output
-void ReplaceUnwantedMakeChars(char * output, char * source)
-{
-   int c, dc;
-   char ch, pch = 0;
+// chars refused for project name: windowsFileNameCharsNotAllowed + " &,."
 
-   for(c = 0, dc = 0; (ch = source[c]); c++, dc++)
+//define gnuMakeCharsNeedEscaping = "$%";
+
+//define windowsFileNameCharsNotAllowed = "*/:<>?\\\"|";
+//define linuxFileNameCharsNotAllowed = "/";
+
+//define windowsFileNameCharsNeedEscaping = " !%&'()+,;=[]^`{}~"; // "#$-.@_" are ok
+//define linuxFileNameCharsNeedEscaping = " !\"$&'()*:;<=>?[\\`{|"; // "#%+,-.@]^_}~" are ok
+
+// NOTES: - this function should get only unescaped unix style paths
+//        - paths with literal $(somestring) in them are not supported
+//          my_$(messed_up)_path would likely become my__path
+void EscapeForMake(char * output, char * input, bool hideSpace, bool allowVars, bool allowDblQuote)
+{
+   char ch, *i = input, *o = output;
+   bool inVar = false;
+#ifdef _DEBUG
+   int len = strlen(input);
+   if(len && ((input[0] == '"' && input[len-1] == '"') || strchr(input, '\\') || strchr(input, 127) || (!allowDblQuote && strchr(input, '"'))))
+      PrintLn("Invalid input for EscapeForMake! -- ", input);
+#endif
+   while((ch = *i++))
    {
-      if(pch != '$')
+      if(ch == '(')
       {
-         if(ch == '(' || ch == ')') output[dc++] = '\\';
-         pch = ch;
+         if(i-1 != input && *(i-2) == '$' && allowVars)
+            inVar = true;
+         else
+            *o++ = '\\';
       }
       else if(ch == ')')
-         pch = 0;
+      {
+         if(inVar == true && allowVars)
+            inVar = false;
+         else
+            *o++ = '\\';
+      }
+      else if(ch == '$' && *i != '(')
+         *o++ = '$';
+      else if(ch == '&')
+         *o++ = '\\';
+      else if(ch == '"')
+         *o++ = '\\';
       if(ch == ' ')
-         output[dc] = 127;
+      {
+         if(hideSpace)
+            *o++ = 127;
+         else
+         {
+            *o++ = '\\';
+            *o++ = ch;
+         }
+      }
       else
-         output[dc] = ch;
+         *o++ = ch;
    }
-   output[dc] = '\0';
+   *o = '\0';
 }
 
-static void OutputNoSpace(File f, char * source)
+void EscapeForMakeToFile(File output, char * input, bool hideSpace, bool allowVars, bool allowDblQuote)
 {
-   char * output = new char[strlen(source)+1024];
-   ReplaceSpaces(output, source);
-   f.Puts(output);
-   delete output;
+   char * buf = new char[strlen(input)*2+1];
+   EscapeForMake(buf, input, hideSpace, allowVars, allowDblQuote);
+   output.Print(buf);
+   delete buf;
 }
 
-enum ListOutputMethod { inPlace, newLine, lineEach };
+void EscapeForMakeToDynString(DynamicString output, char * input, bool hideSpace, bool allowVars, bool allowDblQuote)
+{
+   char * buf = new char[strlen(input)*2+1];
+   EscapeForMake(buf, input, hideSpace, allowVars, allowDblQuote);
+   output.concat(buf);
+   delete buf;
+}
 
 int OutputFileList(File f, char * name, Array<String> list, Map<String, int> varStringLenDiffs, char * prefix)
 {
@@ -582,33 +628,35 @@ void OutputCleanActions(File f, char * name, int parts)
       f.Printf("\t$(call rmq,$(%s))\n", name);
 }
 
-void OutputListOption(File f, char * option, Array<String> list, ListOutputMethod method, bool noSpace)
+enum LineOutputMethod { inPlace, newLine, lineEach };
+enum StringOutputMethod { asIs, escape, escapePath};
+
+enum ToolchainFlag { any, _D, _I, _isystem, _Wl, _L/*, _Wl-rpath*/ };
+String flagNames[ToolchainFlag] = { "", "-D", "-I", "-isystem ", "-Wl,", /*"-Wl,--library-path="*/"-L"/*, "-Wl,-rpath "*/ };
+void OutputFlags(File f, ToolchainFlag flag, Array<String> list, LineOutputMethod lineMethod)
 {
    if(list.count)
    {
-      if(method == newLine)
+      if(lineMethod == newLine)
          f.Puts(" \\\n\t");
       for(item : list)
       {
-         if(method == lineEach)
+         if(lineMethod == lineEach)
             f.Puts(" \\\n\t");
-         f.Printf(" -%s", option);
-         if(noSpace)
-            OutputNoSpace(f, item);
+         f.Printf(" %s", flagNames[flag]);
+         if(flag != _D && flag != _Wl && flag != any)
+         {
+            char * tmp = new char[strlen(item)*2+1];
+            EscapeForMake(tmp, item, false, true, false);
+            f.Printf("$(call quote_path,%s)", tmp);
+            delete tmp;
+         }
          else
-            f.Printf("%s", item);
+            EscapeForMakeToFile(f, item, false, true, true);
       }
    }
 }
 
-void StringNoSpaceToDynamicString(DynamicString s, char * path)
-{
-   char * output = new char[strlen(path)+1024];
-   ReplaceSpaces(output, path);
-   s.concat(output);
-   delete output;
-}
-
 static void OutputLibraries(File f, Array<String> libraries)
 {
    for(item : libraries)
@@ -634,7 +682,7 @@ static void OutputLibraries(File f, Array<String> libraries)
          f.Puts(" \\\n\t$(call _L,");
          usedFunction = true;
       }
-      OutputNoSpace(f, s);
+      EscapeForMakeToFile(f, s, false, false, false);
       if(usedFunction)
          f.Puts(")");
    }
@@ -839,6 +887,11 @@ private:
    String compilerConfigsDir;
    String moduleVersion;
 
+   String lastBuildConfigName;
+   String lastBuildCompilerName;
+
+   Map<String, NameCollisionInfo> lastBuildNamesInfo;
+
 #ifndef MAKEFILE_GENERATOR
    FileMonitor fileMonitor
    {
@@ -873,7 +926,7 @@ private:
       if(MessageBox { type = yesNo, master = ide,
             text = $"Project has been modified", contents = temp }.Modal() == yes)
       {
-         Project project = LoadProject(filePath, config.name);
+         Project project = LoadProject(filePath, config ? config.name : null);
          if(project)
          {
             ProjectView projectView = ide.projectView;
@@ -942,6 +995,9 @@ private:
       delete filePath;
       delete topNode;
       delete name;
+      delete lastBuildConfigName;
+      delete lastBuildCompilerName;
+      if(lastBuildNamesInfo) { lastBuildNamesInfo.Free(); delete lastBuildNamesInfo; }
    }
 
    ~Project()
@@ -963,6 +1019,7 @@ private:
          topNode.info = CopyString(GetConfigName(config));
       }
    }
+
    property char * filePath
    {
       set
@@ -985,6 +1042,29 @@ private:
       }
    }
 
+   ProjectConfig GetConfig(char * configName)
+   {
+      ProjectConfig result = null;
+      if(configName && configName[0] && configurations.count)
+      {
+         for(cfg : configurations; !strcmpi(cfg.name, configName))
+         {
+            result = cfg;
+            break;
+         }
+      }
+      return result;
+   }
+
+   ProjectNode FindNodeByObjectFileName(char * fileName, IntermediateFileType type, bool dotMain, ProjectConfig config)
+   {
+      ProjectNode result;
+      if(!lastBuildNamesInfo)
+         ProjectLoadLastBuildNamesInfo(this, config);
+      result = topNode.FindByObjectFileName(fileName, type, dotMain, lastBuildNamesInfo);
+      return result;
+   }
+
    TargetTypes GetTargetType(ProjectConfig config)
    {
       TargetTypes targetType = localTargetType;
@@ -1004,7 +1084,6 @@ private:
       return false;
    }
 
-
    char * GetObjDirExpression(ProjectConfig config)
    {
       // TODO: Support platform options
@@ -1438,7 +1517,7 @@ private:
       return !ide.projectView.stopBuild;
    }
 
-   bool ProcessBuildPipeOutput(DualPipe f, DirExpression objDirExp, bool isARun, List<ProjectNode> onlyNodes,
+   bool ProcessBuildPipeOutput(DualPipe f, DirExpression objDirExp, BuildType buildType, List<ProjectNode> onlyNodes,
       CompilerConfig compiler, ProjectConfig config, int bitDepth)
    {
       char line[65536];
@@ -1446,7 +1525,6 @@ private:
       int compilingEC = 0;
       int numErrors = 0, numWarnings = 0;
       bool loggedALine = false;
-      char * configName = config ? config.name : "Common";
       int lenMakeCommand = strlen(compiler.makeCommand);
       int testLen = 0;
       char * t;
@@ -1465,6 +1543,7 @@ private:
       DynamicString strip { };
       DynamicString ar { };
       DynamicString windres { };
+
       /*
       if(bitDepth == 64 && compiler.targetPlatform == win32) 
          gnuToolchainPrefix = "x86_64-w64-mingw32-";
@@ -1499,6 +1578,7 @@ private:
       testLen = Max(testLen, strip.size);
       testLen = Max(testLen, ar.size);
       testLen = Max(testLen, windres.size);
+      testLen = Max(testLen, strlen("mkdir "));
       testLen++;
 
       while(!f.Eof() && !ide.projectView.stopBuild)
@@ -1509,7 +1589,7 @@ private:
          while(result)
          {
             //printf("Peeking and GetLine...\n");
-            if((result = f.Peek()) && (result = f.GetLine(line, sizeof(line)-1)))
+            if((result = f.Peek()) && (result = f.GetLine(line, sizeof(line)-1)) && line[0])
             {
                char * message = null;
                char * inFileIncludedFrom = strstr(line, stringInFileIncludedFrom);
@@ -1537,6 +1617,8 @@ private:
                      //numErrors++;
                   //}
                }
+               else if(strstr(test, "mkdir ") == test);
+               else if((t = strstr(line, "cd ")) && (t = strstr(line, "type ")) && (t = strstr(line, "nul ")) && (t = strstr(line, "copy ")) && (t = strstr(line, "cd ")));
                else if(strstr(test, ear) == test);
                else if(strstr(test, strip) == test);
                else if(strstr(test, cc) == test || strstr(test, cxx) == test || strstr(test, ecp) == test || strstr(test, ecc) == test)
@@ -1586,7 +1668,7 @@ private:
                         precompiling = true;
                      }
                      // Changed escapeBackSlashes here to handle paths with spaces
-                     Tokenize(module, 1, tokens, true); // false);
+                     Tokenize(module, 1, tokens, (BackSlashEscaping)true); // fix #139
                      GetLastDirectory(module, moduleName);
                      ide.outputView.buildBox.Logf("%s\n", moduleName);
                   }
@@ -1616,7 +1698,15 @@ private:
                   if(module)
                   {
                      byte * tokens[1];
-                     Tokenize(module, 1, tokens, true);
+                     char * dashF = strstr(module, "-F ");
+                     if(dashF)
+                     {
+                        dashF+= 3;
+                        while(*dashF && *dashF != ' ') dashF++;
+                        while(*dashF && *dashF == ' ') dashF++;
+                        module = dashF;
+                     }
+                     Tokenize(module, 1, tokens, (BackSlashEscaping)true); // fix #139
                      GetLastDirectory(module, moduleName);
                      ide.outputView.buildBox.Logf("%s\n", moduleName);
                   }
@@ -1629,7 +1719,8 @@ private:
                {
                   if(linking || compiling || precompiling)
                   {
-                     char * colon = strstr(line, ":"); //, * bracket;
+                     char * start = inFileIncludedFrom ? inFileIncludedFrom + strlen(stringInFileIncludedFrom) : from ? from + strlen(stringFrom) : line;
+                     char * colon = strstr(start, ":"); //, * bracket;
                      if(colon && (colon[1] == '/' || colon[1] == '\\'))
                         colon = strstr(colon + 1, ":");
                      if(colon)
@@ -1638,7 +1729,6 @@ private:
                         char moduleName[MAX_LOCATION], temp[MAX_LOCATION];
                         char * pointer;
                         char * error;
-                        char * start = inFileIncludedFrom ? inFileIncludedFrom + strlen(stringInFileIncludedFrom) : from ? from + strlen(stringFrom) : line;
                         int len = (int)(colon - start);
                         char ext[MAX_EXTENSION];
                         len = Min(len, MAX_LOCATION-1);
@@ -1859,21 +1949,21 @@ private:
          ide.outputView.buildBox.Logf($"\nBuild cancelled by user.\n", line);
          f.Terminate();
       }
-      else if(loggedALine || !isARun)
+      else if(loggedALine || buildType != run)
       {
          if(f.GetExitCode() && !numErrors)
          {
             bool result = f.GetLine(line, sizeof(line)-1);
             ide.outputView.buildBox.Logf($"Fatal Error: child process terminated unexpectedly\n");
          }
-         else
+         else if(buildType != install)
          {
             if(!onlyNodes)
             {
                char targetFileName[MAX_LOCATION];
                targetFileName[0] = '\0';
                CatTargetFileName(targetFileName, compiler, config);
-               ide.outputView.buildBox.Logf("\n%s (%s) - ", targetFileName, configName);
+               ide.outputView.buildBox.Logf("\n%s (%s) - ", targetFileName, lastBuildConfigName);
             }
             if(numErrors)
                ide.outputView.buildBox.Logf("%d %s, ", numErrors, (numErrors > 1) ? $"errors" : $"error");
@@ -1898,6 +1988,7 @@ private:
       delete cxx;
       delete strip;
       delete ar;
+      delete windres;
 
       return numErrors == 0 && !ide.projectView.stopBuild;
    }
@@ -1913,9 +2004,10 @@ private:
          double lastTime = GetTime();
          while(result)
          {
-            if((result = f.Peek()) && (result = f.GetLine(line, sizeof(line)-1)))
+            if((result = f.Peek()) && (result = f.GetLine(line, sizeof(line)-1)) && line[0])
             {
                if(strstr(line, compiler.makeCommand) == line && line[lenMakeCommand] == ':');
+               else if(strstr(line, "mkdir") == line);
                else if(strstr(line, "del") == line);
                else if(strstr(line, "rm") == line);
                else if(strstr(line, "Could Not Find") == line);
@@ -1937,7 +2029,7 @@ private:
       }
    }
 
-   bool Build(bool isARun, List<ProjectNode> onlyNodes, CompilerConfig compiler, ProjectConfig config, int bitDepth, bool justPrint, SingleFileCompileMode mode)
+   bool Build(BuildType buildType, List<ProjectNode> onlyNodes, CompilerConfig compiler, ProjectConfig config, int bitDepth, bool justPrint, SingleFileCompileMode mode)
    {
       bool result = false;
       DualPipe f;
@@ -1955,9 +2047,14 @@ private:
       bool singleProjectOnlyNode = onlyNodes && onlyNodes.count == 1 && onlyNodes[0].type == project;
       int numJobs = compiler.numJobs;
       char command[MAX_F_STRING*4];
-      char * compilerName;
+      char * compilerName = CopyString(compiler.name);
+
+      delete lastBuildConfigName;
+      lastBuildConfigName = CopyString(config ? config.name : "Common");
+      delete lastBuildCompilerName;
+      lastBuildCompilerName = CopyString(compiler.name);
+      ProjectLoadLastBuildNamesInfo(this, config);
 
-      compilerName = CopyString(compiler.name);
       CamelCase(compilerName);
 
       strcpy(configName, config ? config.name : "Common");
@@ -1970,7 +2067,9 @@ private:
       PathCatSlash(makeFilePath, makeFile);
 
       // TODO: TEST ON UNIX IF \" around makeTarget is ok
-      if(onlyNodes)
+      if(buildType == install)
+         makeTargets.concat(" install");
+      else if(onlyNodes)
       {
          if(compiler.type.isVC)
          {
@@ -1981,7 +2080,6 @@ private:
             int len;
             char pushD[MAX_LOCATION];
             char cfDir[MAX_LOCATION];
-            Map<String, NameCollisionInfo> namesInfo { };
             GetIDECompilerConfigsDir(cfDir, true, true);
             GetWorkingDir(pushD, sizeof(pushD));
             ChangeWorkingDir(topNode.path);
@@ -2003,7 +2101,6 @@ private:
 
             ChangeWorkingDir(pushD);
 
-            topNode.GenMakefileGetNameCollisionInfo(namesInfo, config);
             for(node : onlyNodes)
             {
                if(node.GetIsExcluded(config))
@@ -2011,12 +2108,10 @@ private:
                else
                {
                   if(!eC_Debug)
-                     node.DeleteIntermediateFiles(compiler, config, bitDepth, namesInfo, mode == cObject ? true : false);
-                  node.GetTargets(config, namesInfo, objDirExp.dir, makeTargets);
+                     node.DeleteIntermediateFiles(compiler, config, bitDepth, lastBuildNamesInfo, mode == cObject ? true : false);
+                  node.GetTargets(config, lastBuildNamesInfo, objDirExp.dir, makeTargets);
                }
             }
-            namesInfo.Free();
-            delete namesInfo;
          }
       }
 
@@ -2047,7 +2142,12 @@ private:
       {
          char cfDir[MAX_LOCATION];
          GetIDECompilerConfigsDir(cfDir, true, true);
-         sprintf(command, "%s %sCF_DIR=\"%s\"%s%s%s%s%s COMPILER=%s %s-j%d %s%s%s -C \"%s\"%s -f \"%s\"",
+         sprintf(command, "%s%s %sCF_DIR=\"%s\"%s%s%s%s%s COMPILER=%s %s-j%d %s%s%s -C \"%s\"%s -f \"%s\"",
+#if defined(__WIN32__)
+               "",
+#else
+               buildType == install ? "pkexec --user root " : "",
+#endif
                compiler.makeCommand,
                mode == debugPrecompile ? "ECP_DEBUG=y " : mode == debugCompile ? "ECC_DEBUG=y " : mode == debugGenerateSymbols ? "ECS_DEBUG=y " : "",
                cfDir,
@@ -2077,7 +2177,7 @@ private:
                   bool result = true;
                   while(result)
                   {
-                     if((result = f.Peek()) && (result = f.GetLine(line, sizeof(line)-1)))
+                     if((result = f.Peek()) && (result = f.GetLine(line, sizeof(line)-1)) && line[0])
                      {
                         if(justPrint)
                            ide.outputView.buildBox.Logf("%s\n", line);
@@ -2100,7 +2200,7 @@ private:
             else if(justPrint)
                result = ProcessPipeOutputRaw(f);
             else
-               result = ProcessBuildPipeOutput(f, objDirExp, isARun, onlyNodes, compiler, config, bitDepth);
+               result = ProcessBuildPipeOutput(f, objDirExp, buildType, onlyNodes, compiler, config, bitDepth);
             delete f;
             if(error)
                ide.outputView.buildBox.Logf("%s\n", command);
@@ -2175,14 +2275,17 @@ private:
             ide.outputView.buildBox.Logf("%s\n", command);
          if((f = DualPipeOpen(PipeOpenMode { output = 1, error = 1, input = 2 }, command)))
          {
-            ide.outputView.buildBox.Tell($"Deleting target and object files...");
+            ide.outputView.buildBox.Tellf($"Deleting %s%s...",
+                  cleanType == realClean ? $"intermediate objects directory" : $"target",
+                  cleanType == clean ? $" and object files" : "");
             if(justPrint)
                ProcessPipeOutputRaw(f);
             else
                ProcessCleanPipeOutput(f, compiler, config);
+            ide.outputView.buildBox.Logf($"%s%s deleted\n",
+                  cleanType == realClean ? $"Intermediate objects directory" : $"Target",
+                  cleanType == clean ? $" and object files" : "");
             delete f;
-
-            ide.outputView.buildBox.Logf($"Target and object files deleted\n");
          }
       }
 
@@ -2200,11 +2303,7 @@ private:
 
       // Build(project, ideMain, true, null, false);
 
-   #if defined(__WIN32__)
       strcpy(target, topNode.path);
-   #else
-      strcpy(target, "");
-   #endif
       PathCatSlash(target, targetDirExp.dir);
       CatTargetFileName(target, compiler, config);
       sprintf(target, "%s %s", target, args);
@@ -2219,11 +2318,10 @@ private:
       }
       else
          ChangeWorkingDir(topNode.path);
-      // ChangeWorkingDir(topNode.path);
       SetPath(true, compiler, config, bitDepth);
       if(executableLauncher)
       {
-         char * prefixedTarget = new char[strlen(executableLauncher) + strlen(target) + 2];
+         char * prefixedTarget = new char[strlen(executableLauncher) + strlen(target) + 8];
          prefixedTarget[0] = '\0';
          strcat(prefixedTarget, executableLauncher);
          strcat(prefixedTarget, " ");
@@ -2243,7 +2341,7 @@ private:
 
    bool Compile(List<ProjectNode> nodes, CompilerConfig compiler, ProjectConfig config, int bitDepth, bool justPrint, SingleFileCompileMode mode)
    {
-      return Build(false, nodes, compiler, config, bitDepth, justPrint, mode);
+      return Build(build, nodes, compiler, config, bitDepth, justPrint, mode);
    }
 #endif
 
@@ -2252,8 +2350,6 @@ private:
       fileName[0] = '\0';
       if(targetType == staticLibrary || targetType == sharedLibrary)
          strcat(fileName, "$(LP)");
-      // !!! ReplaceSpaces must be done after all PathCat calls !!!
-      // ReplaceSpaces(s, GetTargetFileName(config));
       strcat(fileName, GetTargetFileName(config));
       switch(targetType)
       {
@@ -2384,7 +2480,7 @@ private:
             f.Printf("CXX := $(CCACHE_COMPILE)$(DISTCC_COMPILE)$(GCC_PREFIX)%s$(_SYSROOT)\n", compiler.cxxCommand);
             f.Printf("ECP := $(if $(ECP_DEBUG),ide -debug-start \"$(ECERE_SDK_SRC)/compiler/ecp/ecp.epj\" -debug-work-dir \"${CURDIR}\" -@,%s)\n", compiler.ecpCommand);
             f.Printf("ECC := $(if $(ECC_DEBUG),ide -debug-start \"$(ECERE_SDK_SRC)/compiler/ecc/ecc.epj\" -debug-work-dir \"${CURDIR}\" -@,%s)$(if $(CROSS_TARGET), -t $(TARGET_PLATFORM),)\n", compiler.eccCommand);
-            f.Printf("ECS := $(if $(ECS_DEBUG),ide -debug-start \"$(ECERE_SDK_SRC)/compiler/ecs/ecs.epj\" -debug-work-dir \"${CURDIR}\" -@,%s)$(if $(CROSS_TARGET), -t $(TARGET_PLATFORM),)\n", compiler.ecsCommand);
+            f.Printf("ECS := $(if $(ECS_DEBUG),ide -debug-start \"$(ECERE_SDK_SRC)/compiler/ecs/ecs.epj\" -debug-work-dir \"${CURDIR}\" -@,%s)$(if $(CROSS_TARGET), -t $(TARGET_PLATFORM),)$(if $(OUTPUT_POT), -outputpot,)\n", compiler.ecsCommand);
             f.Printf("EAR := %s\n", compiler.earCommand);
 
             f.Puts("AS := $(GCC_PREFIX)as\n");
@@ -2438,21 +2534,21 @@ private:
             if(compiler.includeDirs && compiler.includeDirs.count)
             {
                f.Puts("\nCFLAGS +=");
-               OutputListOption(f, gccCompiler ? "isystem " : "I", compiler.includeDirs, lineEach, true);
+               OutputFlags(f, gccCompiler ? _isystem : _I, compiler.includeDirs, lineEach);
                f.Puts("\n");
             }
             if(compiler.prepDirectives && compiler.prepDirectives.count)
             {
                f.Puts("\nCFLAGS +=");
-               OutputListOption(f, "D", compiler.prepDirectives, inPlace, true);
+               OutputFlags(f, _D, compiler.prepDirectives, inPlace);
                f.Puts("\n");
             }
             if(compiler.libraryDirs && compiler.libraryDirs.count)
             {
                f.Puts("\nLDFLAGS +=");
-               OutputListOption(f, "L", compiler.libraryDirs, lineEach, true);
+               OutputFlags(f, _L, compiler.libraryDirs, lineEach);
                // We would need a bool option to know whether we want to add to rpath as well...
-               // OutputListOption(f, "Wl,-rpath ", compiler.libraryDirs, lineEach, true);
+               // OutputFlags(f, _Wl-rpath, compiler.libraryDirs, lineEach);
                f.Puts("\n");
             }
             if(compiler.excludeLibs && compiler.excludeLibs.count)
@@ -2467,13 +2563,13 @@ private:
             if(compiler.compilerFlags && compiler.compilerFlags.count)
             {
                f.Puts("\nCFLAGS +=");
-               OutputListOption(f, "", compiler.compilerFlags, inPlace, true);
+               OutputFlags(f, any, compiler.compilerFlags, inPlace);
                f.Puts("\n");
             }
             if(compiler.linkerFlags && compiler.linkerFlags.count)
             {
                f.Puts("\nLDFLAGS +=");
-               OutputListOption(f, "Wl,", compiler.linkerFlags, inPlace, true);
+               OutputFlags(f, _Wl, compiler.linkerFlags, inPlace);
                f.Puts("\n");
             }
             f.Puts("\n");
@@ -2529,13 +2625,14 @@ private:
          char fixedModuleName[MAX_FILENAME];
          char fixedConfigName[MAX_FILENAME];
          int c, len;
+         int lenObjDirExpNoSpaces, lenTargetDirExpNoSpaces;
          // Non-zero if we're building eC code
          // We'll have to be careful with this when merging configs where eC files can be excluded in some configs and included in others
          int numCObjects = 0;
          int numObjects = 0;
          int numRCObjects = 0;
          bool containsCXX = false; // True if the project contains a C++ file
-         bool sameObjTargetDirs;
+         bool relObjDir, sameOrRelObjTargetDirs;
          String objDirExp = GetObjDirExpression(config);
          TargetTypes targetType = GetTargetType(config);
 
@@ -2569,9 +2666,15 @@ private:
          ReplaceSpaces(fixedConfigName, GetConfigName(config));
          CamelCase(fixedConfigName);
 
-         sameObjTargetDirs = !fstrcmp(objDirExpNoSpaces, targetDirExpNoSpaces);
+         lenObjDirExpNoSpaces = strlen(objDirExpNoSpaces);
+         relObjDir = lenObjDirExpNoSpaces == 0 ||
+               (objDirExpNoSpaces[0] == '.' && (lenObjDirExpNoSpaces == 1 || objDirExpNoSpaces[1] == '.'));
+         lenTargetDirExpNoSpaces = strlen(targetDirExpNoSpaces);
+         sameOrRelObjTargetDirs = lenTargetDirExpNoSpaces == 0 ||
+               (targetDirExpNoSpaces[0] == '.' && (lenTargetDirExpNoSpaces == 1 || targetDirExpNoSpaces[1] == '.')) ||
+               !fstrcmp(objDirExpNoSpaces, targetDirExpNoSpaces);
 
-         f.Printf(".PHONY: all objdir%s cleantarget clean realclean distclean\n\n", sameObjTargetDirs ? "" : " targetdir");
+         f.Printf(".PHONY: all objdir%s cleantarget clean realclean distclean\n\n", sameOrRelObjTargetDirs ? "" : " targetdir");
 
          f.Puts("# CORE VARIABLES\n\n");
 
@@ -2858,17 +2961,13 @@ private:
                      {
                         f.Puts(" \\\n\t ");
                         for(s : projectPlatformOptions.options.compilerOptions)
-                        {
                            f.Printf(" %s", s);
-                        }
                      }
                      if(configPlatformOptions && configPlatformOptions.options.compilerOptions && configPlatformOptions.options.compilerOptions.count)
                      {
                         f.Puts(" \\\n\t ");
                         for(s : configPlatformOptions.options.compilerOptions)
-                        {
                            f.Printf(" %s", s);
-                        }
                      }
                      f.Puts("\n");
                      f.Puts("\n");
@@ -2931,13 +3030,13 @@ private:
                      {
                         f.Puts("OFLAGS +=");
                         if(configPlatformOptions && configPlatformOptions.options.libraryDirs)
-                           OutputListOption(f, "L", configPlatformOptions.options.libraryDirs, lineEach, true);
+                           OutputFlags(f, _L, configPlatformOptions.options.libraryDirs, lineEach);
                         if(projectPlatformOptions && projectPlatformOptions.options.libraryDirs)
-                           OutputListOption(f, "L", projectPlatformOptions.options.libraryDirs, lineEach, true);
+                           OutputFlags(f, _L, projectPlatformOptions.options.libraryDirs, lineEach);
                         f.Puts("\n");
                      }
 
-                     if((configPlatformOptions && configPlatformOptions.options.libraries))
+                     if(configPlatformOptions && configPlatformOptions.options.libraries)
                      {
                         if(configPlatformOptions.options.libraries.count)
                         {
@@ -2968,6 +3067,26 @@ private:
             f.Puts("\n");
          }
 
+         if((config && config.options && config.options.compilerOptions && config.options.compilerOptions.count) ||
+               (options && options.compilerOptions && options.compilerOptions.count))
+         {
+            f.Puts("CFLAGS +=");
+            f.Puts(" \\\n\t");
+
+            if(config && config.options && config.options.compilerOptions && config.options.compilerOptions.count)
+            {
+               for(s : config.options.compilerOptions)
+                  f.Printf(" %s", s);
+            }
+            if(options && options.compilerOptions && options.compilerOptions.count)
+            {
+               for(s : options.compilerOptions)
+                  f.Printf(" %s", s);
+            }
+            f.Puts("\n");
+            f.Puts("\n");
+         }
+
          if((config && config.options && config.options.linkerOptions && config.options.linkerOptions.count) ||
                (options && options.linkerOptions && options.linkerOptions.count))
          {
@@ -3010,9 +3129,9 @@ private:
                         f.Printf(",%s", s);
                }
             }
+            f.Puts("\n");
+            f.Puts("\n");
          }
-         f.Puts("\n");
-         f.Puts("\n");
 
          f.Puts("CECFLAGS += -cpp $(_CPP)");
          f.Puts("\n");
@@ -3026,9 +3145,9 @@ private:
             f.Puts("ifndef STATIC_LIBRARY_TARGET\n");
             f.Puts("OFLAGS +=");
             if(config && config.options && config.options.libraryDirs)
-               OutputListOption(f, "L", config.options.libraryDirs, lineEach, true);
+               OutputFlags(f, _L, config.options.libraryDirs, lineEach);
             if(options && options.libraryDirs)
-               OutputListOption(f, "L", options.libraryDirs, lineEach, true);
+               OutputFlags(f, _L, options.libraryDirs, lineEach);
             f.Puts("\n");
             f.Puts("endif\n");
             f.Puts("\n");
@@ -3037,12 +3156,14 @@ private:
          f.Puts("# TARGETS\n");
          f.Puts("\n");
 
-         f.Printf("all: objdir%s $(TARGET)\n", sameObjTargetDirs ? "" : " targetdir");
+         f.Printf("all: objdir%s $(TARGET)\n", sameOrRelObjTargetDirs ? "" : " targetdir");
          f.Puts("\n");
 
          f.Puts("objdir:\n");
+         if(!relObjDir)
             f.Puts("\t$(if $(wildcard $(OBJ)),,$(call mkdirq,$(OBJ)))\n");
-            f.Puts("\t$(if $(ECERE_SDK_SRC),$(if $(wildcard $(call escspace,$(ECERE_SDK_SRC)/crossplatform.mk)),,@$(call echo,Ecere SDK Source Warning: The value of ECERE_SDK_SRC is pointing to an incorrect ($(ECERE_SDK_SRC)/crossplatform.mk) location.)),)\n");
+
+            f.Puts("\t$(if $(ECERE_SDK_SRC),$(if $(wildcard $(call escspace,$(ECERE_SDK_SRC)/crossplatform.mk)),,@$(call echo,Ecere SDK Source Warning: The value of ECERE_SDK_SRC is pointing to an incorrect ($(ECERE_SDK_SRC)) location.)),)\n");
             f.Puts("\t$(if $(ECERE_SDK_SRC),,$(if $(ECP_DEBUG)$(ECC_DEBUG)$(ECS_DEBUG),@$(call echo,ECC Debug Warning: Please define ECERE_SDK_SRC before using ECP_DEBUG, ECC_DEBUG or ECS_DEBUG),))\n");
          //f.Puts("# PRE-BUILD COMMANDS\n");
          if(options && options.prebuildCommands)
@@ -3093,7 +3214,7 @@ private:
          }
          f.Puts("\n");
 
-         if(!sameObjTargetDirs)
+         if(!sameOrRelObjTargetDirs)
          {
             f.Puts("targetdir:\n");
                f.Printf("\t$(if $(wildcard %s),,$(call mkdirq,%s))\n", targetDirExpNoSpaces, targetDirExpNoSpaces);
@@ -3105,7 +3226,7 @@ private:
             // Main Module (Linking) for ECERE C modules
             f.Puts("$(OBJ)$(MODULE).main.ec: $(SYMBOLS) $(COBJECTS)\n");
             // use of objDirExpNoSpaces used instead of $(OBJ) to prevent problematic joining of arguments in ecs
-            f.Printf("\t$(ECS)%s $(ARCH_FLAGS) $(ECSLIBOPT) $(SYMBOLS) $(IMPORTS) -symbols %s -o $@\n",
+            f.Printf("\t$(ECS)%s $(ARCH_FLAGS) $(ECSLIBOPT) $(SYMBOLS) $(IMPORTS) -symbols %s -o $(call escspace,$@)\n",
                GetConsole(config) ? " -console" : "", objDirExpNoSpaces);
             f.Puts("\n");
             // Main Module (Linking) for ECERE C modules
@@ -3113,14 +3234,14 @@ private:
             f.Puts("\t$(ECP) $(CFLAGS) $(CECFLAGS) $(ECFLAGS) $(PRJ_CFLAGS)"
                   " -c $(OBJ)$(MODULE).main.ec -o $(OBJ)$(MODULE).main.sym -symbols $(OBJ)\n");
             f.Puts("\t$(ECC) $(CFLAGS) $(CECFLAGS) $(ECFLAGS) $(PRJ_CFLAGS) $(FVISIBILITY)"
-                  " -c $(OBJ)$(MODULE).main.ec -o $@ -symbols $(OBJ)\n");
+                  " -c $(OBJ)$(MODULE).main.ec -o $(call escspace,$@) -symbols $(OBJ)\n");
             f.Puts("\n");
          }
 
          // *** Target ***
 
          // This would not rebuild the target on updated objects
-         // f.Printf("$(TARGET): $(SOURCES) $(RESOURCES) | objdir $(SYMBOLS) $(OBJECTS)%s\n", sameObjTargetDirs ? "" : " targetdir");
+         // f.Printf("$(TARGET): $(SOURCES) $(RESOURCES) | objdir $(SYMBOLS) $(OBJECTS)%s\n", sameOrRelObjTargetDirs ? "" : " targetdir");
 
          // This should fix it for good!
          f.Puts("$(SYMBOLS): | objdir\n");
@@ -3128,7 +3249,7 @@ private:
 
          // This alone was breaking the tarball, object directory does not get created first (order-only rules happen last it seems!)
          f.Printf("$(TARGET): $(SOURCES)%s $(RESOURCES) $(SYMBOLS) $(OBJECTS) | objdir%s\n",
-               rcSourcesParts ? " $(RCSOURCES)" : "", sameObjTargetDirs ? "" : " targetdir");
+               rcSourcesParts ? " $(RCSOURCES)" : "", sameOrRelObjTargetDirs ? "" : " targetdir");
 
          f.Printf("\t@$(call rmq,$(OBJ)linkobjects.lst)\n");
          f.Printf("\t@$(call touch,$(OBJ)linkobjects.lst)\n");
@@ -3161,8 +3282,10 @@ private:
                   f.Puts("\t$(UPX) $(UPXFLAGS) $(TARGET)\n");
                f.Puts("endif\n");
                f.Puts("else\n");
+               f.Puts("ifneq \"$(TARGET_ARCH)\" \"x86_64\"");
                   f.Puts("\t$(UPX) $(UPXFLAGS) $(TARGET)\n");
                f.Puts("endif\n");
+               f.Puts("endif\n");
             }
          }
          if(resNode.files && resNode.files.count && !noResources)
@@ -3229,6 +3352,74 @@ private:
          }
          f.Puts("\n");
 
+         test = false;
+         if(platforms || (config && config.platforms))
+         {
+            for(platform = (Platform)1; platform < Platform::enumSize; platform++)
+            {
+               PlatformOptions projectPOs, configPOs;
+               MatchProjectAndConfigPlatformOptions(config, platform, &projectPOs, &configPOs);
+
+               if((projectPOs && projectPOs.options.installCommands && projectPOs.options.installCommands.count) ||
+                     (configPOs && configPOs.options.installCommands && configPOs.options.installCommands.count))
+               {
+                  test = true;
+                  break;
+               }
+            }
+         }
+         if(test || (options && options.installCommands) ||
+               (config && config.options && config.options.installCommands))
+         {
+            f.Puts("install:\n");
+            if(options && options.installCommands)
+            {
+               for(s : options.installCommands)
+                  if(s && s[0]) f.Printf("\t%s\n", s);
+            }
+            if(config && config.options && config.options.installCommands)
+            {
+               for(s : config.options.installCommands)
+                  if(s && s[0]) f.Printf("\t%s\n", s);
+            }
+            if(platforms || (config && config.platforms))
+            {
+               ifCount = 0;
+               for(platform = (Platform)1; platform < Platform::enumSize; platform++)
+               {
+                  PlatformOptions projectPOs, configPOs;
+                  MatchProjectAndConfigPlatformOptions(config, platform, &projectPOs, &configPOs);
+
+                  if((projectPOs && projectPOs.options.installCommands && projectPOs.options.installCommands.count) ||
+                        (configPOs && configPOs.options.installCommands && configPOs.options.installCommands.count))
+                  {
+                     if(ifCount)
+                        f.Puts("else\n");
+                     ifCount++;
+                     f.Printf("ifdef %s\n", PlatformToMakefileTargetVariable(platform));
+
+                     if(projectPOs && projectPOs.options.installCommands && projectPOs.options.installCommands.count)
+                     {
+                        for(s : projectPOs.options.installCommands)
+                           if(s && s[0]) f.Printf("\t%s\n", s);
+                     }
+                     if(configPOs && configPOs.options.installCommands && configPOs.options.installCommands.count)
+                     {
+                        for(s : configPOs.options.installCommands)
+                           if(s && s[0]) f.Printf("\t%s\n", s);
+                     }
+                  }
+               }
+               if(ifCount)
+               {
+                  int c;
+                  for(c = 0; c < ifCount; c++)
+                     f.Puts("endif\n");
+               }
+            }
+            f.Puts("\n");
+         }
+
          f.Puts("# SYMBOL RULES\n");
          f.Puts("\n");
 
@@ -3248,7 +3439,7 @@ private:
          if(numCObjects)
             GenMakefilePrintMainObjectRule(f, config);
 
-         f.Printf("cleantarget: objdir%s\n", sameObjTargetDirs ? "" : " targetdir");
+         f.Printf("cleantarget: objdir%s\n", sameOrRelObjTargetDirs ? "" : " targetdir");
          f.Puts("\t$(call rmq,$(TARGET))\n");
          f.Puts("ifdef SHARED_LIBRARY_TARGET\n");
          f.Puts("ifdef LINUX_TARGET\n");
@@ -3283,14 +3474,15 @@ private:
 
          f.Puts("realclean: cleantarget\n");
          f.Puts("\t$(call rmrq,$(OBJ))\n");
-         if(!sameObjTargetDirs)
+         if(!sameOrRelObjTargetDirs)
             f.Printf("\t$(call rmdirq,%s)\n", targetDirExpNoSpaces);
          f.Puts("\n");
 
          f.Puts("distclean: cleantarget\n");
-         if(!sameObjTargetDirs)
+         if(!sameOrRelObjTargetDirs)
             f.Printf("\t$(call rmdirq,%s)\n", targetDirExpNoSpaces);
-         f.Puts("\t$(call rmrq,obj/)\n");
+         if(!relObjDir)
+            f.Puts("\t$(call rmrq,obj/)\n");
 
          delete f;
 
@@ -3403,7 +3595,7 @@ private:
          {
 #endif
             f.Puts("$(OBJ)$(MODULE).main$(O): $(OBJ)$(MODULE).main.c\n");
-            f.Printf("\t$(CC) $(CFLAGS) $(PRJ_CFLAGS) $(FVISIBILITY) -c $(OBJ)$(MODULE).main.%s -o $@\n", extension);
+            f.Printf("\t$(CC) $(CFLAGS) $(PRJ_CFLAGS) $(FVISIBILITY) -c $(OBJ)$(MODULE).main.%s -o $(call escspace,$@)\n", extension);
             f.Puts("\n");
 #if 0
          }
@@ -3425,13 +3617,11 @@ private:
                else
                   f.Printf("CUSTOM%d_%s =", v-1, variableName);
                f.Puts(&v ? &v : "");
-               f.Puts("\n");
-               f.Puts("\n");
+               f.Puts("\n\n");
                break;
             }
          }
       }
-      f.Puts("\n");
    }
 
    void MatchProjectAndConfigPlatformOptions(ProjectConfig config, Platform platform,
@@ -3464,6 +3654,17 @@ private:
    }
 }
 
+static inline void ProjectLoadLastBuildNamesInfo(Project prj, ProjectConfig cfg)
+{
+   if(prj.lastBuildNamesInfo)
+   {
+      prj.lastBuildNamesInfo.Free();
+      delete prj.lastBuildNamesInfo;
+   }
+   prj.lastBuildNamesInfo = { };
+   prj.topNode.GenMakefileGetNameCollisionInfo(prj.lastBuildNamesInfo, cfg);
+}
+
 Project LegacyBinaryLoadProject(File f, char * filePath)
 {
    Project project = null;
@@ -4297,6 +4498,17 @@ Project LoadProject(char * filePath, char * activeConfigName)
             delete project.topNode.files;
             if(!project.files) project.files = { };
             project.topNode.files = project.files;
+
+            {
+               char topNodePath[MAX_LOCATION];
+               GetWorkingDir(topNodePath, sizeof(topNodePath)-1);
+               MakeSlashPath(topNodePath);
+               PathCatSlash(topNodePath, filePath);
+               project.filePath = topNodePath;//filePath;
+            }
+
+            project.topNode.FixupNode(insidePath);
+
             project.resNode = project.topNode.Add(project, "Resources", project.topNode.files.last, resources, archiveFile, false);
             delete project.resNode.path;
             project.resNode.path = project.resourcesPath;
@@ -4308,15 +4520,7 @@ Project LoadProject(char * filePath, char * activeConfigName)
             project.resources = null;
             if(!project.configurations) project.configurations = { };
 
-            {
-               char topNodePath[MAX_LOCATION];
-               GetWorkingDir(topNodePath, sizeof(topNodePath)-1);
-               MakeSlashPath(topNodePath);
-               PathCatSlash(topNodePath, filePath);
-               project.filePath = topNodePath;//filePath;
-            }
-
-            project.topNode.FixupNode(insidePath);
+            project.resNode.FixupNode(insidePath);
          }
          delete parser;
       }
@@ -4329,16 +4533,7 @@ Project LoadProject(char * filePath, char * activeConfigName)
       {
          if(!project.options) project.options = { };
          if(activeConfigName && activeConfigName[0] && project.configurations)
-         {
-            for(cfg : project.configurations)
-            {
-               if(!strcmpi(cfg.name, activeConfigName))
-               {
-                  project.config = cfg;
-                  break;
-               }
-            }
-         }
+            project.config = project.GetConfig(activeConfigName);
          if(!project.config && project.configurations)
             project.config = project.configurations.firstIterator.data;