Re: svn info --detail [was: Time to branch 1.9]

On 2/16/15 3:57 AM, Branko Čibej wrote:
> At first I thought I'd just lowercase the untranslated keys that 'svn
> info' displays, replacing spaces with dashes. But there's a strong case
> for using the same names as the --xml output; not because it's easier to
> implement, but because it's at least marginally consistent with
> something we already have.

The XML names are pretty bad in my opinion. These are the two blocks out of
the XML output that have the current revision and the last changed revision
respectively:
[[[
<entry
   kind="file"
   path="install.sh"
   revision="148">
<commit
   revision="35">
]]]

I think command line users are going to find the distinction between
"entry-revision" and "commit-revision" to be obscure.

Entry is leakage from our WC data model leaking into the XML output, where it
really doesn't belong.

The XML values taken literally would also be way more verbose than necessary.
"entry-path" versus just "path".

For that matter are you going to keep some of the parent blocks like "wc-info"
which would make things like "schedule" into "wc-info-schedule"

The repository root key would end up being "entry-repository-root" if you kept
the nesting as well.

So in my opinion just using the untranslated info keys with spaces turned into
"-" is far better.

You're almost certainly going to end up making arbitrary decisions anyway since
some of the names start getting silly.
Received on 2015-02-17 04:36:36 CET