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

Re: [PATCH] Fix issue 2030 for "svn status"

From: Ed Price <ed.price_at_gmail.com>
Date: 2006-03-19 16:57:29 CET

Any thoughts on this patch?

I'm attaching a new version of the patch. The only difference
is a very minor improvement in the testcase. (Runs status
on two dirs instead of one, to be more sure that it's not
erroring out.) Same log message as before.

Thanks,
-Ed

On 3/10/06, Ed Price <ed.price@gmail.com> wrote:
> For both convenience and consistency, it would be nice if "svn status"
> skipped non-versioned items, like (eg) "svn update" does:
>
> $ svnadmin create repos
> $ svn co file://`pwd`/repos wc
> Checked out revision 0.
> $ mkdir notwc
> $ svn up *
> Skipped 'notwc'
> Skipped 'repos'
> At revision 0.
> $ svn st *
> subversion/libsvn_wc/lock.c:460: (apr_err=155007)
> svn: 'notwc' is not a working copy
> subversion/libsvn_subr/io.c:2386: (apr_err=2)
> svn: Can't open file 'notwc/.svn/format': No such file or directory
> $
>
> It seems this is issue 2030 ("Errorful targets should be skipped, not
> end the entire operation"):
>
> http://subversion.tigris.org/issues/show_bug.cgi?id=2030
>
> I've attached a patch which fixes this issue for "svn status".
>
> It includes a testcase, and modifies the error message (which now
> includes "warning: " since it is now a warning instead of an error)
> for another testcase.
>
> There is one difference in pool usage (XML used to be in the main pool
> instead of the subpool) but I believe the change is actually a slight
> improvement (the XML could/should have used the subpool before).
>
> Thanks,
> -Ed
>
> [[[
> Fix issue #2030 (Errorful targets should be skipped, not end the
> entire operation) for "svn status".
>
> * subversion/svn/status-cmd.c
> (do_status): New helper function.
> (svn_cl__status): Wrap do_status with svn_cl__try, allowing
> SVN_ERR_WC_NOT_DIRECTORY (not a working copy).
>
> * subversion/tests/cmdline/stat_tests.py
> (status_on_unversioned_dotdot): Adjust expected message; status on
> non-WC is now a warning instead of an error.
> (status_unversioned_dir): New test.
> (test_list): Add new test.
> ]]]

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Received on Sun Mar 19 17:00:49 2006

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