Re: Clarification for paths argument to svn_ra_get_log2

Martin von Gagern wrote:
> Resending, as I forgot to cc the list... sorry.
>
> C. Michael Pilato wrote:
> | I seem to recall that recently I realized that NULL and empty had two
> | different interpretations in 'svn log'. I think empty resulted in (b),
> | and (a surprise to me) NULL in (c).
>
> If that is indeed the case and intended this way, it would certainly
> merit documentation.

Okay, looking in svn_repos_get_logs4(), the code does the following pretty
early:

   if (! paths)
     paths = apr_array_make(pool, 0, sizeof(const char *));

So it would seem that NULL and empty have the same meaning.

> |> Current assertions enforce a). Does this mean there is no way to
> |> access the full log of a repository while the session is for a
> |> subtree?
> |
> | That is correct.
>
> If what you just wrote is true, then passing NULL would be just such a
> way to access the full log. Or did I misunderstand something?

I think it turns out to be false, though.

> | If you need to do something at a different location, you
> | can use svn_ra_reparent() to change the current session URL.
>
> One of the bzr devs, Jelmer Vernooij, argued that changing URL is often
> not an option due to access restrictions.

As I think I said already (but maybe in private mail?), if changing the URL
to a shorter (higher-pathed) one is no good due to access restrictions,
wouldn't those same access restrictions get in the way of us reporting logs
for those higher-level directories anyway?

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