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

Re: [PATCH] Fix a deprecation warning.

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Thu, 12 Nov 2009 10:00:24 +0000

On Wed, 2009-11-11, Kannan wrote:
> Stefan Sperling wrote:
> > Thanks. There are still more test failures caused by this patch.
> > Please run all tests and make sure they pass. You can run 'make check'
> > in the working copy root to run all tests, and it will tell you which
> > tests are failing.
>
> Should've run `make check` rather than testing info tests alone, sorry
> for that. Two tests(basic_tests.py 26 and changelist_tests.py 4) failed
> expecting relative paths for info, which is fixed with this patch to
> expect absolute ones. Make check passes.
>
> [[[
> Updated Log:
> Resolve a deprecation warning.

Hold on a minute... This patch doesn't just resolve a deprecation
warning, it changes the command-line output format. You need to say
things like that in the log message.

We try not to change the output format without very good reason. Why
can't we keep the output format the same?

- Julian

> * subversion/svn/info-cmd.c
> (svn_cl__info): Make the path as absolute and pass to the
> new `svn_client_info3()' and update the receiver type to
> `svn_info_receiver2_t'.
>
> * subversion/libsvn_client/info.c
> (svn_client_info3, push_dir_info,
> crawl_entries, found_entry_baton): Update the receiver type to
> `svn_info_receiver2_t'.
>
> [in subversion/tests/cmdline]:
>
> * info-tests.py
> (info_on_added_file, info_on_mkdir): Expect absolute paths for `svn
> info'.
>
> * basic_tests.py
> (basic_info): Same.
>
> * changelist_tests.py
> (info_with_changelists): Same.
>
> ]]]

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2417506
Received on 2009-11-13 12:08:14 CET

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.