Added logging to /var/log/presto.log

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

1 files changed, 11 insertions, 1 deletions

diff --git a/presto.py b/presto.py
index 1427a90..3db6e93 100644
--- a/presto.py
+++ b/presto.py
@@ -29,12 +29,15 @@ import deltarpm
 from prestoRepo import PrestoRepository
 from prestomdparser import PrestoMDParser
 import prestoTransaction
+import prestoLog
 
 requires_api_version = '2.1'
+LOG_FILE = "/var/log/presto.log"
 plugin_type = (TYPE_INTERACTIVE,)
 
 rpm_size = 0
 drpm_size = 0
+was_drpm = False
 
 # Configuration stuff
 def config_hook(conduit):    
@@ -74,6 +77,7 @@ def prereposetup_hook(conduit):
 def postresolve_hook(conduit):
     global rpm_size
     global drpm_size
+    global was_drpm
     
     opts, commands = conduit.getCmdLine()
     if not opts.disablepresto:
@@ -85,6 +89,7 @@ def postresolve_hook(conduit):
                 # If a drpm was found, change certain package information so it reflects
                 # the drpm, not the rpm.
                 if chosen_drpm != None:
+                    was_drpm = True
                     rpm_size += int(newpkg.po.simple['packagesize'])
                     drpm_size += int(chosen_drpm['size'])
                     newpkg.po.hasdrpm = True
@@ -123,8 +128,10 @@ def postresolve_hook(conduit):
                 del active_repo.p_repo
 
 def postdownload_hook(conduit):
+    global was_drpm
+    
     opts, commands = conduit.getCmdLine()
-    if not opts.disablepresto:
+    if not opts.disablepresto and was_drpm:
         failure = False
         # Cycle through packages to see if we've downloaded a deltarpm
         conduit.info(2, "Rebuilding full packages from deltas")
@@ -171,8 +178,11 @@ def postdownload_hook(conduit):
 def posttrans_hook(conduit):
     global rpm_size
     global drpm_size
+    global LOG_FILE
     
     if rpm_size > 0:
+        prestoLog.log(conduit, LOG_FILE, rpm_size, drpm_size)
+                    
         conduit.info(2, "Size of all updates downloaded from Presto-enabled repositories: %i bytes" % drpm_size)
         conduit.info(2, "Size updates would have been downloaded if Presto wasn't enabled: %i bytes" % rpm_size)
         conduit.info(2, "This is a savings of %i percent" % (100 - ((drpm_size * 100) / rpm_size)))