You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Stefan Sperling <st...@elego.de> on 2010/08/04 10:40:22 UTC

Re: svn commit: r982173 - /subversion/trunk/subversion/svn/log-cmd.c

On Wed, Aug 04, 2010 at 09:21:16AM -0000, rhuijben@apache.org wrote:
> Author: rhuijben
> Date: Wed Aug  4 09:21:13 2010
> New Revision: 982173
> 
> URL: http://svn.apache.org/viewvc?rev=982173&view=rev
> Log:
> * subversion/svn/log-cmd.c
>   (svn_cl__log): Check for paths starting with a '/' instead of using
>     svn_dirent_is_absolute() as a '/a' style path is not absolute on
>     Windows and OS/2, and some valid relpaths theoretically might be.

Thanks for fixing this.

> 
> Modified:
>     subversion/trunk/subversion/svn/log-cmd.c
> 
> Modified: subversion/trunk/subversion/svn/log-cmd.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svn/log-cmd.c?rev=982173&r1=982172&r2=982173&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/svn/log-cmd.c (original)
> +++ subversion/trunk/subversion/svn/log-cmd.c Wed Aug  4 09:21:13 2010
> @@ -648,7 +648,7 @@ svn_cl__log(apr_getopt_t *os,
>          {
>            target = APR_ARRAY_IDX(targets, i, const char *);
>  
> -          if (svn_path_is_url(target) || svn_dirent_is_absolute(target))
> +          if (svn_path_is_url(target) || target[0] == '/')
>              return svn_error_return(svn_error_createf(
>                                        SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
>                                        _("Only relative paths can be specified"
>