[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