[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mercurial-devel
Subject:    [PATCH 2 of 2 STABLE] statichttprepo: implement wlock() (issue5613)
From:       Gregory Szorc <gregory.szorc () gmail ! com>
Date:       2017-07-29 20:05:53
Message-ID: c258cf8ca58011edd324.1501358753 () ubuntu-vm-main
[Download RAW message or body]

# HG changeset patch
# User Gregory Szorc <gregory.szorc@gmail.com>
# Date 1501357856 25200
#      Sat Jul 29 12:50:56 2017 -0700
# Branch stable
# Node ID c258cf8ca58011edd324feb65464e1f2db07b4a4
# Parent  e0500c5adb19b60203274bc2f63e500b9b73bf2d
statichttprepo: implement wlock() (issue5613)

statichttprepo inherits from localrepository. In doing so, it
obtains default implementations of various methods, like wlock().

Before this change, tags cache writing would call repo.wlock().
This failed on statichttprepo due to localrepository's wlock()
looking for an instance attribute that doesn't exist on statichttprepo
(statichttprepo doesn't call localrepository.__init__).

We /could/ define missing attributes until the base wlock() works.
However, a statichttprepo is remote and read-only and can't be
locked. The class already has a lock() that short circuits. So
it makes sense to implement a short-circuited wlock() as well. That
is what this patch does.

LockError is expected to be raised when locking fails. The constructor
takes a number of arguments that are local repository centric. Rather
than rework LockError to not require them (which would not be
appropriate for stable), this commit populates dummy values. I don't
believe they'll ever be seen by the user, as lock failures on
static http repos should be limited to well-defined (and tested)
scenarios. We can and should revisit the LockError type to improve
this.

diff --git a/mercurial/statichttprepo.py b/mercurial/statichttprepo.py
--- a/mercurial/statichttprepo.py
+++ b/mercurial/statichttprepo.py
@@ -182,6 +182,10 @@ class statichttprepository(localrepo.loc
     def peer(self):
         return statichttppeer(self)
 
+    def wlock(self, wait=True):
+        raise error.LockUnavailable(0, _('lock not available'), 'lock',
+                                    _('cannot lock static-http repository'))
+
     def lock(self, wait=True):
         raise error.Abort(_('cannot lock static-http repository'))
 
diff --git a/tests/test-static-http.t b/tests/test-static-http.t
--- a/tests/test-static-http.t
+++ b/tests/test-static-http.t
@@ -187,12 +187,22 @@ Clone with tags and branches works
 
 Clone a specific branch works
 
-  $ hg clone -r mybranch static-http://localhost:$HGPORT/remote-with-names \
                local-with-names-branch 2> /dev/null
-  [1]
+  $ hg clone -r mybranch static-http://localhost:$HGPORT/remote-with-names \
local-with-names-branch +  adding changesets
+  adding manifests
+  adding file changes
+  added 4 changesets with 4 changes to 2 files
+  updating to branch mybranch
+  2 files updated, 0 files merged, 0 files removed, 0 files unresolved
 
 Clone a specific tag works
 
-  $ hg clone -r default-tag static-http://localhost:$HGPORT/remote-with-names \
                local-with-names-tag 2> /dev/null
-  [1]
+  $ hg clone -r default-tag static-http://localhost:$HGPORT/remote-with-names \
local-with-names-tag +  adding changesets
+  adding manifests
+  adding file changes
+  added 2 changesets with 2 changes to 1 files
+  updating to branch default
+  1 files updated, 0 files merged, 0 files removed, 0 files unresolved
 
   $ killdaemons.py
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


[prev in list] [next in list] [prev in thread] [next in thread]