Changeset be6ff03

Timestamp:

Jan 11, 2012, 12:57:32 AM (12 years ago)

Author:

Jessica B. Hamrick <jhamrick@…>

Branches:

no-cups

Children:

eee389a

Parents:

b01b6d1

git-author:

Jessica B. Hamrick <jhamrick@…> (01/11/12 00:57:32)

git-committer:

Jessica B. Hamrick <jhamrick@…> (01/11/12 00:57:32)

Message:

Log mplayer output better and run it in slave mode

File:

• server/lib/gutenbach/server/job.py (modified) (1 diff)

server/lib/gutenbach/server/job.py

@@ -116 +116 @@
     def play(self):
         logger.info("playing job %s" % str(self))
-        # TODO: add external call to music player
         self.status = States.PROCESSING
         self.player = subprocess.Popen(
-            "/usr/bin/mplayer -quiet %s" % self.document.name,
-            shell=True)
-            #stderr=subprocess.PIPE,
-            #stdout=subprocess.PIPE)
+            "/usr/bin/mplayer -really-quiet -slave %s" % self.document.name,
+            shell=True,
+            stderr=subprocess.PIPE,
+            stdout=subprocess.PIPE)
         while self.player.poll() is None:
             time.sleep(0.1)
         logger.info("mplayer finished with code %d" % self.player.returncode)
-        #if self.player.returncode < 0:
-        #    logger.error(self.player.stderr)
-        #logger.debug(self.player.stdout)
+        stderr = self.player.stderr.read()
+        stdout = self.player.stdout.read()
+        if stderr.strip() != "":
+            logger.error(stderr)
+        logger.debug(stdout)
         self.player = None
         self.printer.complete_job(self.id)