Bug/doc report

[Robert Herman <rl...@po.cwru.edu>]

Hello,

I'm a new user of Subversion who just installed the server.  I'm using 
version 0.14.3.

After installation, I created a new empty directory (/usr/local/svn/repos) 
to serve as a repository.  I then issued the command:

snvadmin create /usr/local/svn/repos/

This fails with a bizarre error message:
subversion/libsvn_subr/path.c:164: svn_path_join: Assertion 
'is_canonical_nts (base, blen)' failed

After asking for help on IRC (thank you pretzelgod!), I isolated the cause 
of the problem: the trailing slash is illegal, even though the directory 
already exists.  If this is not a bug, a line in the documentation would 
certainly save a lot of confusion.

In the IRC discussion, another issue arose: should Subversion create a 
repository in a preexisting, nonempty directory?  It seems that this is 
likely not what the user actually wants to do.  Perhaps it could ask 
confirmation?

Thank you for your help.
--|Robert


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Bug/doc report

[Karl Fogel <kf...@newton.ch.collab.net>]

Philip Martin <ph...@codematters.co.uk> writes:
> > Huh, frankly, it should just work with the trailing slash.
> 
> Fixed in rev 3156.

And, it works whether or not the empty directory exists previously.

Thanks, Philip!  No need for an issue when things get fixed that fast.

-K


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Bug/doc report

[Philip Martin <ph...@codematters.co.uk>]

Karl Fogel <kf...@newton.ch.collab.net> writes:

> > snvadmin create /usr/local/svn/repos/
> > 
> > This fails with a bizarre error message:

It's not bizarre, it explicitly identifies the error :)

> > subversion/libsvn_subr/path.c:164: svn_path_join: Assertion
> > 'is_canonical_nts (base, blen)' failed
> > 
> 
> Huh, frankly, it should just work with the trailing slash.

Fixed in rev 3156.

-- 
Philip Martin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: Bug/doc report

[Karl Fogel <kf...@newton.ch.collab.net>]

Robert Herman <rl...@po.cwru.edu> writes:
> I'm a new user of Subversion who just installed the server.  I'm using
> version 0.14.3.
> 
> After installation, I created a new empty directory
> (/usr/local/svn/repos) to serve as a repository.  I then issued the
> command:
> 
> snvadmin create /usr/local/svn/repos/
> 
> This fails with a bizarre error message:
> subversion/libsvn_subr/path.c:164: svn_path_join: Assertion
> 'is_canonical_nts (base, blen)' failed
> 
> After asking for help on IRC (thank you pretzelgod!), I isolated the
> cause of the problem: the trailing slash is illegal, even though the
> directory already exists.  If this is not a bug, a line in the
> documentation would certainly save a lot of confusion.

Huh, frankly, it should just work with the trailing slash.  Why make
users' lives difficult?  And the directory's existence shouldn't
matter, either -- even if it did not exist previously, the command
should still work with the trailing slash.

> In the IRC discussion, another issue arose: should Subversion create a
> repository in a preexisting, nonempty directory?  It seems that this
> is likely not what the user actually wants to do.  Perhaps it could
> ask confirmation?

I think it should just make the repository in the empty directory.
If the directory is not empty, then it should error of course.

Robert, could you file an issue on these?  Although they're
technically separate questions, it's fine to put them together in one
issue, as they're probably easy to fix at the same time.  Mark the
issue with the `bite-sized' keyword.

(If you don't already have `Observer' role on subversion.tigris.org,
you'll need to get it, but that's a pretty quick process.)

Thanks,
-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org