[Bug 86919] Re: transport shouldn't indirect through control_files

2011-01-20 Thread Jelmer Vernooij
** Changed in: bzr (Ubuntu)
   Status: Triaged = Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/86919

Title:
  transport shouldn't indirect through control_files

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 86919] Re: transport shouldn't indirect through control_files

2009-08-26 Thread Martin Pool
The use of LockableFiles as a wrapper for a Transport no longer exists.

This class could still be cleaned up a bit, for which see bug 88848.

** Changed in: bzr
   Status: Confirmed = Fix Released

-- 
transport shouldn't indirect through control_files
https://bugs.launchpad.net/bugs/86919
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 86919] Re: transport shouldn't indirect through control_files

2009-07-27 Thread Martin Pool
bzr no longer indirects through control_files for IO-type operations,
which was the original purpose of this bug.   The main purpose of
LockableFiles as it currently exists is to coordinate a transaction
object with the state of the lock.   It also still provides the
_find_modes method.

_find_modes is already softly deprecated in favor of
BzrDir._find_creation_modes; if/when the class as a whole is deprecated
the attributes that it creates will be gone.  Alternatively we could
turn them into deprecated properties.

If this reasoning is correct, then Branch, which does not use
get_transaction, should never need a control_files.  (Removing it may
strictly be a compatibility break for Branch though.)

There are also quite a large number of methods on Branch that just
forward to the control_files, that can go to a CountedLock instead.

-- 
transport shouldn't indirect through control_files
https://bugs.launchpad.net/bugs/86919
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 86919] Re: transport shouldn't indirect through control_files

2009-02-09 Thread Andrea Corbellini
** Changed in: bzr (Ubuntu)
   Importance: Undecided = Low
   Status: New = Triaged

-- 
transport shouldn't indirect through control_files
https://bugs.launchpad.net/bugs/86919
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 86919] Re: transport shouldn't indirect through control_files

2008-04-16 Thread Martin Pool
** Also affects: bzr
   Importance: Undecided
   Status: New

** Changed in: bzr
   Importance: Undecided = Medium
 Assignee: (unassigned) = Martin Pool (mbp)
   Status: New = Confirmed

-- 
transport shouldn't indirect through control_files
https://bugs.launchpad.net/bugs/86919
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs