From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from pty.hi.pengutronix.de ([2001:67c:670:100:1d::c5] helo=[IPv6:::1]) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1jZYRl-0006X0-GZ for ptxdist@pengutronix.de; Fri, 15 May 2020 13:24:09 +0200 References: <20200514134300.16105-1-bst@pengutronix.de> <20200514134300.16105-13-bst@pengutronix.de> <476c88b3-fef5-52b5-40f7-86e3be08d588@pengutronix.de> <20200515075555.GD32420@pengutronix.de> From: Bastian Krause Message-ID: <41e398bb-3a8a-caea-d320-ef7ff0dd4fe2@pengutronix.de> Date: Fri, 15 May 2020 13:24:09 +0200 MIME-Version: 1.0 In-Reply-To: <20200515075555.GD32420@pengutronix.de> Content-Language: en-US Subject: Re: [ptxdist] [PATCH 13/13] code-signing: introduce for ptxconfig, add sanity check List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ptxdist-bounces@pengutronix.de Sender: "ptxdist" To: ptxdist@pengutronix.de On 5/15/20 9:55 AM, Michael Olbrich wrote: > On Thu, May 14, 2020 at 05:36:54PM +0200, Bastian Krause wrote: >> On 5/14/20 3:43 PM, Bastian Krause wrote: >>> Create a new rules/code-signing.in to be able to select CODE_SIGNING >>> not only in platformconfig but also in ptxconfig. >>> >>> Also make sure that PTXCONF_CODE_SIGNING_PROVIDER is set correctly if >>> PTXCONF_CODE_SIGNING is set. >>> >>> Signed-off-by: Bastian Krause >>> --- >>> rules/code-signing.make | 13 +++++++++++++ >>> 1 file changed, 13 insertions(+) >>> create mode 100644 rules/code-signing.make >>> >>> diff --git a/rules/code-signing.make b/rules/code-signing.make >>> new file mode 100644 >>> index 000000000..db1c5ab88 >>> --- /dev/null >>> +++ b/rules/code-signing.make >>> @@ -0,0 +1,13 @@ >>> +# -*-makefile-*- >>> +# >>> +# Copyright (C) 2020 by Bastian Krause >>> +# >>> +# For further information about the PTXdist project and license conditions >>> +# see the README file. >>> +# >>> + >>> +ifdef PTXCONF_CODE_SIGNING >>> +ifeq ($(call remove_quotes,$(PTXCONF_CODE_SIGNING_PROVIDER)),) >>> +$(error PTXCONF_CODE_SIGNING_PROVIDER must be set correctly) >>> +endif >>> +endif >>> >> >> I accidentally dropped: >> >> diff --git a/rules/code-signing.in b/rules/code-signing.in >> new file mode 100644 >> --- /dev/null >> +++ b/rules/code-signing.in >> @@ -0,0 +1,4 @@ >> +## SECTION=hosttools_noprompt_ptx >> + >> +config CODE_SIGNING >> + bool >> >> Will add it again in v2. > > And sort the patch before the rauc changes. It's needed there. True. Regards, Bastian -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de