From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751329AbeEVEuS (ORCPT ); Tue, 22 May 2018 00:50:18 -0400 Received: from mail-ot0-f195.google.com ([74.125.82.195]:36349 "EHLO mail-ot0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750979AbeEVEuP (ORCPT ); Tue, 22 May 2018 00:50:15 -0400 X-Google-Smtp-Source: AB8JxZrNSUm744tle0gJc6LnGNw4FBFmARvQWJfw7XDyz2iowSR66+h9fSLo8nu3DQVVm8rdSM53f52v/ZWEm4c7R1o= MIME-Version: 1.0 In-Reply-To: References: <1526537830-22606-1-git-send-email-yamada.masahiro@socionext.com> <1526537830-22606-8-git-send-email-yamada.masahiro@socionext.com> <20180520145031.GB9826@ravnborg.org> From: Ulf Magnusson Date: Tue, 22 May 2018 06:50:14 +0200 Message-ID: Subject: Re: [PATCH v4 07/31] kconfig: add built-in function support To: Masahiro Yamada Cc: Sam Ravnborg , Linux Kbuild mailing list , Linus Torvalds , "Luis R . Rodriguez" , Linux Kernel Mailing List , Nicholas Piggin , Kees Cook , Emese Revfy , X86 ML Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 22, 2018 at 5:11 AM, Masahiro Yamada wrote: > 2018-05-22 0:10 GMT+09:00 Ulf Magnusson : >> On Mon, May 21, 2018 at 4:32 PM, Ulf Magnusson wrote: >>> On Mon, May 21, 2018 at 4:23 PM, Ulf Magnusson wrote: >>>> On Sun, May 20, 2018 at 4:50 PM, Sam Ravnborg wrote: >>>>> Will the following be equal: >>>>> >>>>> $(foo,abc,$(x),$(y)) >>>>> $(foo, abc, $(x), $(y)) >>>>> >>>>> make is rather annoying as space is significant, but there seems no good reason >>>>> for kconfig to inheritate this. >>>>> So unless there are good arguments consider alloing the spaces. >>>>> If the current implmentation already supports optional spaces then I just missed >>>>> it whie reviewing. >>>>> >>>>> Sam >>>> >>>> +1 from me. >>>> >>>> I also find the rules for whitespace in Make confusing, and always >>>> have to look them up when doing trickier stuff. Maybe they're the >>>> result of people not considering whitespace initially, and stuff >>>> getting tacked on later. GNU Make adds some alternate syntaxes with >>>> quotes. >>>> >>>> I was going to mention shell, but it looks like you already did. :) >>>> >>>> If we go with Make-like syntax, maybe we could at least have a variant >>>> with fewer whitespace gotchas. >>>> >>>> Cheers, >>>> Ulf >>> >>> Maybe it'd be a pain to implement, but something like $(foo $(x) "two >>> words" "interpolated $(stuff)") seems pretty nice, with three >>> arguments there. >> >> Guess that might interact poorly with $(shell foo "bar baz") though. >> Kinda nice to have a syntax that doesn't overlap with shell when >> building shell commands. > > > Right. I can easily imagine > that would end up with more gotchas due to quoting and escaping. Yeah, you're right. It's probably trying to fix something that isn't broken. Make's syntax really isn't bad there, just slightly non-obvious at first... Think it's fine now. Better to commit to the syntax than trying to be "helpful" by adding a bunch of random exceptions too. That probably gives a bigger mess in the end... Cheers, Ulf