[MERGE][#299254] Fix RemoteTransport's translation of errors involving paths; it wasn't passing orig_path to _translate_error.

[John Arbash Meinel]

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

Vincent Ladeuil wrote:
> Vincent Ladeuil has voted approve.
> Status is now: Approved
> Comment:
> I'm not a big fan of error translation factorization because it makes it
> hard to refine diagnosis under some circumstances. But since this patch
> doesn't make that part worse, I think it should go in. And in that
> particular case, having a single point to translate the errors may be
> more appropriate than usual (since a serialization is involved anyway).
> 
> I also find a bit strange (not to say it violates some abstraction) that
> 'orig_path' seems to pop out of nowhere but at least the comment
> explains that.
> 
> In a followup patch, you may want to review the other calls to
> _translate_error to make them more consistent (_readv for example
> doesn't pass relpath as a parameter and the parameter name itself is a
> bit unclear since path and orig_path are used in different places).

Just as a follow up, why call it "orig_path" rather than "relpath" like
all of the transport functions call it.

Otherwise this seems fine to me.

John
=:->

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

iEYEARECAAYFAkklgS0ACgkQJdeBCYSNAAN7sgCdFhs883cPAbMg8J9aeX/lOcwp
v6QAnRRzqJy73519hyP4CxVULzH1Lo96
=Tx1t
-----END PGP SIGNATURE-----