Re: [RESEND][PATCH] Fix diffing unmodified against modified remote file

Patrick Steinhardt <[email protected]>
Newsgroups gmane.comp.version-control.subversion.tortoisesvn.devel
Message-ID <20170321073909.GA475@pks-xps>
On Mon, Mar 20, 2017 at 08:37:41PM +0100, Stefan Küng wrote:
> On 20.03.2017 12:51, Patrick Steinhardt wrote:
> 
> > When having an unmodified but out-of-date local file which is to
> > be diffed with an updated remote file (via "Check for
> > modifications" -> "Check repository" -> "Compare with HEAD and
> > BASE"), the resulting diff window is confusing the local and
> > remote files. The left side will display the read-only local file
> > while the right side will display the writable remote file, which
> > is obviously wrong.
> 
> Not really, no: the right side of the diff should always show the newest 
> part of a two-way diff, and the left side the older one.
> And it shows that diff correct under that assumption.
> 
> Stefan

Okay, this still leaves the issue that the remote side is
editable while the locale file is read-only. Furthermore, saving
changes to the modified remote file will prompt you to save the
file at the remote file's temporary location. I guess we can
agree that at least this behavior is wrong?

If so, what's your proposed fix to this? Keep the sides as-is but
toggle which of both sides is actually read-only and which is
not?

Regards
Patrick
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEtmscHsieVjl9VyNUEXxntp6r8SwFAljQ2J0ACgkQEXxntp6r
8SyuBA//dq/od1ditC/+H64cszqnyBmau39qAYJ+u7B0yoWXFBG9grwPLPF+aKzk
Z55GRCL4/dDxHU1+K3bXH0UHGGRDuUDKMP4/Fy6skOumqDp5DwGxIG7Pix4pa5yj
iwHvZ/fNe42Zw9VvXFRuxpl4rO7Q0wSlg4mS3plozDgbxxJVIUEVJi10q+nv5IWj
dA5w0KAObarEA9rsEyG8F2c5Hx+VoqPUxpu9XJbbjidbP0ZZv1LEM84T7TqY2No4
omJAmz40Jkn/5t7QU9EkcB5lwJdABJjlL6N9ysjbai+6TRFJUa41A4CnHhmJc02k
WZ4IfLpdGRh7ZERRvGJlkMS1HW5KaHWmBXw6GkmDzqU5wHdjtSQogISGenWoITD4
hc6pq1p16hRnUC2YkdYusCfMxuCi2/6VqE31uGiST7iMsF7zd2BfPZ6qwXdGtADm
5UHCm9fZHBgGYv/n9/uuczSUv4SkUzGTzmuSUaljoBD2mbpXd9v4q23W7BBu6SZg
SrIo79sifL0+CiGaN85tW6yO0v+p26Ni8t9UaaZ6lA5VaFoV9zfzqLrtLpwt5Fh2
Kglvma2gul0G8+PGkfN4EyI18n9tY6gWk/fvNWOasQYDDQqZLT3xqB4FgnWy4OVo
USQhfDQeudwbqqoQWsYcgb4V8HWYuEvcmVHkJllPY6r0ZOPQTKc=
=QVGJ
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.