You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Noorul Islam K M <no...@collab.net> on 2010/11/10 03:30:51 UTC

Re: svn commit: r1033320 - in /subversion/trunk/subversion: include/svn_error_codes.h svn/cl.h svn/export-cmd.c svn/notify.c svn/switch-cmd.c svn/update-cmd.c tests/cmdline/externals_tests.py

cmpilato@apache.org writes:

> Author: cmpilato
> Date: Wed Nov 10 01:44:35 2010
> New Revision: 1033320
>
> URL: http://svn.apache.org/viewvc?rev=1033320&view=rev
> Log:
> Fix issue #3622 ("svn should exit with exit code 1 if updating
> externals fails").
>
> * subversion/include/svn_error_codes.h
>   (SVN_ERR_CL_ERROR_PROCESSING_EXTERNALS): New error code.
>  
> * subversion/svn/cl.h
>   (struct svn_cl__check_externals_failed_notify_baton): New baton.
>   (svn_cl__check_externals_failed_notify_wrapper): New function.
>
> * subversion/svn/notify.c
>   (svn_cl__check_externals_failed_notify_wrapper): New function.
>
> * subversion/svn/update-cmd.c
>   (svn_cl__update): Use the new wrapper baton and function to track
>     externals processing failures, and return an error after all else
>     is finished if such failures occured.
>
> * subversion/svn/export-cmd.c
>   (svn_cl__export): Use the new wrapper baton and function to track
>     externals processing failures, and return an error after all else
>     is finished if such failures occured.
>
> * subversion/svn/switch-cmd.c
>   (svn_cl__switch): Use the new wrapper baton and function to track
>     externals processing failures, and return an error after all else
>     is finished if such failures occured.
>
> * subversion/tests/cmdline/externals_tests.py
>   (old_style_externals_ignore_peg_reg,
>    can_place_file_external_into_dir_external): Change expected exit code.
>
> Modified:
>     subversion/trunk/subversion/include/svn_error_codes.h
>     subversion/trunk/subversion/svn/cl.h
>     subversion/trunk/subversion/svn/export-cmd.c
>     subversion/trunk/subversion/svn/notify.c
>     subversion/trunk/subversion/svn/switch-cmd.c
>     subversion/trunk/subversion/svn/update-cmd.c
>     subversion/trunk/subversion/tests/cmdline/externals_tests.py
>

I think I can have similar approach for issue 3713. If you are not
already on it then I will try implementing the same for 3713.

Thanks and Regards
Noorul

Re: svn commit: r1033320 - in /subversion/trunk/subversion: include/svn_error_codes.h svn/cl.h svn/export-cmd.c svn/notify.c svn/switch-cmd.c svn/update-cmd.c tests/cmdline/externals_tests.py

Posted by "C. Michael Pilato" <cm...@collab.net>.
On 11/09/2010 10:30 PM, Noorul Islam K M wrote:
> I think I can have similar approach for issue 3713. If you are not
> already on it then I will try implementing the same for 3713.

Nope, I'm not "on it".

-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand