Re: svn commit: r15002 - in trunk/subversion: clients/cmdline include libsvn_client tests/clients/cmdline

Julian Foad <julianfoad@btopenworld.com> writes:
> kfogel@collab.net wrote:
> > julianfoad@tigris.org writes:
> >>+ * @deprecated Provided for backward compatibility with the 1.2 API.
> >> */
> >> svn_error_t *svn_client_import (svn_client_commit_info_t **commit_info,
> >> const char *path,
> > Should we be saying "1.2 API" there, or "1.0 API"?
>
> I think I sometimes had a clear notion of which was more appropriate,
> but right now I've no idea. Anyone?

I guess I think earliest applicable interface is best.

> That puzzled me when I reviewed the patch, but I investigated and
> found that ignores are only applied to items found by recursion, not
> to explicitly given targets. A directory made by "mkdir" is always an
> explicit target, so ignores don't apply.

Ah, thanks, that makes sense.

> I mentioned that that behaviour (only applicable in recursion) is not
> documented, and I'm not sure it's what I really want (but it's a hard
> call), but I don't suppose that comment is enough to make anyone
> document it ... :-)
> Any volunteers?

I think I no longer have enough context to understand this paragraph,
but I'd be happy to volunteer to write a doc string if you can give me
a general pointer to what you had in mind (if that's what you meant).

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jun 13 19:13:41 2005