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/18 05:52:24 UTC

[PATCH] Fix for issue 3620 - switch command.

Log

[[[

Make 'svn switch' verify that the target path is local.

* subversion/libsvn_client/switch.c,
  subversion/svn/switch-cmd.c
  (svn_client_switch2, svn_cl__switch): Raise an error if target path is
  not local.

* subversion/tests/cmdline/input_validation_tests.py
  (invalid_switch_targets, test_list): New test.

Patch by: Noorul Islam K M <noorul{_AT_}collab.net>

]]]

Thanks and Regards
Noorul


Re: [PATCH] Fix for issue 3620 - switch command.

Posted by Julian Foad <ju...@wandisco.com>.
Noorul Islam K M wrote:
> Make 'svn switch' verify that the target path is local.

Thanks.  Committed in r1036373.

> +  if (svn_path_is_url(path))
> +    return svn_error_return(svn_error_createf(SVN_ERR_ILLEGAL_TARGET, NULL,

I tweaked this line, removing svn_error_return() as it's redundant when
using svn_error_create*().

> +                                              _("'%s' is not a local path"),
> +                                              path));

- Julian