--- a/mercurial/subrepo.py	Tue Jan 10 22:35:25 2012 +0800
+++ b/mercurial/subrepo.py	Tue Jan 10 13:48:42 2012 -0600
@@ -484,8 +484,8 @@
                 self._repo.ui.status(_('pulling subrepo %s from %s\n')
                                      % (subrelpath(self), srcurl))
                 self._repo.pull(other)
-            bookmarks.updatefromremote(self._repo.ui, self._repo, other,
-                                       srcurl)
+                bookmarks.updatefromremote(self._repo.ui, self._repo, other,
+                                           srcurl)
 
     def get(self, state, overwrite=False):
         self._get(state)