make: don't use .target in paths or printouts

make: don't use .target in paths or printouts

Instead, .target is considered the default, unnamed configuration,
while .host remains printed and pathed as before.

Committed: https://code.google.com/p/gyp/source/detail?r=1209

[Evan Martin, 2012-02-16 20:17:02]

The tests seem to pass.  And my build output is pretty:

$ make base_unittests
  CXX out/Debug/obj/base/base/debug/debugger_posix.o
  CXX out/Debug/obj/base/base/debug/profiler.o
  CXX out/Debug/obj/base/base/debug/stack_trace.o

Who knows what may eventually break, though.

[Nico, 2012-02-16 20:30:08]

LGTM as far as I can tell.

(I know of one script that will need to be updated once this rolls in. I'll do
that, and it doesn't have to happen right after the roll.)

This clashes more with the ninja output paths than the previous scheme, but
that's probably fine.

Curious: Is this on the path for a nicer cross-compile setup (don't build base
twice when not cross compiling; support cross-compiling in ninja), or just an
OCD fit?

https://chromiumcodereview.appspot.com/9418017/diff/8/pylib/gyp/generator/mak...
File pylib/gyp/generator/make.py (right):

https://chromiumcodereview.appspot.com/9418017/diff/8/pylib/gyp/generator/mak...
pylib/gyp/generator/make.py:1807: path = path.replace('$(obj)/',
'$(obj)%s/$(TARGET)/' % self._ToolsetSuffix())
80 COLS!!!!!!!!!!!!

https://chromiumcodereview.appspot.com/9418017/diff/8/pylib/gyp/generator/mak...
pylib/gyp/generator/make.py:1861: assert self.toolset  # 'target' maps to '', so
don't allow '' as input.
s/#/, "/ maybe

[Evan Martin, 2012-02-16 20:35:19]

On 2012/02/16 20:30:08, Nico wrote:
> Curious: Is this on the path for a nicer cross-compile setup (don't build base
> twice when not cross compiling; support cross-compiling in ninja), or just an
> OCD fit?

OCD.  This is the first time I've used make for Chrome in a long time, but other
projects like node will benefit too.

[piman, 2012-03-06 01:54:03]

Belated LGTM