[PATCH] make add skip unversioned res..V2

[vi...@collab.net]

[[[
 Use svn_cl__try for skipping non-fatal errors.

* subversion/clients/cmdline/add-cmd.c
  (svn_cl__add): Removed the variable err, use svn_cl__try.
]]]

Re: [PATCH] make add skip unversioned res..V2

[Julian Foad <ju...@btopenworld.com>]

vivek@collab.net wrote:
> [[[
>  Use svn_cl__try for skipping non-fatal errors.
> 
> * subversion/clients/cmdline/add-cmd.c
>   (svn_cl__add): Removed the variable err, use svn_cl__try.
> ]]]

The most important thing to document in the log message is the functional 
change: treating nonexistent paths as warnings rather than errors.  Then the 
use of svn_cl__try, which is perhaps a noteworthy implementation detail.  The 
removal of any local variables that are unused because of the change is 
implicit, so not worth mentioning.

Also, this should say that it is related to issue #2030: "Errorful targets 
should be skipped, not end the entire operation", as that explains why the 
change is wanted.

So:

[[[
Make "svn add" skip nonexistent items with a warning.  Use svn_cl__try to
simplify the handling of warnings.  This is part of issue #2030: "Errorful
targets should be skipped, not end the entire operation".

* subversion/clients/cmdline/add-cmd.c
   (svn_cl__add): Warn if path not found; use svn_cl__try.
]]]

As far as I can see, the patch is fine (though the patch file unexpectedly ends 
in the middle of a line).  I've tested it by hand.

Committed with this log message in r15043.

Thanks for the patch.

- Julian


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