Stop recording branch.*.remote to '.' for local parents.

[stgit] / stgit / commands / pull.py

diff --git a/stgit/commands/pull.py b/stgit/commands/pull.py
index 990244ebf77d178bcd1bb6dc30fb40799e65de7a..f551b981d1f3d07a442f36322f384a4251cc1978 100644 (file)
--- a/stgit/commands/pull.py
+++ b/stgit/commands/pull.py
@@ -20,6 +20,7 @@ from optparse import OptionParser, make_option
 
 from stgit.commands.common import *
 from stgit.utils import *
 
 from stgit.commands.common import *
 from stgit.utils import *

+from stgit.config import GitConfigException

 from stgit import stack, git
 
 
 from stgit import stack, git
 
 


@@ -49,13 +50,25 @@ options = [make_option('-n', '--nopush',
 def func(parser, options, args):
     """Pull the changes from a remote repository
     """
 def func(parser, options, args):
     """Pull the changes from a remote repository
     """

-    if len(args) > 1:
-        parser.error('incorrect number of arguments')
+    policy = config.get('branch.%s.stgit.pull-policy' % crt_series.get_branch()) or \
+             config.get('stgit.pull-policy')
+
+    if policy == 'rebase':
+        # parent is local
+        if len(args) == 1:
+            parser.error('specifying a repository is meaningless for policy="%s"' % policy)
+        if len(args) > 0:
+            parser.error('incorrect number of arguments')

 
 

-    if len(args) >= 1:
-        repository = args[0]

     else:
     else:

-        repository = crt_series.get_parent_remote()
+        # parent is remote
+        if len(args) > 1:
+            parser.error('incorrect number of arguments')
+
+        if len(args) >= 1:
+            repository = args[0]
+        else:
+            repository = crt_series.get_parent_remote()

 
     if crt_series.get_protected():
         raise CmdException, 'This branch is protected. Pulls are not permitted'
 
     if crt_series.get_protected():
         raise CmdException, 'This branch is protected. Pulls are not permitted'


@@ -64,14 +77,27 @@ def func(parser, options, args):
     check_conflicts()
     check_head_top_equal()
 
     check_conflicts()
     check_head_top_equal()
 

-    must_rebase = (config.get('stgit.pull-does-rebase') == 'yes')
+    if policy == 'pull':
+        must_rebase = 0
+    elif policy == 'fetch-rebase':
+        must_rebase = 1
+    elif policy == 'rebase':
+        must_rebase = 1
+    else:
+        raise GitConfigException, 'Unsupported pull-policy "%s"' % policy
+

     applied = prepare_rebase(real_rebase=must_rebase, force=options.force)
 
     # pull the remote changes
     applied = prepare_rebase(real_rebase=must_rebase, force=options.force)
 
     # pull the remote changes

-    print 'Pulling from "%s"...' % repository
-    git.fetch(repository)
-    if must_rebase:
+    if policy == 'pull':
+        print 'Pulling from "%s"...' % repository
+        git.pull(repository)
+    elif policy == 'fetch-rebase':
+        print 'Fetching from "%s"...' % repository
+        git.fetch(repository)

         rebase(git.fetch_head())
         rebase(git.fetch_head())

+    elif policy == 'rebase':
+        rebase(crt_series.get_parent_branch())

 
     post_rebase(applied, options.nopush, options.merged)
 
 
     post_rebase(applied, options.nopush, options.merged)