100 + # This is naive; we can make it more sophisticated (e.g., allowing
101 + # escaping dollar signs) if we discover we need it. For now,
102 + # simplicity wins.
103 + replaced_args = args.replace('$INSTALLDIR', buildbot_dir)
Thank you for fixing all the script retrieval functions.
I have to make a note to myself: command helper returns a function!
Also thanks for your change on how the su context manager retrieves the user home directory:
please see the comment on the master MP for a suggestion on how to fix the exceptions problem too.
I've seen now buildslave is created in the install hook (great IMHO), but the same code is still present in config-changed: this could generate errors (buildslave created twice?), and requires further investigation.
Thank you Gary for this nice branch.
100 + # This is naive; we can make it more sophisticated (e.g., allowing '$INSTALLDIR' , buildbot_dir)
101 + # escaping dollar signs) if we discover we need it. For now,
102 + # simplicity wins.
103 + replaced_args = args.replace(
We could also do something like::
formatted_args = args.format( **config)
And then, in the config file::
script-args: "-u buildbot -e <email address hidden> -f 'Launchpad PQM' {installdir}"
It's as naive as your version, but maybe a little more generic?
60 - command('hg', 'clone', source, target)
61 + run('hg', 'clone', source, target)
Thank you for fixing all the script retrieval functions.
I have to make a note to myself: command helper returns a function!
Also thanks for your change on how the su context manager retrieves the user home directory:
please see the comment on the master MP for a suggestion on how to fix the exceptions problem too.
I've seen now buildslave is created in the install hook (great IMHO), but the same code is still present in config-changed: this could generate errors (buildslave created twice?), and requires further investigation.