[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1135635 - /subversion/trunk/subversion/libsvn_wc/props.c

From: Stefan Sperling <stsp_at_elego.de>
Date: Wed, 15 Jun 2011 12:12:27 +0200

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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.