X-Git-Url: https://git.distorted.org.uk/~mdw/stgit/blobdiff_plain/e7dc3866ae7dc43c8497080de2f541d44dd083d8..3bd6274e247baa8c9f652fcdfd74180b0933773c:/stgit/commands/common.py diff --git a/stgit/commands/common.py b/stgit/commands/common.py index 9747981..88f1433 100644 --- a/stgit/commands/common.py +++ b/stgit/commands/common.py @@ -1,4 +1,4 @@ -"""Function/variables commmon to all the commands +"""Function/variables common to all the commands """ __copyright__ = """ @@ -33,29 +33,29 @@ class CmdException(Exception): # Utility functions -def git_id(string): +def git_id(rev): """Return the GIT id """ - if not string: + if not rev: return None - string_list = string.split('/') - if len(string_list) == 2: - patch_id = string_list[1] + rev_list = rev.split('/') + if len(rev_list) == 2: + patch_id = rev_list[1] if not patch_id: patch_id = 'top' - elif len(string_list) == 1: + elif len(rev_list) == 1: patch_id = 'top' else: patch_id = None - patch_branch = string_list[0].split('@') + patch_branch = rev_list[0].split('@') if len(patch_branch) == 1: series = crt_series elif len(patch_branch) == 2: series = stack.Series(patch_branch[1]) else: - raise CmdException, 'Unknown id: %s' % string + raise CmdException, 'Unknown id: %s' % rev patch_name = patch_branch[0] if not patch_name: @@ -77,11 +77,11 @@ def git_id(string): return series.get_patch(patch_name).get_old_bottom() # base - if patch_name == 'base' and len(string_list) == 1: + if patch_name == 'base' and len(rev_list) == 1: return read_string(series.get_base_file()) # anything else failed - return git.rev_parse(string) + return git.rev_parse(rev) def check_local_changes(): if git.local_changes(): @@ -92,11 +92,11 @@ def check_head_top_equal(): if not crt_series.head_top_equal(): raise CmdException, \ 'HEAD and top are not the same. You probably committed\n' \ - ' changes to the tree ouside of StGIT. If you know what you\n' \ + ' changes to the tree outside of StGIT. If you know what you\n' \ ' are doing, use the "refresh -f" command' def check_conflicts(): - if os.path.exists(os.path.join(git.base_dir, 'conflicts')): + if os.path.exists(os.path.join(git.get_base_dir(), 'conflicts')): raise CmdException, 'Unsolved conflicts. Please resolve them first' def print_crt_patch(branch = None): @@ -130,26 +130,26 @@ def resolved_all(reset = None): if conflicts: for filename in conflicts: resolved(filename, reset) - os.remove(os.path.join(git.base_dir, 'conflicts')) + os.remove(os.path.join(git.get_base_dir(), 'conflicts')) -def name_email(string): +def name_email(address): """Return a tuple consisting of the name and email parsed from a standard 'name ' string """ - string = re.sub('([^\w\s<>@.])', '\\\\\\1', string) - str_list = re.findall('^(.*)\s*<(.*)>\s*$', string) + address = re.sub('([^\w\s<>@.])', '\\\\\\1', address) + str_list = re.findall('^(.*)\s*<(.*)>\s*$', address) if not str_list: - raise CmdException, 'Incorrect "name " string: %s' % string + raise CmdException, 'Incorrect "name " string: %s' % address return str_list[0] -def name_email_date(string): +def name_email_date(address): """Return a tuple consisting of the name, email and date parsed from a 'name date' string """ - string = re.sub('([^\w\s<>@.])', '\\\\\\1', string) - str_list = re.findall('^(.*)\s*<(.*)>\s*(.*)\s*$', string) + address = re.sub('([^\w\s<>@.])', '\\\\\\1', address) + str_list = re.findall('^(.*)\s*<(.*)>\s*(.*)\s*$', address) if not str_list: - raise CmdException, 'Incorrect "name date" string: %s' % string + raise CmdException, 'Incorrect "name date" string: %s' % address return str_list[0]