Julian Foad wrote on Mon, 17 Sep 2018 20:39 +0100:
> Daniel Shahaf wrote:
> > From subversion/svn/info-cmd.c:
> > 272 if (TRUE)
> > 274 /* svn-viewspec.py format */
> > 281 else
> > 283 /* svn command-line format */
> >
> > Why is there an «if (TRUE)» there? Should that condition be exposed a
> > command-line option, or removed?
>
> For experimenting with viewspec output, I feel both formats are about
> equally useful at this stage, neither is ideal, and neither is likely
> what an eventual stable format should be. That's based on knowing that
> cmdline is practical for testing and assuming that there are some users
> using viewspec.py (e.g. Paul Hammant recently contributed a regression
> test suite for it).
>
> I was thinking to add a cmdline option to select, for the time being.
> IIRC Brane said "pick one and be done with it". That's as far as we got.
> I still favour adding a user-selectable option, e.g. "--x-viewspec={cmdline|svn-viewspec.py}" (ugly, I know).
I agree. The API is experimental so exposing a second output mode
doesn't restrict our options going forward. r1841272.
Cheers,
Daniel
Received on 2018-09-19 00:53:13 CEST