add specific options for the modules to modify the compiler and linker options

Author: yldrefruz

ebuild.api/ArgumentBuilder.cs

@@ -10,11 +10,11 @@ public ArgumentBuilder Add(string command)
 
     public ArgumentBuilder AddRange(IEnumerable<string> commands)
     {
-        _args.AddRange(commands);
+        _args.UnionWith(commands);
         return this;
     }
 
-    private readonly List<string> _args = new();
+    private readonly HashSet<string> _args = new();
 
     public override string ToString()
     {

ebuild.api/ModuleBase.cs

@@ -71,6 +71,9 @@ public abstract class ModuleBase
 
     public ModuleContext Context;
 
+    public List<string> CompilerOptions = [];
+    public List<string> LinkerOptions = [];
+
     protected ModuleBase(ModuleContext context)
     {
         Context = context;

ebuild/Compilers/GccCompiler.cs

@@ -242,7 +242,7 @@ private string GenerateCompileCommand(bool includeSourceFiles)
 
         // Add additional compiler options
         args += AdditionalCompilerOptions;
-        
+        args += CurrentModule.CompilerOptions;
         // Add source files if requested
         if (includeSourceFiles)
         {

ebuild/Compilers/GppCompiler.cs

@@ -309,7 +309,7 @@ private string GenerateCompileCommand(bool includeSourceFiles)
 
         // Add additional compiler options
         args += AdditionalCompilerOptions;
-        
+        args += CurrentModule.CompilerOptions;
         // Add source files if requested
         if (includeSourceFiles)
         {

ebuild/Compilers/MSVCCompiler.cs

@@ -154,6 +154,7 @@ private void AddModuleCompileArguments(ModuleBase inputModule, bool compilingInp
         if (compilingInputModule)
         {
             args += $"/D\"{(inputModule.Name ?? inputModule.Context.ModuleDirectory!.Name).ToUpperInvariant()}_BUILDING\"";
+            args += inputModule.CompilerOptions;
             // TODO: Disabled for now to test.
             // args += module.SourceFiles.Select(s => GetModuleFilePath(s, module));
         }

ebuild/Linkers/GccLinker.cs

@@ -326,6 +326,7 @@ private string BuildLinkCommand()
 
         // Add additional linker options
         args += AdditionalLinkerOptions;
+        args += CurrentModule.LinkerOptions;
 
         return args.ToString();
     }

ebuild/Linkers/MsvcLibLinker.cs

@@ -101,6 +101,7 @@ private async Task<bool> CallLibExe()
         argumentBuilder +=
             $"/OUT:\"{Path.Join(GetBinaryOutputFolder(), (CurrentModule.Name ?? CurrentModule.GetType().Name) + ".lib")}\"";
         argumentBuilder += AdditionalLinkerOptions;
+        argumentBuilder += CurrentModule.LinkerOptions;
         argumentBuilder += CurrentModule.LibrarySearchPaths.Joined().Select(s => $"/LIBPATH:\"{s}\"");
         argumentBuilder += files.Select(f => GetModuleFilePath(f, CurrentModule));
         argumentBuilder += CurrentModule.Libraries.Joined()

ebuild/Linkers/MsvcLinkLinker.cs

@@ -99,7 +99,7 @@ private async Task<bool> CallLinkExe()
         }
 
         argumentBuilder += AdditionalLinkerOptions;
-
+        argumentBuilder += CurrentModule.LinkerOptions;
         var outType = ".exe";
         switch (CurrentModule.Type)
         {

examples/zlib/zlib.ebuild.cs

@@ -33,7 +33,7 @@ public ZlibEbuild(ModuleContext context) : base(context)
         Name = "zlib";
 
         // Use C99 standard for zlib (it's a C library)
-        CStandard = CStandards.C99;
+        CStandard = CStandards.C17;
 
         // Setup should be called in constructor as per README
         // Call setup synchronously instead of using async/await
@@ -259,7 +259,16 @@ public void SetupSourceFiles(string extractPath)
 
         // Add include directories
         Includes.Add(extractPath);
-        
+        if (Context.Platform == "Unix")
+        {
+            CompilerOptions.Add("-Wno-error=implicit-function-declaration");
+        }
+        if (Context.Platform == "Win32")
+        {
+            Definitions.Private.Add(new Definition("_CRT_SECURE_NO_DEPRECATE"));
+            Definitions.Private.Add(new Definition("_CRT_NONSTDC_NO_DEPRECATE"));
+        }
+
         // Apply module options to definitions
         if (EnableDebug)
         {