Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(221)

Issue 7606025: Support OUTDIR variable for tests triggered by Makefile (Closed)

Created:
9 years, 4 months ago by Jakob Kummerow
Modified:
9 years, 4 months ago
Reviewers:
William Hesse
CC:
v8-dev
Visibility:
Public.

Description

Support OUTDIR variable for tests triggered by Makefile TEST="make -j16 check OUTDIR=whatever" works as expected Committed: http://code.google.com/p/v8/source/detail?r=8871

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -12 lines) Patch
M Makefile View 1 chunk +5 lines, -8 lines 0 comments Download
M tools/test.py View 1 chunk +3 lines, -3 lines 0 comments Download
M tools/test-wrapper-gypbuild.py View 2 chunks +4 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Jakob Kummerow
PTAL. Or just try it locally ;-)
9 years, 4 months ago (2011-08-10 09:44:56 UTC) #1
William Hesse
LGTM. The linter does not like the tabs in the makefile - are those necessary? ...
9 years, 4 months ago (2011-08-10 10:23:20 UTC) #2
Jakob Kummerow
On 2011/08/10 10:23:20, William Hesse wrote: > LGTM. The linter does not like the tabs ...
9 years, 4 months ago (2011-08-10 10:46:26 UTC) #3
Jakob Kummerow
9 years, 4 months ago (2011-08-10 10:50:21 UTC) #4
On 2011/08/10 10:46:26, jkummerow wrote:
> On 2011/08/10 10:23:20, William Hesse wrote:
> > LGTM.  The linter does not like the tabs in the makefile - are those
> necessary?
> 
> http://www.gnu.org/software/make/manual/make.html#Rule-Introduction says:
> "Please note: you need to put a tab character at the beginning of every recipe
> line! This is an obscurity that catches the unwary. If you prefer to prefix
your
> recipes with a character other than tab, you can set the .RECIPEPREFIX
variable
> to an alternate character (see Special Variables)."
> 
> I'd prefer to not deviate from the general rule, but I don't feel strongly
about
> it.

Update: .RECIPEPREFIX is only supported since make 3.82, but Ubuntu Lucid
contains 3.81. So we don't have a choice for the time being.

Powered by Google App Engine
This is Rietveld 408576698