On Wed, Dec 12, 2018 at 10:23 PM Daniel Shahaf <d...@daniel.shahaf.name> wrote:
> Stefan Kueng wrote on Wed, 12 Dec 2018 21:16 +0100:
> > And as I repeatedly said: TSVN does validate the input as good as it
> > can. But if svn does neither describe the *exact* specs in the docs nor
> > provide any APIs that do that, then TSVN has to guess.
> > And no: specifying that paths/uris have to be "canonicalized" is not
> > enough because I do that, using the svn APIs.
> > So apparently that's not enough.
>
> Stefan, there is nothing we can do with this information.  If you have
> found a value for which svn_foo_is_canonical(svn_foo_canonicalize(value))
> returns false, please tell us what that value is.

Just thinking out loud here, but could we perhaps log the
non-canonical value as part of the assertion output?
Instead of:

  line 10238: assertion failed (svn_dirent_is_absolute(local_abspath))

something like

 line 10238: assertion failed
(svn_dirent_is_absolute(local_abspath='C:ImNotCanonical'))

That way, we would at least have some concrete information ...

-- 
Johan

Reply via email to