You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Madan U S <ma...@collab.net> on 2006/02/03 18:16:58 UTC

[PATCH] Remove ambiguity in subversion/tests/cmdline/README file

[[[
Removed ambiguity in the subverion/tests/cmdline/README file.

* wc/workarea/subversion/tests/cmdline/README :
(How To Run The Tests):
(Running over ra_dav):
(Running over ra_svn): Explicitly mentioned the location to execute from.
]]]

Re: [PATCH] Remove ambiguity in subversion/tests/cmdline/README file

Posted by Daniel Rall <dl...@collab.net>.
On Fri, 03 Feb 2006, Garrett Rooney wrote:

> On 2/3/06, Madan U S <ma...@collab.net> wrote:
> >
> > [[[
> >  Removed ambiguity in the subverion/tests/cmdline/README file.
> >
> >  * wc/workarea/subversion/tests/cmdline/README :
> >  (How To Run The Tests):
> >  (Running over ra_dav):
> >  (Running over ra_svn): Explicitly mentioned the location to execute from.
> >  ]]]
> 
> Is it really necessary to be explicit about this?  If you're running
> ./basic_tests.py then by definition you must be in the
> subversion/tests/cmdline directory, and to run make you have to be in
> the base directory because that's where the makefile is...

As is actually a common pitfall, I have added a little more emphasis
(and example) about the need to be in the tests/cmdline/ directory.
-- 

Daniel Rall

RE: [PATCH] Remove ambiguity in subversion/tests/cmdline/README file

Posted by Madan U S <ma...@collab.net>.
> grooney wrote...
> > madan@collab.net <ma...@collab.net>  wrote...
> >
> > [[[
> >  Removed ambiguity in the subverion/tests/cmdline/README file.

[snip]

> Is it really necessary to be explicit about this?  If you're running
> ./basic_tests.py then by definition you must be in the
> subversion/tests/cmdline directory, and to run make you have to be in
> the base directory because that's where the makefile is...

  you are right about the makefile, and in fact about the './' usage, as the README

resides inside subversion/tests/cmdline. But what about the usages like

subversion/tests/cmdline/basic_tests.py 1 ? Not every use of such path mentions that the cmd has

to be executed from the top level dir of the source tree (tho it is obvious, it is ambiguous when seen in context with the './' using examples.)

Regards,
Madan.
PS : Apologize about the formatting.. hand to hand-format in my webmail client...



Re: [PATCH] Remove ambiguity in subversion/tests/cmdline/README file

Posted by Garrett Rooney <ro...@electricjellyfish.net>.
On 2/3/06, Madan U S <ma...@collab.net> wrote:
>
> [[[
>  Removed ambiguity in the subverion/tests/cmdline/README file.
>
>  * wc/workarea/subversion/tests/cmdline/README :
>  (How To Run The Tests):
>  (Running over ra_dav):
>  (Running over ra_svn): Explicitly mentioned the location to execute from.
>  ]]]

Is it really necessary to be explicit about this?  If you're running
./basic_tests.py then by definition you must be in the
subversion/tests/cmdline directory, and to run make you have to be in
the base directory because that's where the makefile is...

-garrett

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