From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cantor2.suse.de ([195.135.220.15]:55539 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966834AbaFTNXH (ORCPT ); Fri, 20 Jun 2014 09:23:07 -0400 Message-ID: <53A435B9.605@suse.cz> Date: Fri, 20 Jun 2014 15:23:05 +0200 From: Michal Marek MIME-Version: 1.0 Subject: Re: [PATCH] scripts/Makefile.clean: clean also $(extra-m) and $(extra-) References: <539963D4.3040409@suse.cz> <53A1EE1E.3090906@suse.cz> <20140620212212.9339.AA925319@jp.panasonic.com> In-Reply-To: <20140620212212.9339.AA925319@jp.panasonic.com> Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Masahiro Yamada Cc: linux-kbuild@vger.kernel.org Dne 20.6.2014 14:22, Masahiro Yamada napsal(a): > Hi Michal, > > > On Wed, 18 Jun 2014 21:53:02 +0200 > Michal Marek wrote: > >> Dne 12.6.2014 10:24, Michal Marek napsal(a): >>> Dne 6.6.2014 03:43, Masahiro Yamada napsal(a): >>>> Signed-off-by: Masahiro Yamada >>>> --- >>>> scripts/Makefile.clean | 4 ++-- >>>> 1 file changed, 2 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/scripts/Makefile.clean b/scripts/Makefile.clean >>>> index 686cb0d..a651cee 100644 >>>> --- a/scripts/Makefile.clean >>>> +++ b/scripts/Makefile.clean >>>> @@ -40,8 +40,8 @@ subdir-ymn := $(addprefix $(obj)/,$(subdir-ymn)) >>>> # build a list of files to remove, usually relative to the current >>>> # directory >>>> >>>> -__clean-files := $(extra-y) $(always) \ >>>> - $(targets) $(clean-files) \ >>>> +__clean-files := $(extra-y) $(extra-m) $(extra-) \ >>>> + $(always) $(targets) $(clean-files) \ >>> >>> Do you have an example where generated files are left in the tree after >>> make clean? This should go into the changelog. >> >> Hi Masahiro, >> >> do you have a reproducer, i.e. an example where make clean leaves >> generated files in the tree? > > No, I don't. > > I thought > arch/powerpc/kernel/Makefile is using > extra-$(CONFIG_...) += ... > but *.o files are simply cleaned-up. So this isn't the case. OK, so there is no urgent need to add this to 'rc-fixes'. I'll queue it up in 'kbuild' for 3.17-rc1. Thanks, Michal