Changeset 31 for mergebot


Ignore:
Timestamp:
Dec 18, 2009 3:41:19 AM (15 years ago)
Author:
retracile
Message:

subprocess.check_call is new in 2.5, use subprocess.call instead

Location:
mergebot/trunk/mergebot
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • mergebot/trunk/mergebot/ticket_actions.py

    r17 r31  
    11import socket
    22import time
    3 from subprocess import check_call
     3from subprocess import call
    44
    55from genshi.builder import tag
     
    6666
    6767    def start_daemon(self):
    68         check_call(['mergebotdaemon', self.env.path])
     68        retval = call(['mergebotdaemon', self.env.path])
     69        if retval:
     70            raise Exception('mergebotdaemon failed to start (%s)' % retval)
    6971        time.sleep(1) # bleh
    7072
  • mergebot/trunk/mergebot/web_ui.py

    r17 r31  
    55import os
    66import base64
    7 from subprocess import check_call
     7from subprocess import call
    88
    99from trac.core import *
     
    9696
    9797    def start_daemon(self):
    98         check_call(['mergebotdaemon', self.env.path])
     98        retval = call(['mergebotdaemon', self.env.path])
     99        if retval:
     100            raise Exception('mergebotdaemon failed to start' % retval)
    99101        time.sleep(1) # bleh
    100102
Note: See TracChangeset for help on using the changeset viewer.