chore: remove useless logs

shdwmtr · Jul 21, 2024 · 2efe5a7 · 2efe5a7
1 parent 0606641
commit 2efe5a7
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 17 deletions.
diff --git a/src/core/py_controller/co_spawn.cc b/src/core/py_controller/co_spawn.cc
@@ -55,10 +55,10 @@ PythonManager::PythonManager() : m_InterpreterThreadSave(nullptr)
     this->m_instanceCount = 0;
 
     // initialize global modules
-    Logger.LogItem("+", "Redirecting Standard Output...");    PyImport_AppendInittab("hook_stdout", &PyInit_CustomStdout);
-    Logger.LogItem("+", "Redirecting Standard Error...");    PyImport_AppendInittab("hook_stderr", &PyInit_CustomStderr);
-    Logger.LogItem("+", "Inserting PluginUtils..."); PyImport_AppendInittab("PluginUtils", &PyInit_Logger);
-    Logger.LogItem("+", "Inserting Millennium...");  PyImport_AppendInittab("Millennium",  &PyInit_Millennium);
+    PyImport_AppendInittab("hook_stdout", &PyInit_CustomStdout);
+    PyImport_AppendInittab("hook_stderr", &PyInit_CustomStderr);
+    PyImport_AppendInittab("PluginUtils", &PyInit_Logger);
+    PyImport_AppendInittab("Millennium",  &PyInit_Millennium);
 
     Logger.LogItem("Python", "Initializing Python...");
 

diff --git a/src/git/git.cc b/src/git/git.cc
@@ -19,7 +19,6 @@ namespace Dependencies {
     {
         if (refCallbackIsMerge)
         {
-            Logger.LogItem("Head", "Found merge reference...");
             git_oid_cpy((git_oid *)payload, oid);
         }
         return 0;
@@ -35,7 +34,7 @@ namespace Dependencies {
         if (std::filesystem::exists(packageLocalPath) && !std::filesystem::is_empty(packageLocalPath)) 
         {
             std::uintmax_t removedCount = std::filesystem::remove_all(packageLocalPath);
-            Logger.LogItem("Status", fmt::format("Flushed {} items", removedCount));
+            Logger.LogItem("+", fmt::format("Flushed {} items", removedCount));
         }
         else if (!std::filesystem::exists(packageLocalPath))
         {
@@ -50,18 +49,18 @@ namespace Dependencies {
         }
         else 
         {
-            Logger.LogItem("Status", "Ready to clone module...");
+            Logger.LogItem("+", "Ready to clone module...");
         }
 
-        Logger.LogItem("status", "Cloning modules...");
+        Logger.LogItem("+", "Cloning modules...");
         int cloneSuccessStatus = git_clone(&repository, remoteObject.c_str(), packageLocalPath.c_str(), &gitCloneOpts);
 
         if (cloneSuccessStatus != 0) 
         {
             const git_error* lastError = git_error_last();
             const std::string strErrorMessage = fmt::format("Error cloning frontend modules -> {}", lastError->message);
 
-            Logger.LogItem("Status", strErrorMessage, true);
+            Logger.LogItem("+", strErrorMessage, true);
             boxer::show(strErrorMessage.c_str(), "Fatal Error", boxer::Style::Error);
         }
 
@@ -73,7 +72,7 @@ namespace Dependencies {
         int gitErrorCode;
         git_remote *remote;
 
-        Logger.LogItem("Status", "Checking for updates...");
+        Logger.LogItem("+", "Checking for updates...");
         gitErrorCode = git_remote_lookup(&remote, repo, "origin");
 
         if (gitErrorCode < 0) 
@@ -113,7 +112,6 @@ namespace Dependencies {
         git_merge_analysis_t analysisOut;
         git_merge_preference_t preferenceOut;
 
-        Logger.LogItem("Status", "Analyzing repository information...");
         gitErrorCode = git_merge_analysis(&analysisOut, &preferenceOut, repo, (const git_annotated_commit **) annotatedCommit, 1);
 
         if (gitErrorCode < 0) 
@@ -124,7 +122,7 @@ namespace Dependencies {
 
         if (analysisOut & GIT_MERGE_ANALYSIS_UP_TO_DATE) 
         {
-            Logger.LogItem("Status", "Repository is up to date.");
+            Logger.LogItem("+", "Repository is up to date.");
 
             git_annotated_commit_free(annotatedCommit[0]);
             git_repository_state_cleanup(repo);
@@ -134,7 +132,7 @@ namespace Dependencies {
         } 
         else if (analysisOut & GIT_MERGE_ANALYSIS_FASTFORWARD) 
         {
-            Logger.LogItem("Status", "Fast-forwarding analysis...");
+            Logger.LogItem("+", "Fast-forwarding analysis...");
 
             git_reference *referenceOut;
             git_reference *newTargetReference;
@@ -172,21 +170,19 @@ namespace Dependencies {
         git_libgit2_init();
 
         Logger.LogHead(fmt::format("Package Manager - {} [{} ms]", common_name, duration_cast<milliseconds>(steady_clock::now() - startTime).count()));
-        Logger.LogItem("Path", package_path);
 
         git_repository* repo = nullptr;
         int repositoryOpenStatus = git_repository_open(&repo, package_path.c_str());
 
         switch (repositoryOpenStatus) {
             case GIT_ENOTFOUND: 
             {
-                Logger.LogItem("Status", "Repository was not found...");
+                Logger.LogItem("+", "Repository was not found...");
                 repositoryOpenStatus = CloneRepository(repo, package_path, remote_object);
                 break;
             }
             case 0: // no error occured
             {
-                Logger.LogItem("Status", "Fetching repository head...");
                 repositoryOpenStatus = FetchHead(repo, package_path);
                 break;
             } 
@@ -196,7 +192,7 @@ namespace Dependencies {
             }
         }
 
-        Logger.LogItem("Status", fmt::format("{} in {} ms", repositoryOpenStatus == 0 ? "Succeeded" : "Failed", duration_cast<milliseconds>(steady_clock::now() - startTime).count()), true);
+        Logger.LogItem("+", fmt::format("{} in {} ms", repositoryOpenStatus == 0 ? "Succeeded" : "Failed", duration_cast<milliseconds>(steady_clock::now() - startTime).count()), true);
         // Free resources
         git_repository_free(repo);
         git_libgit2_shutdown();