From b858a8cad04d7229e03a002f56436391a1c9f431 Mon Sep 17 00:00:00 2001 From: Anas Nashif Date: Sat, 30 Jun 2012 10:34:36 -0700 Subject: [PATCH] Revert "Fixed messaging and option help and make it more clear." This reverts commit 7adfa9f83b52c0a41b553f808b2bc5df4ddf4f5c --- gitbuildsys/cmd_remotebuild.py | 6 ++---- tools/gbs | 25 +++++++++++++------------ 2 files changed, 15 insertions(+), 16 deletions(-) diff --git a/gitbuildsys/cmd_remotebuild.py b/gitbuildsys/cmd_remotebuild.py index 6c3f5c9..3ce7e43 100644 --- a/gitbuildsys/cmd_remotebuild.py +++ b/gitbuildsys/cmd_remotebuild.py @@ -112,11 +112,9 @@ def do(opts, args): prj = obspkg.ObsProject(target_prj, apiurl = APISERVER, oscrc = oscrcpath) msger.info('checking status of obs project: %s ...' % target_prj) if prj.is_new(): - # FIXME: How do you know that a certain user does not have permission to - # create any project, anywhewre? if opts.target_obsprj and not target_prj.startswith('home:%s:' % USER): - msger.error('no permission to create project %s, only sub projects'\ - 'of home:%s are allowed ' % (target_prj, USER)) + msger.error('no permission to create project %s, only subpackage '\ + 'of home:%s is allowed ' % (target_prj, USER)) msger.info('creating %s for package build ...' % target_prj) prj.branch_from(base_prj) diff --git a/tools/gbs b/tools/gbs index 784ceee..c6c1d41 100755 --- a/tools/gbs +++ b/tools/gbs @@ -39,7 +39,7 @@ def handle_project(option, opt_str, value, parser): "start with '-'" % (opt_str, value)) if not re.match(r'^(\w|:|\.|-)+$', value): raise errors.Usage("option %s: invalid project name %s, only word " \ - "character, ':', '.' and '-' are supportted" \ + "character, ':', '.' and '-' supportted" \ % (opt_str, value)) setattr(parser.values, option.dest, value) @@ -137,20 +137,20 @@ class Gbs(cmdln.Cmdln): @cmdln.option('-D', '--dist', default=None, dest='dist', - help='Specify distribution configuration file, which should' \ - 'be a full path') + help='Specify distribution configure file, which should' \ + 'be full path') @cmdln.option('-R', '--repository', action="callback", default=None, type='string', dest='repositories', callback=handle_repository, - help='Specify package repositories, Only rpm-md format' \ - 'is supported') + help='Specify package repositories, Supported format ' \ + 'is rpm-md') @cmdln.option('-B', '--buildroot', default=None, dest='buildroot', - help='Specify build root to setup chroot environment') + help='Specify build rootdir to setup chroot environment') @cmdln.option('-A', '--arch', default=None, dest='arch', @@ -159,7 +159,7 @@ class Gbs(cmdln.Cmdln): action='store_true', default=False, dest='clean', - help='Delete old build root before initialization') + help='Delete old build root before initializing it') @cmdln.option('--noinit', action='store_true', default=False, @@ -175,7 +175,7 @@ class Gbs(cmdln.Cmdln): action="store_true", default=False, dest='incremental', - help='Do incremental build, which needs support of spec file,' + help='Do incremental build, which need support of spec file,' ' and tar ball unpack stage should be enabled with the ' 'condition of RPM macro RSYNCDONE') @cmdln.option('--spec', @@ -222,15 +222,16 @@ class Gbs(cmdln.Cmdln): dest='target_obsprj', type='string', callback=handle_project, - help='OBS project where package will be checked in' \ - '(default: home::gbs:Tizen:Main)') + help='OBS target project being used to build package, ' \ + 'use "home::gbs:Tizen:Main" if not specified') @cmdln.option('-B', '--base-obsprj', action='callback', default=None, dest='base_obsprj', type='string', callback=handle_project, - help='OBS project to branch from (default: Tizen:Main)') + help='Base OBS project being used to branch from, ' \ + 'use "Tizen:Main" if not specified') @cmdln.option('--spec', default=None, dest='spec', @@ -238,7 +239,7 @@ class Gbs(cmdln.Cmdln): @cmdln.option('-c', '--commit', default=None, dest='commit', - help='Specify a commit ID to build') + help='Specify a commit to build') def do_remotebuild(self, subcmd, opts, *args): """${cmd_name}: remote build package -- 2.7.4