Tools: Make bootstrap return -1 on error

This has been covering multiple issues as the CI was not failing as it should have been

Author: Pagghiu

Libraries/Http/HttpServer.h

@@ -143,6 +143,7 @@ struct SC_COMPILER_EXPORT HttpServer
     /// @param loop The event loop to be used, where to add the listening socket
     /// @param address The address of local interface where to listen to
     /// @param port The local port where to start listening to
+    /// @param memory Memory buffers to be used by the http server
     /// @return Valid Result if http listening has been started successfully
     Result start(AsyncEventLoop& loop, StringSpan address, uint16_t port, Memory& memory);
 
@@ -182,6 +183,6 @@ struct SC_COMPILER_EXPORT HttpServer
 
     AsyncEventLoop* eventLoop = nullptr;
 };
-} // namespace SC
-
 //! @}
+
+} // namespace SC

Tests/Libraries/Http/HttpServerTest.cpp

@@ -46,13 +46,13 @@ void SC::HttpServerTest::httpServerTest()
         if (request.getParser().method != HttpParser::Method::HttpGET)
         {
             SC_TEST_EXPECT(response.startResponse(405));
-            SC_TEST_EXPECT(response.end(""));
+            SC_TEST_EXPECT(response.end());
             return;
         }
         if (request.getURL() != "/index.html" and request.getURL() != "/")
         {
             SC_TEST_EXPECT(response.startResponse(404));
-            SC_TEST_EXPECT(response.end(""));
+            SC_TEST_EXPECT(response.end());
             return;
         }
         serverContext.numRequests++;

Tools/SC-build/Build.inl

@@ -334,13 +334,13 @@ bool SC::Build::Definition::findConfiguration(StringView workspaceName, StringVi
                                               StringView configurationName, Workspace*& workspace, Project*& project,
                                               Configuration*& configuration)
 {
-    size_t workspaceIdx;
+    size_t workspaceIdx = 0;
     SC_TRY(workspaces.find([&](auto& it) { return it.name == workspaceName; }, &workspaceIdx));
-    workspace = &workspaces[workspaceIdx];
-    size_t projectIdx;
+    workspace         = &workspaces[workspaceIdx];
+    size_t projectIdx = 0;
     SC_TRY(workspace->projects.find([&](auto& it) { return it.name == projectName; }, &projectIdx));
-    project = &workspace->projects[projectIdx];
-    size_t configurationIdx;
+    project                 = &workspace->projects[projectIdx];
+    size_t configurationIdx = 0;
     SC_TRY(project->configurations.find([&](auto& it) { return it.name == configurationName; }, &configurationIdx));
     configuration = &project->configurations[configurationIdx];
     return true;

Tools/ToolsBootstrap.c

@@ -1368,10 +1368,18 @@ int main(int argc, char* argv[]) {
     }
 
     int ret = executeTool(&args, &ci);
+    if(ret != 0)
+    {
+        fprintf(stderr, "Tool \"%s\" execution failed with code %d\n", args.toolName, ret);
+    }
+    else
+    {
+        printf("Tool \"%s\" executed successfully\n", args.toolName);
+    }
     CompilationInfo_destroy(&ci);
     BootloaderArgs_destroy(&args);
     freeArgs(new_argc, new_argv);
-    return ret;
+    return ret == 0 ? 0 : -1;
 }
 
 int compileWindows(CompilationInfo* ci, int* objsCompiled) {