diff options
| author | Chris McDonough <chrism@plope.com> | 2011-10-09 00:10:53 -0400 |
|---|---|---|
| committer | Chris McDonough <chrism@plope.com> | 2011-10-09 00:10:53 -0400 |
| commit | b77e7feba77b29adbe47ac0e5c18a905bb7555e8 (patch) | |
| tree | aee0b9db31d6ca3ab415a08907eb2654d9d5dad0 | |
| parent | cebd4345494eb5507099770022b40575590215c0 (diff) | |
| download | pyramid-b77e7feba77b29adbe47ac0e5c18a905bb7555e8.tar.gz pyramid-b77e7feba77b29adbe47ac0e5c18a905bb7555e8.tar.bz2 pyramid-b77e7feba77b29adbe47ac0e5c18a905bb7555e8.zip | |
get options vs. command attrs right
| -rw-r--r-- | pyramid/scaffolds/template.py | 6 | ||||
| -rw-r--r-- | pyramid/scripts/pcreate.py | 8 | ||||
| -rw-r--r-- | pyramid/tests/test_scaffolds/test_template.py | 11 |
3 files changed, 13 insertions, 12 deletions
diff --git a/pyramid/scaffolds/template.py b/pyramid/scaffolds/template.py index 4ffa946d3..c7797cc1d 100644 --- a/pyramid/scaffolds/template.py +++ b/pyramid/scaffolds/template.py @@ -65,7 +65,7 @@ class Template(object): template_dir = self.template_dir() if not self.exists(output_dir): self.out("Creating directory %s" % output_dir) - if not command.simulate: + if not command.options.simulate: # Don't let copydir create this top-level directory, # since copydir will svn add it sometimes: self.makedirs(output_dir) @@ -73,9 +73,9 @@ class Template(object): template_dir, output_dir, vars, - verbosity=command.verbose, + verbosity=command.verbosity, simulate=command.options.simulate, - interactive=command.interactive, + interactive=command.options.interactive, overwrite=command.options.overwrite, indent=1, template_renderer=self.template_renderer diff --git a/pyramid/scripts/pcreate.py b/pyramid/scripts/pcreate.py index dfd9f691b..5c8d3120e 100644 --- a/pyramid/scripts/pcreate.py +++ b/pyramid/scripts/pcreate.py @@ -15,9 +15,7 @@ def main(argv=sys.argv, quiet=False): return command.run() class PCreateCommand(object): - verbose = True - interactive = False - simulate = False + verbosity = 1 usage = "usage: %prog [options] distribution_name" parser = optparse.OptionParser(usage) parser.add_option('-s', '--scaffold', @@ -37,6 +35,10 @@ class PCreateCommand(object): dest='overwrite', action='store_true', help='Always overwrite') + parser.add_option('--interactive', + dest='interactive', + action='store_true', + help='When a file would be overwritten, interrogate') def __init__(self, argv, quiet=False): self.quiet = quiet diff --git a/pyramid/tests/test_scaffolds/test_template.py b/pyramid/tests/test_scaffolds/test_template.py index 88a31e802..e674a338d 100644 --- a/pyramid/tests/test_scaffolds/test_template.py +++ b/pyramid/tests/test_scaffolds/test_template.py @@ -85,8 +85,8 @@ class TestTemplate(unittest.TestCase): self.assertEqual(copydir.kw, {'template_renderer':inst.template_renderer, 'indent':1, - 'verbosity':True, - 'simulate':True, + 'verbosity':1, + 'simulate':False, 'overwrite':False, 'interactive':False, }) @@ -125,13 +125,12 @@ class DummyCopydir(object): self.kw = kw class DummyOptions(object): - simulate = True + simulate = False overwrite = False + interactive = False class DummyCommand(object): options = DummyOptions() - verbose = True - interactive = False - simulate = False + verbosity = 1 |
