[MERGE] Stop treating Permission Denied as lock contention

[John Arbash Meinel]

Aaron Bentley wrote:
> Hi all,
> 
> Currently, bzr treats "Permission Denied" as lock contention.  This is
> bad, because it means that bzr will wait on a lock that will never be
> released.  (Barring an external chmod, etc.)  Treating all PathErrors as
> lock contention is a change originally introduced to support SFTP better.
> 
> This bundle causes "Permission Denied" to specifically be permitted,
> while other classes derived from PathError continue to be treated as
> lock contention.
> 
> This bundle should probably be tested on win32 before we apply it,
> because win32 is known be be a bit weird about the "permission denied"
> error code.
> 
> Aaron


+1 from me, though I agree it should be tested on win32. So +1 (conditional)

John
=:->