remove --multiple/-m option from all CLI commands: closes #1262
authorBrian Warner <warner@lothar.com>
Sat, 27 Nov 2010 08:38:09 +0000 (00:38 -0800)
committerBrian Warner <warner@lothar.com>
Sat, 27 Nov 2010 08:38:09 +0000 (00:38 -0800)
commit69b42c6cb7ec5d4a6ba1f54b75d1fceb94ef3fac
tree0e8b407c937e32bf41da5e5e09465cf522758789
parentf36bda278014589ac47091f47cd0411d0b070c8a
remove --multiple/-m option from all CLI commands: closes #1262

I personally used "tahoe start/restart -m ../MY-TESTNET/node*" all the time,
to spin up or update a local testgrid while iterating over new code. However,
with the recent switch from "subprocess.Popen(/bin/twistd)" to "import and
call twistd.run()" in scripts/startstop_node.py (yay fewer processes!),
"start -m" broke, and fixing it requires os.fork, which is unavailable on
windows (boo windows!). And I was probably the only one using -m. So in the
interests of uniformity among platforms and simpler code (yay negative code
days!), we're just removing -m from everything. I will start using a little
shell script or something to simulate the removed functionality.

This patch also cleans up CLI-function calling a bit: get the basedir from
the config dict (instead of sometimes from a separate argument), and always
return a numeric exit code.
src/allmydata/scripts/common.py
src/allmydata/scripts/create_node.py
src/allmydata/scripts/keygen.py
src/allmydata/scripts/runner.py
src/allmydata/scripts/startstop_node.py
src/allmydata/scripts/stats_gatherer.py
src/allmydata/test/test_runner.py