List Info

Thread: New - do_update()'s update_




New - do_update()'s update_
user name
2006-07-29 16:13:40
http://subversion.tigris.org/issues/show_bug.cgi?id=2588

                 Issue #|2588
                 Summary|do_update()'s update_target
argument sometimes allows 
                        |multi-component paths
               Component|subversion
                 Version|trunk
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|libsvn_ra
             Assigned to|issuessubversion
             Reported by|jrvernooij






------- Additional comments from jrvernooijtigris.org Sat Jul 29 09:13:39 -0700 2006 -------
According to the apidoc, the `update_target' argument to
svn_ra_do_update() can
only contain a single path component. However, the backends
all support multi
path components to some degree:

 * ra_dav truncates the path and uses the last component
(so, branches/1.4.x
becomes 1.4.x)
 * ra_svn and ra_local support multi component paths

It would be nice to have consistency across backends here,
so I would propose
(after a discussion with maxb on IRC) that a check be added
to libsvn_ra that
checks whether update_target is a single-component path and
returns an
appropriate error if it isn't. 

This will however, break existing 3rd-party apps that rely
on the existing
behaviour. Is that a problem?

------------------------------------------------------------
---------
To unsubscribe, e-mail: issues-unsubscribesubversion.tigris.org
For additional commands, e-mail: issues-helpsubversion.tigris.org

[1]

about | contact  Other archives ( Real Estate discussion Medical topics )