Merge pull request #55 from djfun/logging

using the builtin logging module

1 files changed, 11 insertions, 5 deletions

diff --git a/src/components/spectrum.py b/src/components/spectrum.py
index 666e20a..32763c0 100644
--- a/src/components/spectrum.py
+++ b/src/components/spectrum.py
@@ -4,6 +4,7 @@ import os
 import math
 import subprocess
 import time
+import logging
 
 from component import Component
 from toolkit.frame import BlankFrame, scale
@@ -13,6 +14,9 @@ from toolkit.ffmpeg import (
 )
 
 
+log = logging.getLogger('AVP.Components.Spectrum')
+
+
 class Component(Component):
     name = 'Spectrum'
     version = '1.0.0'
@@ -68,6 +72,7 @@ class Component(Component):
         if not changedSize \
                 and not self.changedOptions \
                 and self.previewFrame is not None:
+            log.debug('Comp #%s is reusing old preview frame' % self.compPos)
             return self.previewFrame
 
         frame = self.getPreviewFrame()
@@ -131,13 +136,14 @@ class Component(Component):
             '-frames:v', '1',
         ])
         logFilename = os.path.join(
-            self.core.dataDir, 'preview_%s.log' % str(self.compPos))
-        with open(logFilename, 'w') as log:
-            log.write(" ".join(command) + '\n\n')
-        with open(logFilename, 'a') as log:
+            self.core.logDir, 'preview_%s.log' % str(self.compPos))
+        log.debug('Creating ffmpeg process (log at %s)' % logFilename)
+        with open(logFilename, 'w') as logf:
+            logf.write(" ".join(command) + '\n\n')
+        with open(logFilename, 'a') as logf:
             pipe = openPipe(
                 command, stdin=subprocess.DEVNULL, stdout=subprocess.PIPE,
-                stderr=log, bufsize=10**8
+                stderr=logf, bufsize=10**8
             )
         byteFrame = pipe.stdout.read(self.chunkSize)
         closePipe(pipe)