X-Git-Url: https://git.distorted.org.uk/~mdw/stgit/blobdiff_plain/a2597fc74859f62872cef838377568811f2b6362..0f92637c75d03080d43a9b8a95527660d61d67a5:/stgit/git.py diff --git a/stgit/git.py b/stgit/git.py index 7d99338..86630ce 100644 --- a/stgit/git.py +++ b/stgit/git.py @@ -19,10 +19,12 @@ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA """ import sys, os, popen2, re, gitmergeonefile +from shutil import copyfile from stgit import basedir from stgit.utils import * from stgit.config import config +from sets import Set # git exception class class GitException(Exception): @@ -38,6 +40,7 @@ class Person: """An author, committer, etc.""" def __init__(self, name = None, email = None, date = '', desc = None): + self.name = self.email = self.date = None if name or email or date: assert not desc self.name = name @@ -295,14 +298,18 @@ def set_head_file(ref): [os.path.join('refs', 'heads', ref)]) != 0: raise GitException, 'Could not set head to "%s"' % ref +def set_branch(branch, val): + """Point branch at a new commit object.""" + if __run('git-update-ref', [branch, val]) != 0: + raise GitException, 'Could not update %s to "%s".' % (branch, val) + def __set_head(val): """Sets the HEAD value """ global __head if not __head or __head != val: - if __run('git-update-ref HEAD', [val]) != 0: - raise GitException, 'Could not update HEAD to "%s".' % val + set_branch('HEAD', val) __head = val # only allow SHA1 hashes @@ -426,6 +433,71 @@ def add(names): if __run('git-update-index --add --', files): raise GitException, 'Unable to add file' +def __copy_single(source, target, target2=''): + """Copy file or dir named 'source' to name target+target2""" + + # "source" (file or dir) must match one or more git-controlled file + realfiles = _output_lines(['git-ls-files', source]) + if len(realfiles) == 0: + raise GitException, '"%s" matches no git-controled files' % source + + if os.path.isdir(source): + # physically copy the files, and record them to add them in one run + newfiles = [] + re_string='^'+source+'/(.*)$' + prefix_regexp = re.compile(re_string) + for f in [f.strip() for f in realfiles]: + m = prefix_regexp.match(f) + if not m: + print '"%s" does not match "%s"' % (f, re_string) + assert(m) + newname = target+target2+'/'+m.group(1) + if not os.path.exists(os.path.dirname(newname)): + os.makedirs(os.path.dirname(newname)) + copyfile(f, newname) + newfiles.append(newname) + + add(newfiles) + else: # files, symlinks, ... + newname = target+target2 + copyfile(source, newname) + add([newname]) + + +def copy(filespecs, target): + if os.path.isdir(target): + # target is a directory: copy each entry on the command line, + # with the same name, into the target + target = target.rstrip('/') + + # first, check that none of the children of the target + # matching the command line aleady exist + for filespec in filespecs: + entry = target+ '/' + os.path.basename(filespec.rstrip('/')) + if os.path.exists(entry): + raise GitException, 'Target "%s" already exists' % entry + + for filespec in filespecs: + filespec = filespec.rstrip('/') + basename = '/' + os.path.basename(filespec) + __copy_single(filespec, target, basename) + + elif os.path.exists(target): + raise GitException, 'Target "%s" exists but is not a directory' % target + elif len(filespecs) != 1: + raise GitException, 'Cannot copy more than one file to non-directory' + + else: + # at this point: len(filespecs)==1 and target does not exist + + # check target directory + targetdir = os.path.dirname(target) + if targetdir != '' and not os.path.isdir(targetdir): + raise GitException, 'Target directory "%s" does not exist' % targetdir + + __copy_single(filespecs[0].rstrip('/'), target) + + def rm(files, force = False): """Remove a file from the repository """ @@ -449,12 +521,9 @@ def user(): """ global __user if not __user: - if config.has_option('user', 'name') \ - and config.has_option('user', 'email'): - __user = Person(config.get('user', 'name'), - config.get('user', 'email')) - else: - raise GitException, 'unknown user details' + name=config.get('user.name') + email=config.get('user.email') + __user = Person(name, email) return __user; def author(): @@ -607,13 +676,15 @@ def merge(base, head1, head2, recursive = False): """ refresh_index() + err_output = None if recursive: # this operation tracks renames but it is slower (used in # general when pushing or picking patches) try: # use _output() to mask the verbose prints of the tool _output('git-merge-recursive %s -- %s %s' % (base, head1, head2)) - except GitException: + except GitException, ex: + err_output = str(ex) pass else: # the fast case where we don't track renames (used when the @@ -641,6 +712,11 @@ def merge(base, head1, head2, recursive = False): files[path][stage] = (mode, hash) + if err_output and not files: + # if no unmerged files, there was probably a different type of + # error and we have to abort the merge + raise GitException, err_output + # merge the unmerged files errors = False for path in files: @@ -695,20 +771,28 @@ def status(files = None, modified = False, new = False, deleted = False, else: print '%s' % fs[1] -def diff(files = None, rev1 = 'HEAD', rev2 = None, out_fd = None): +def diff(files = None, rev1 = 'HEAD', rev2 = None, out_fd = None, + binary = False): """Show the diff between rev1 and rev2 """ if not files: files = [] + args = [] + if binary: + args.append('--binary') + if rev1 and rev2: - diff_str = _output(['git-diff-tree', '-p', rev1, rev2, '--'] + files) + diff_str = _output(['git-diff-tree', '-p'] + args + + [rev1, rev2, '--'] + files) elif rev1 or rev2: refresh_index() if rev2: - diff_str = _output(['git-diff-index', '-p', '-R', rev2, '--'] + files) + diff_str = _output(['git-diff-index', '-p', '-R'] + + args + [rev2, '--'] + files) else: - diff_str = _output(['git-diff-index', '-p', rev1, '--'] + files) + diff_str = _output(['git-diff-index', '-p'] + + args + [rev1, '--'] + files) else: diff_str = '' @@ -807,19 +891,37 @@ def reset(files = None, tree_id = None, check_out = True): if not files: __set_head(tree_id) +def fetch(repository = 'origin', refspec = None): + """Fetches changes from the remote repository, using 'git-fetch' + by default. + """ + # we update the HEAD + __clear_head_cache() + + args = [repository] + if refspec: + args.append(refspec) + + command = config.get('branch.%s.stgit.fetchcmd' % get_head_file()) or \ + config.get('stgit.fetchcmd') + if __run(command, args) != 0: + raise GitException, 'Failed "%s %s"' % (command, repository) + def pull(repository = 'origin', refspec = None): - """Pull changes from the remote repository. At the moment, just - use the 'git-pull' command + """Fetches changes from the remote repository, using 'git-pull' + by default. """ - # 'git-pull' updates the HEAD + # we update the HEAD __clear_head_cache() args = [repository] if refspec: args.append(refspec) - if __run(config.get('stgit', 'pullcmd'), args) != 0: - raise GitException, 'Failed "git-pull %s"' % repository + command = config.get('branch.%s.stgit.pullcmd' % get_head_file()) or \ + config.get('stgit.pullcmd') + if __run(command, args) != 0: + raise GitException, 'Failed "%s %s"' % (command, repository) def repack(): """Repack all objects into a single pack @@ -874,10 +976,100 @@ def clone(repository, local_dir): raise GitException, 'Failed "git-clone %s %s"' \ % (repository, local_dir) -def modifying_revs(files, base_rev): +def modifying_revs(files, base_rev, head_rev): """Return the revisions from the list modifying the given files """ - cmd = ['git-rev-list', '%s..' % base_rev, '--'] + cmd = ['git-rev-list', '%s..%s' % (base_rev, head_rev), '--'] revs = [line.strip() for line in _output_lines(cmd + files)] return revs + + +def refspec_localpart(refspec): + m = re.match('^[^:]*:([^:]*)$', refspec) + if m: + return m.group(1) + else: + raise GitException, 'Cannot parse refspec "%s"' % line + +def refspec_remotepart(refspec): + m = re.match('^([^:]*):[^:]*$', refspec) + if m: + return m.group(1) + else: + raise GitException, 'Cannot parse refspec "%s"' % line + + +def __remotes_from_config(): + return config.sections_matching(r'remote\.(.*)\.url') + +def __remotes_from_dir(dir): + d = os.path.join(basedir.get(), dir) + if os.path.exists(d): + return os.listdir(d) + else: + return None + +def remotes_list(): + """Return the list of remotes in the repository + """ + + return Set(__remotes_from_config()) | \ + Set(__remotes_from_dir('remotes')) | \ + Set(__remotes_from_dir('branches')) + +def remotes_local_branches(remote): + """Returns the list of local branches fetched from given remote + """ + + branches = [] + if remote in __remotes_from_config(): + for line in config.getall('remote.%s.fetch' % remote): + branches.append(refspec_localpart(line)) + elif remote in __remotes_from_dir('remotes'): + stream = open(os.path.join(basedir.get(), 'remotes', remote), 'r') + for line in stream: + # Only consider Pull lines + m = re.match('^Pull: (.*)\n$', line) + if m: + branches.append(refspec_localpart(m.group(1))) + stream.close() + elif remote in __remotes_from_dir('branches'): + # old-style branches only declare one branch + branches.append('refs/heads/'+remote); + else: + raise GitException, 'Unknown remote "%s"' % remote + + return branches + +def identify_remote(branchname): + """Return the name for the remote to pull the given branchname + from, or None if we believe it is a local branch. + """ + + for remote in remotes_list(): + if branchname in remotes_local_branches(remote): + return remote + + # if we get here we've found nothing, the branch is a local one + return None + +def fetch_head(): + """Return the git id for the tip of the parent branch as left by + 'git fetch'. + """ + + fetch_head=None + stream = open(os.path.join(basedir.get(), 'FETCH_HEAD'), "r") + for line in stream: + # Only consider lines not tagged not-for-merge + m = re.match('^([^\t]*)\t\t', line) + if m: + if fetch_head: + raise GitException, "StGit does not support multiple FETCH_HEAD" + else: + fetch_head=m.group(1) + stream.close() + + # here we are sure to have a single fetch_head + return fetch_head