phase1: use Interpolate instead of WithProperties
authorThibaut VARÈNE <hacks@slashdirt.org>
Mon, 25 Jun 2018 10:27:00 +0000 (12:27 +0200)
committerJo-Philipp Wich <jo@mein.io>
Tue, 26 Jun 2018 20:09:13 +0000 (22:09 +0200)
commitd1a92bab36ff9818f56d60d173ff438db5bafe47
tree783418e7a8151363a46a9b934386ea3c73cdee1f
parentb3532e022603aa5c038c69f12079dd1887f204ec
phase1: use Interpolate instead of WithProperties

WithProperties is marked as deprecated since buildbot 0.8.7.

Trying to make the config future-proof, this patch replaces every
instance of WithProperties() by the corresponding Interpolate()
equivalent.

As of buildbot 0.8.9 it appears that Interpolate() doesn't correctly
handle integer format with parameters that call a function, so make
GetNumJobs() return a string:

  File "phase1/master.cfg", line 672, in <module>
    command = ["make", Interpolate("-j%(kw:jobs)d", jobs=GetNumJobs), "tools/tar/compile", "V=s"],
    fmtstring % dd
exceptions.TypeError: %d format: a number is required, not str
Configuration Errors:
  error while parsing config file: %d format: a number is required, not str (traceback in logfile)

Signed-off-by: Thibaut VARÈNE <hacks@slashdirt.org>
phase1/master.cfg