[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: svn_client_status5 and depth_as_sticky

From: Bert Huijben <bert_at_qqmail.nl>
Date: Fri, 8 Jul 2011 17:47:51 +0200

> -----Original Message-----
> From: C. Michael Pilato [mailto:cmpilato_at_collab.net]
> Sent: vrijdag 8 juli 2011 17:40
> To: Stefan Küng
> Cc: Subversion Development
> Subject: Re: svn_client_status5 and depth_as_sticky
>
> On 07/08/2011 10:57 AM, Stefan Küng wrote:
> > Hi,
> >
> > svn_client_status5 has a parameter depth_as_sticky. According to the doc
> > string:
> >
> > If depth_as_sticky is set and depth is not svn_depth_unknown, then the
> > status is calculated as if depth_is_sticky was passed to an equivalent
> > update command.
> >
> > this should report *everything* there is, even items that are excluded in
> > the working copy (svn_depth_exclude) if also 'update' is true and the
> passed
> > depth is svn_depth_infinity.
> >
> > This used to work a while ago, but now it does not return anything that
> > isn't in the working copy already anymore.
> >
> > I don't know when this stopped working since I don't use this feature very
> > often, but I'm now testing sparse working copies and so I've noticed that it
> > stopped working.
> >
> > May I suggest to also add this option to the CL client? currently 'svn st'
> > does not accept a 'set-depth' or equivalent parameter.
>
> --set-depth is a data-changing option, and 'svn st' doesn't change data.
> That option would be entirely inappropriate for use with the status
> operation. The log message of the commit which added this new API
> parameter
> (r986510) understood this:
>
> Following up on r957917, make it possible to switch of the ambient depth
> filter when using svn_client_status5, like it is possible with
> svn_client_update3().
>
> Currently we don't expose this information to the svn client, as using
> --set-depth on svn status would be illogical.
>
> I think the natural UI hook would be to bind this to the --depth option,
> allowing an explicitly-set --depth to be honored by the "remote" side of an
> 'svn status --show-updates', while svn_depth_unknown would continue to
> mean
> "only show me stuff that matches my working copy's ambient depth".

This is how it +- used to work.
And then 'svn st --depth infinity -u' has a different result than 'svn up --depth infinity'
(same applies for every other depth)

So that doesn't help making it work 'ok' in all situations :(

If there was a way to make it magically work ok, we probably would have gotten it right in 1.5.

(Stefan: I'll check if I can find where the regression occurred later this weekend)

        Bert
Received on 2011-07-08 17:48:30 CEST

This is an archived mail posted to the Subversion Dev mailing list.