[Checkins] SVN: grokproject/trunk/ Remove the --newer option for
bin/grokproject. Use the 'newest'
Maurits van Rees
m.van.rees at zestsoftware.nl
Fri May 2 10:18:37 EDT 2008
Log message for revision 86095:
Remove the --newer option for bin/grokproject. Use the 'newest'
option from the template instead.
Changed:
U grokproject/trunk/CHANGES.txt
U grokproject/trunk/grokproject/__init__.py
-=-
Modified: grokproject/trunk/CHANGES.txt
===================================================================
--- grokproject/trunk/CHANGES.txt 2008-05-02 13:48:21 UTC (rev 86094)
+++ grokproject/trunk/CHANGES.txt 2008-05-02 14:18:36 UTC (rev 86095)
@@ -4,6 +4,9 @@
0.8 (unreleased)
------------------
+* Remove the --newer option for bin/grokproject. Use the 'newest'
+ option from the template instead.
+
* Move version_info_url to the template vars without making it a real
question.
Modified: grokproject/trunk/grokproject/__init__.py
===================================================================
--- grokproject/trunk/grokproject/__init__.py 2008-05-02 13:48:21 UTC (rev 86094)
+++ grokproject/trunk/grokproject/__init__.py 2008-05-02 14:18:36 UTC (rev 86095)
@@ -32,14 +32,14 @@
ask_var('user', 'Name of an initial administrator user', default=NoDefault),
ask_var('passwd', 'Password for the initial administrator user',
default=NoDefault, should_echo=False),
- ask_var('newest', 'Check for newer versions of packages', default='false'),
+ ask_var('newest', 'Check for newer versions of packages',
+ default='false', should_ask=False),
ask_var('version_info_url',
"The URL to a *.cfg file containing a [versions] section.",
default=None, should_ask=False),
]
def check_vars(self, vars, cmd):
- #[var.name for var in self.vars if not var.should_ask]
skipped_vars = {}
for var in self.vars:
if not var.should_ask:
@@ -71,6 +71,12 @@
extends = urlparse.urljoin(VERSIONINFO_INFO_URL, info)
vars['extends'] = extends
vars['app_class_name'] = vars['project'].capitalize()
+
+ # We want to have newest be 'false' or 'true'.
+ if vars['newest'].lower() in ('1', 'true'):
+ vars['newest'] = 'true'
+ else:
+ vars['newest'] = 'false'
return vars
def main():
@@ -83,8 +89,6 @@
help="Import project to given repository location (this "
"will also create the standard trunk/ tags/ branches/ "
"hierarchy).")
- parser.add_option('--newer', action="store_true", dest="newer",
- default=False, help="Check for newer versions of packages.")
parser.add_option('-v', '--verbose', action="store_true", dest="verbose",
default=False, help="Be verbose.")
@@ -116,10 +120,6 @@
extra_args = []
extra_args.append('%s=%s' % ('app_class_name', app_class_name))
- if options.newer:
- extra_args.append('newest=true')
- else:
- extra_args.append('newest=false')
# Process the options that override the interactive part of filling
# the templates.
More information about the Checkins
mailing list