Re: [PATCH] tools: fix cross-compile var export

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

 



[Re: [PATCH] tools: fix cross-compile var export] On 07/01/2018 (Sun 10:31) Martin Kelly wrote:

[...]

> With the change, we add do CC = $(CROSS_COMPILE)gcc if and only if CC is not
> already set. I'm happy to add all these details to the commit description.

That is probably a step in the right direction.  I contribute to yocto
on a regular basis and hence am sympathetic to these frustrating SDK
type issues.  But after a quick scan of this patch it wasn't obvious to
me that existing behaviour was preserved, or that it would be a pain to
separate it into chunks. (key word here being "quick")

So I'd recommend updating the commit log, and adding a Cc: line for the
maintainers of each subsystem below, and then if nobody complains you
might get akpm to pick it up as he does for other patches w/o a clear
maintainer or subsystem.

P.
--

> 
> >>
> >>  tools/cgroup/Makefile            |  1 -
> >>  tools/gpio/Makefile              |  2 --
> >>  tools/hv/Makefile                |  1 -
> >>  tools/iio/Makefile               |  2 --
> >>  tools/laptop/freefall/Makefile   |  1 -
> >>  tools/leds/Makefile              |  1 -
> >>  tools/perf/Makefile.perf         |  6 ------
> >>  tools/power/acpi/Makefile.config |  3 ---
> >>  tools/scripts/Makefile.include   | 18 ++++++++++++++++++
> >>  tools/spi/Makefile               |  2 --
> >>  tools/usb/Makefile               |  1 -
> >>  tools/vm/Makefile                |  1 -
> >>  tools/wmi/Makefile               |  1 -
> >>  13 files changed, 18 insertions(+), 22 deletions(-)
> >>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux