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

RE: Re: svn -v list shows '.' in new version

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Wed, 01 Jul 2009 10:36:09 +0100

Andy Salnikov wrote:
> > Andy Salnikov wrote:
> > > We have upgraded from SVN 1.4 to 1.6 and I noticed one small
> > > inconsistency in the output produced by 'svn -v list' command.
> > > ...
> >
> > Andy,
> >
> > Thanks for the report. I think this is an unintentional change - in
> > other words, a regression.
> >
> > - Julian
>
> Hi Julian,
>
> thank you for reply. I presume it's going to be fixed then, or should
> I submit a bug report too?

Hi Andy.

I should have said more in my reply. No, one can't assume that a bug
that's reported will be fixed. Yes, everyone (users and developers)
would like all the bugs to be fixed but there are only a very few people
actually working on Subversion at any one time, and of course they're
already working on whatever they're working on.

Please do file an issue in the issue tracker. At least, that way, it
won't be forgotten.

If you are willing and able to get involved in fixing this, the first
thing that would really really boost the likelihood and speed of a fix
is if you can add a regression test for this behaviour.

I'm looking... The tests for "svn list" are in
subversion/tests/cmdline/basic_tests.py:

  basic_ls()
  ls_nonhead()
  ls_space_in_repo_name()

None of them lists a directory in verbose mode.

A good approach would be to add the new test as a regression test
specifically for this issue, so it need only test one instance of
listing a directory and not try to check any other aspect of the
behaviour.

If you copy and paste one of the existing test functions to the end of
the file, with a new name, and add its name to the list at the end of
the file, that's all. To run test number 19:

  (cd obj-dir/subversion/tests/cmdline/ && ./basic_tests.py 19)

or to find the number of your new test:

  (cd obj-dir/subversion/tests/cmdline/ && ./basic_tests.py list)

If you can't get involved, that's fine too, of course.

- Julian

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2366979
Received on 2009-07-01 11:36:29 CEST

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