bug(stacked-git#83): fix stg pull when no upstream is configured

Author: NonLogicalDev

stgit/commands/pull.py

@@ -101,6 +101,7 @@ def func(parser, options, args):
         'stgit.pull-policy'
     )
 
+    remote_name = None
     if policy == 'rebase':
         # parent is local
         if len(args) == 1:
@@ -119,6 +120,13 @@ def func(parser, options, args):
         else:
             remote_name = stack.parent_remote
 
+    if remote_name is None:
+        parser.error(
+            'There is no tracking information for the current branch.\n'
+            'Please specify which branch you want to merge with.'
+        )
+
+
     if stack.protected:
         raise CmdException('This branch is protected. Pulls are not permitted')
 

t/t1210-pull.sh

@@ -0,0 +1,41 @@
+#!/bin/sh
+#
+# Copyright (c) 2006 Yann Dirson
+#
+
+test_description='test
+
+'
+
+. ./test-lib.sh
+
+# Need a repo to clone
+test_create_repo upstream
+
+test_expect_success \
+    'Setup upstream repo, clone it, and add patches to the clone' \
+    '
+    (cd upstream && stg init) &&
+    stg clone upstream clone
+    (cd clone && git config pull.rebase false)
+    '
+
+test_expect_success \
+    'Test that pull works' \
+    '
+    (cd clone &&
+      git checkout master &&
+      stg pull
+    )
+    '
+
+test_expect_success \
+    'Test that pull without upstream setup produces friendly error' \
+    '
+    (cd clone &&
+      stg branch --create without-upstream &&
+      ( stg pull 2>&1 | ( ! grep -q "Traceback" ) ) || ! stg pull 2>&1
+    )
+    '
+
+test_done