lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: describing regression checking more explicitely (issue4621041)


From: percival . music . ca
Subject: Re: describing regression checking more explicitely (issue4621041)
Date: Tue, 14 Jun 2011 22:01:49 +0000


http://codereview.appspot.com/4621041/diff/1/Documentation/contributor/regressions.itexi
File Documentation/contributor/regressions.itexi (right):

http://codereview.appspot.com/4621041/diff/1/Documentation/contributor/regressions.itexi#newcode191
Documentation/contributor/regressions.itexi:191: Ensure that currently
built binary is the one without your changes.
On 2011/06/14 21:40:37, Janek Warchol wrote:
I think a beginner can overlook this step.

Let's change that to:
Run @code{make} with current git master without any of your changes.

http://codereview.appspot.com/4621041/diff/1/Documentation/contributor/regressions.itexi#newcode201
Documentation/contributor/regressions.itexi:201: in
@file{lilypond-git/build/} directory.
On 2011/06/14 21:40:37, Janek Warchol wrote:
i remember that it wasn't obvious for me whether to run
make test-baseline
in lilypond-git/ or in lilypond-git/build/.

ok.  Could we make this:

Before making changes to the code, establish a baseline for the
comparison by going to the @file{lilypond-git/build/} directory and
running:

?  I prefer not to have text underneath an example.

http://codereview.appspot.com/4621041/diff/1/Documentation/contributor/regressions.itexi#newcode220
Documentation/contributor/regressions.itexi:220:
build/out/test-results/index.html
hmm.  Relative to the current directory, it's only
  out/test-results/index.html

since the build directory itself has a build/ subdir (Colin calls this
"elephants all the way down"), I think this could be confusing.

What about specifying
  lilypond-git/build/...

?

http://codereview.appspot.com/4621041/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]