On Tue, Jun 18, 2013 at 09:34:00PM +0400, Michael Tokarev wrote:
> The foo.cflags isn't really necessary, but when you specify one
> thing one way (target-specific variable), and another thing completely
> different way, resulting code does not look nice.  In particular, the
> two curl definitions in block/Makefile.objs look somewhat funky if
> curl.cflags isn't used.

Specifying per-object file or per-module LIBS/CFLAGS is great.  I wonder
if we can do it like kbuild though:
CFLAGS_i915_trace_points.o := -I$(src)

It only takes the object file name, not the path.  Could be a problem if
we have files with the same name in different directories I guess.

But I wonder if this approach would let you drop the $(obj)filename.o
change.  I didn't review closely enough to really understand how that
works though :-).

Stefan

Reply via email to