From 350da3dba4bbd44a2da04cf6cb3cfbf8185506fe Mon Sep 17 00:00:00 2001
From: Lorenzo Bracciale <lorenzo.bracciale@gmail.com>
Date: Thu, 8 Sep 2022 18:26:35 +0200
Subject: [PATCH] logging improved

---
 .gitignore         | 1 +
 eclat.py           | 4 +---
 package_manager.py | 5 +++--
 settings.py        | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/.gitignore b/.gitignore
index 55e28f3..3c3fd8c 100755
--- a/.gitignore
+++ b/.gitignore
@@ -20,3 +20,4 @@ hike/
 !/scripts/temp_credentials/config
 scapy/
 eclatd.log
+log/*.log
diff --git a/eclat.py b/eclat.py
index 6bf4e6b..d6b989e 100755
--- a/eclat.py
+++ b/eclat.py
@@ -44,7 +44,7 @@ def load(scriptfile, package, defines=[]):
 
     # print the outcome
     print(f"{response.status}: {response.message}")
-    
+
     return response
 
 
@@ -165,7 +165,6 @@ def main():
 
     # implementing the commands
     args = parser.parse_args()
-    print(args)
 
     if args.cmd == 'load':
         defines = args.define if hasattr(args, 'define') else []
@@ -186,7 +185,6 @@ def main():
     else:
         parser.error('No command specified.')
 
-    print(f"status: {ret.status}")
     if ret.status == 'OK':
         sys.exit(0)
     else:
diff --git a/package_manager.py b/package_manager.py
index 9effeff..948f3a4 100644
--- a/package_manager.py
+++ b/package_manager.py
@@ -4,7 +4,7 @@
 import os
 import logging
 
-logger = logging.getLogger(__name__)
+logger = logging.getLogger()
 
 class PackageManager:
     """
@@ -34,10 +34,11 @@ def pull(self, package_name):
             if d['name'] == package_name:
                 logger.info(
                     f'Found {package_name} in the package list. Start cloning...')
-                logger.info(d)
+                logger.info(f"Package information retrieved: {str(d)}")
                 is_found = True
                 cal.clone_repo(
                     d['git_url'], f"{settings.COMPONENTS_DIR}/{package_name}", d['tag'])
+                logger.info(f"Package {package_name} cloned successfully")
 
         if not is_found:
             raise Exception(
diff --git a/settings.py b/settings.py
index 5f7ccdd..ab33b23 100755
--- a/settings.py
+++ b/settings.py
@@ -38,7 +38,7 @@
             "formatter":"std_out",
             "class":"logging.FileHandler",
             "level":"DEBUG",
-            "filename":"eclatd.log"
+            "filename":"log/eclatd.log"
         }
     },
     "formatters":{