Fixed: Archiver now putting object files correctly

Author: MrcSnm

source/redub/building/compile.d

@@ -126,13 +126,13 @@ CompilationResult execCompilation(immutable ThreadBuildData data, shared Project
             if(!pack.isCopyEnough)
                 ret = cast(ExecutionResult)execCompiler(cfg, compiler.binOrPath, getCompilationFlags(cfg, info, hash.rootHash, data.extra.isRoot), res.compilationCommand, compiler, inDir);
 
-            if(!isDCompiler(compiler) && !ret.status) //Always requires link.
+            if(!isDCompiler(compiler) && !ret.status && isStaticLibrary(data.cfg.targetType)) //Must call archiver when
             {
                 string cmd ;
                 auto archiverRes = executeArchiver(data, info, cmd);
                 ret.status = archiverRes.status;
                 ret.output~= archiverRes.output;
-                res.compilationCommand~= cmd;
+                res.compilationCommand~= "\n\nArchiving: \n\t"~cmd;
             }
 
             copyDir(inDir, dirName(outDir));

source/redub/building/utils.d

@@ -60,6 +60,13 @@ auto linkBase(const ThreadBuildData data, CompilingSession session, string rootH
     );
 }
 
+/**
+ * Generates a static library using archiver. FIXME: BuildRequirements should know its files.
+ * Params:
+ *   data = The data containing project information
+ *   s = Compiling Session
+ *   command = Command for being able to print it later
+ */
 auto executeArchiver(const ThreadBuildData data, CompilingSession s, out string command)
 {
     import std.process;
@@ -72,17 +79,25 @@ auto executeArchiver(const ThreadBuildData data, CompilingSession s, out string
     string[] cmd = [a.bin];
     final switch(a.type) with(AcceptedArchiver)
     {
-        case ar, llvmAr: cmd~= "rcs"; break;
+        case ar, llvmAr: cmd~= ["rcs"]; break;
         case libtool: cmd~= ["-static", "-o"]; break;
         case none: break;
     }
 
-    cmd~= getOutputName(data.cfg, s.os, s.isa);
-
-    string objExt = getObjectExtension(s.os);
-    cmd = mapAppend(cmd, data.cfg.sourceFiles, (string src) => stripExtension(src)~ objExt);
+    cmd~= buildNormalizedPath(data.cfg.outputDirectory, getOutputName(data.cfg, s.os, s.isa));
 
+    putObjectFiles(cmd, data.cfg, s.os, s.compiler.compiler.gcc ? cExt : cppExt);
     command = cmd.join(" ");
 
     return executeShell(command);
+}
+
+private void putObjectFiles(ref string[] target, const BuildConfiguration b, OS os, scope const string[] extensions...)
+{
+    import redub.command_generators.commons;
+    import std.file;
+    import std.path;
+    string[] objectFiles;
+    putSourceFiles(objectFiles, b.workingDir, b.sourcePaths, b.sourceFiles, b.excludeSourceFiles, extensions);
+    target = mapAppend(target, objectFiles, (string src) => setExtension(src, getObjectExtension(os)));
 }

source/redub/command_generators/automatic.d

@@ -23,12 +23,13 @@ string escapeCompilationCommands(string compilerBin, string[] flags)
  */
 string[] getCompilationFlags(const BuildConfiguration cfg, CompilingSession s, string mainPackHash, bool isRoot)
 {
+    import redub.command_generators.commons;
     switch(s.compiler.compiler) with(AcceptedCompiler)
     {
         case gxx:
-            return redub.command_generators.gnu_based.parseBuildConfiguration(cfg, s, mainPackHash, isRoot, ".c", ".cpp", ".cc", ".i", ".cxx", ".c++");
+            return redub.command_generators.gnu_based.parseBuildConfiguration(cfg, s, mainPackHash, isRoot, cppExt);
         case gcc:
-            return redub.command_generators.gnu_based.parseBuildConfiguration(cfg, s, mainPackHash, isRoot, ".c", ".i");
+            return redub.command_generators.gnu_based.parseBuildConfiguration(cfg, s, mainPackHash, isRoot, cExt);
         case dmd:
             return redub.command_generators.dmd.parseBuildConfiguration(cfg, s, mainPackHash, isRoot);
         case ldc2:

source/redub/command_generators/commons.d

@@ -7,6 +7,13 @@ public import redub.compiler_identification;
 import redub.buildapi;
 public import std.file:DirEntry;
 
+///Valid d extensions
+immutable string[] dExt   = [".d"];
+///Valid c extensions
+immutable string[] cExt   = [".c", ".i"];
+///Valid c++ extensions
+immutable string[] cppExt = [".c", ".cpp", ".cc", ".i", ".cxx", ".c++"];
+
 
 OS osFromArch(string arch)
 {

source/redub/command_generators/gnu_based.d

@@ -4,6 +4,7 @@ public import redub.buildapi;
 public import std.system;
 import redub.command_generators.commons;
 import redub.logging;
+import redub.building.cache;
 
 /// Parse G++ configuration
 string[] parseBuildConfiguration(const BuildConfiguration b, CompilingSession s, string requirementCache, bool isRoot, const string[] extensions...)
@@ -29,10 +30,9 @@ string[] parseBuildConfiguration(const BuildConfiguration b, CompilingSession s,
         if(targetType.isLinkedSeparately)
         {
             commands~= "-o";
-            commands ~= buildNormalizedPath(outputDirectory, getOutputName(targetType, name, os));
+            string cacheDir = getCacheOutputDir(requirementCache, b, s, isRoot);
+            commands ~= getObjectDir(cacheDir).escapePath;
         }
-
-
     }
 
     return commands;

source/redub/command_generators/linkers.d

@@ -57,13 +57,6 @@ string[] parseLinkConfiguration(const ThreadBuildData data, CompilingSession s,
             commands = mapAppend(commands, libraries, (string l) => "-L-l"~stripLibraryExtension(l));
 
         }
-        else if(!s.compiler.isDCompiler) //Generates a static library using archiver. FIXME: BuildRequirements should know its files.
-        {
-            commands~= "--format=default";
-            commands~= "rcs";
-            commands~= buildNormalizedPath(outputDirectory, getOutputName(b, s.os));
-            putObjectFiles(commands, b, s.os, ".c", ".cpp", ".cc", ".i", ".cxx", ".c++");
-        }
     }
 
     return commands;
@@ -133,11 +126,3 @@ string getTargetTypeFlag(TargetType o, Compiler compiler)
 }
 
 
-private void putObjectFiles(ref string[] target, const BuildConfiguration b, OS os, scope const string[] extensions...)
-{
-    import std.file;
-    import std.path;
-    string[] objectFiles;
-    putSourceFiles(objectFiles, b.workingDir, b.sourcePaths, b.sourceFiles, b.excludeSourceFiles, extensions);
-    target = mapAppend(target, objectFiles, (string src) => setExtension(src, getObjectExtension(os)));
-}