On Wed, Jun 15, 2011 at 06:55:19AM +0300, Daniel Shahaf wrote:
> Daniel Shahaf wrote on Wed, Jun 15, 2011 at 06:33:46 +0300:
> > stsp_at_apache.org wrote on Tue, Jun 14, 2011 at 15:10:50 -0000:
> > > + svn_stringbuf_appendcstr(buf, _("Local property value:\n"));
> > > + if (mine_is_binary)
> > > + svn_stringbuf_appendcstr(buf, _("Cannot display: property value is "
> > > + "binary data\n"));
> >
> > Could we just print a hex dump of the value?
>
> Concretely:
>
> [[[
> but the property has already been locally changed to a different value.
> Local property value:
> mine
> Incoming property value:
> 0x7E454B4602010100000000000000000002003D0001000000A01440000000000040000000000000
> ]]]
While I understand your reasoning I think this would be very
confusing to many users. Not all SVN users are programmers.
And I guess people will in general be using special tools anyway to view
such properties. E.g. if the prop value was a PNG image, they will want
to see the image. A string of hex numbers won't help them much.
There is precedence here, too. We don't show diffs of binary files,
we show a placeholder instead that says a diff cannot be shown.
Would you also prefer svn diff to print a diff of the hex dumps?
I'd doubt it.
I'd say let them propget the value and look at it in whatever manner
they see fit. This is a total edge case anyway. No user using standard
svn: properties is ever going to run into this.
> Patch to produce that:
Received on 2011-06-15 12:13:05 CEST