clarify log messages

buildbot · Jan 21, 2014 · 34a9530 · 34a9530
1 parent b7f6032
commit 34a9530
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/master/buildbot/process/buildstep.py b/master/buildbot/process/buildstep.py
@@ -729,7 +729,7 @@ def __init__(self, logfiles=None, lazylogfiles=None, log_eval_func=None,
 
         if any([x is not None for x in [logfiles, lazylogfiles, log_eval_func]]):
             config.error(
-                "'logfiles', 'lazylogfiles', 'log_eval_func' paramaters are no longer available")
+                "The LoggingBuildStep parameters 'logfiles', 'lazylogfiles' and 'log_eval_func' are no longer available")
         self.addLogObserver('stdio', OutputProgressObserver("output"))
 
     def addLogFile(self, logname, filename):
@@ -743,7 +743,7 @@ def setupLogsRunCommandAndProcessResults(self, cmd, stdioLog=None, closeLogWhenF
             logfiles = {}
         if logfiles:
             cmd.setLogFiles(logfiles)
-        log.msg("%s.startCommandWithoutEnding(cmd=%s)" % (self.__class__.__name__, cmd))
+        log.msg("%s.setupLogsRunCommandAndProcessResults(cmd=%s)" % (self.__class__.__name__, cmd))
         log.msg("  cmd.args = %r" % (cmd.args,))
         self.cmd = cmd  # so we can interrupt it
         # TODO: when the status.LogFile object no longer exists, then this

diff --git a/master/buildbot/test/unit/test_process_buildstep.py b/master/buildbot/test/unit/test_process_buildstep.py
@@ -326,8 +326,9 @@ def test_isNewStyle(self):
 
 
 class TestLoggingBuildStep(config.ConfigErrorsMixin, unittest.TestCase):
-    deprecatedMsg = ("'logfiles', 'lazylogfiles', 'log_eval_func' paramaters "
-                     "are no longer available")
+    deprecatedMsg = ("The LoggingBuildStep parameters 'logfiles', "
+                     "'lazylogfiles' and 'log_eval_func' are no longer "
+                     "available")
 
     def makeRemoteCommand(self, rc, stdout, stderr=''):
         cmd = fakeremotecommand.FakeRemoteCommand('cmd', {})