From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.suse.de ([195.135.220.15]:40826 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752228AbbJNK4O (ORCPT ); Wed, 14 Oct 2015 06:56:14 -0400 Subject: Re: [PATCH 1/2] kconfig/merge_config.sh: Support KCONFIG_CONFIG References: <55F06D73.70303@gmail.com> From: Michal Marek Message-ID: <561E34CB.9050903@suse.cz> Date: Wed, 14 Oct 2015 12:56:11 +0200 MIME-Version: 1.0 In-Reply-To: <55F06D73.70303@gmail.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Gabriel de Perthuis Cc: linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org On 2015-09-09 19:33, Gabriel de Perthuis wrote: > All make targets support $KCONFIG_CONFIG because they > run scripts/kconf. Make sure merge_config.sh accesses the > correct file in all cases. > > Previously this script broke in two different code paths, > one for targets like kvmconfig (which use merge_config.sh -m > then call a target that respects KCONFIG_CONFIG) and one for > direct use of the script without -m, which called make rules > that edit KCONFIG_CONFIG but verified a different file. > > Signed-off-by: Gabriel de Perthuis > Cc: Michal Marek > --- > scripts/kconfig/merge_config.sh | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/scripts/kconfig/merge_config.sh > b/scripts/kconfig/merge_config.sh > index 0d883b3..5f1e110 100755 > --- a/scripts/kconfig/merge_config.sh > +++ b/scripts/kconfig/merge_config.sh > @@ -122,13 +122,13 @@ for MERGE_FILE in $MERGE_LIST ; do > done > cat $MERGE_FILE >> $TMP_FILE > done > if [ "$RUNMAKE" = "false" ]; then > - cp $TMP_FILE $OUTPUT/.config > + cp $TMP_FILE ${KCONFIG_CONFIG:=$OUTPUT/.config} > echo "#" > - echo "# merged configuration written to $OUTPUT/.config (needs make)" > + echo "# merged configuration written to $KCONFIG_CONFIG (needs make)" The message will not contain the name of the output file when invoked directly and without the variable set. Also, it will be more readable if you set the default value once and then use simply $KCONFIG_CONFIG. Michal