[merge] remove SFTPLock and support for weaves over SFTP

John Arbash Meinel john at arbash-meinel.com
Mon Jan 26 18:19:18 GMT 2009


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Martin Pool wrote:
> This removes lock_write and lock_read on SFTPTransport so that they
> just raise TransportNotPossible; as a consequence you can no longer
> directly access weave branches (which are an extremely old format)
> over SFTP.
> 
> 

So IIRC, Weave repos used an OS lock, but for SFTP access we created a
file with O_EXCL. Which meant that locking over SFTP was not actually
lock-safe versus local access. (It was safe versus another SFTP access,
which was, in general, good enough, but we went a different route to be
truly safe against all forms of access.)

So I'm happy to remove it, though it might be nice to have a good error
of "cannot access Weave repositories over sftp" somewhere.

What brought this up, btw?

John
=:->

BB:approve
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkl9/qYACgkQJdeBCYSNAAMiiwCdHhm9e5qkZBO4CdxFzhI/sqnv
IK0AnRgsxSPioZov9f5icH6SDDLORTmF
=JVpL
-----END PGP SIGNATURE-----



More information about the bazaar mailing list