From patchwork Mon Dec 11 16:53:33 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gylstorff Quirin X-Patchwork-Id: 13487685 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id F042BC4167B for ; Mon, 11 Dec 2023 16:53:55 +0000 (UTC) Received: from mta-64-227.siemens.flowmailer.net (mta-64-227.siemens.flowmailer.net [185.136.64.227]) by mx.groups.io with SMTP id smtpd.web10.84.1702313629243542749 for ; Mon, 11 Dec 2023 08:53:50 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=Quirin.Gylstorff@siemens.com header.s=fm1 header.b=Mrt15/qZ; spf=pass (domain: rts-flowmailer.siemens.com, ip: 185.136.64.227, mailfrom: fm-51332-202312111653465f2c5a6d204f6170a8-au7nwr@rts-flowmailer.siemens.com) Received: by mta-64-227.siemens.flowmailer.net with ESMTPSA id 202312111653465f2c5a6d204f6170a8 for ; Mon, 11 Dec 2023 17:53:47 +0100 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; s=fm1; d=siemens.com; i=Quirin.Gylstorff@siemens.com; h=Date:From:Subject:To:Message-ID:MIME-Version:Content-Type:Content-Transfer-Encoding; bh=BBCYotTCIhobkMG1gfD0uFHP6LXRAGlfsHrco2/hX8U=; b=Mrt15/qZPXAXjXvnz2zfQ+auVTmKVP9daXoC5bnCI1JHjcsghZQIEQ6RuAnBHn3LjmRNa3 fmzDWgxXAS10R84PFEXOQIiFb/JOBBhSdL266K8bQJlT6C+T5gzx/BXGZ3lzoadmJFqVH37Q EwVQ8bcmelAs/tCqv2xFxzmCHOsRQ=; From: Quirin Gylstorff To: cip-dev@lists.cip-project.org, jan.kiszka@siemens.com Subject: [cip-dev][isar-cip-core][PATCH] swupdate-certificates: Fix shell error Date: Mon, 11 Dec 2023 17:53:33 +0100 Message-ID: <20231211165345.821930-1-Quirin.Gylstorff@siemens.com> MIME-Version: 1.0 X-Flowmailer-Platform: Siemens Feedback-ID: 519:519-51332:519-21489:flowmailer List-Id: X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Mon, 11 Dec 2023 16:53:55 -0000 X-Groupsio-URL: https://lists.cip-project.org/g/cip-dev/message/14034 From: Quirin Gylstorff An Additional bracket was added to all if cases, which leads to build errors. Signed-off-by: Quirin Gylstorff --- @Jan This should be added as soon as possible to master .../swupdate-certificates/swupdate-certificates-key.inc | 4 ++-- .../swupdate-certificates/swupdate-certificates.inc | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/recipes-devtools/swupdate-certificates/swupdate-certificates-key.inc b/recipes-devtools/swupdate-certificates/swupdate-certificates-key.inc index 0b3e045..56d006a 100644 --- a/recipes-devtools/swupdate-certificates/swupdate-certificates-key.inc +++ b/recipes-devtools/swupdate-certificates/swupdate-certificates-key.inc @@ -21,11 +21,11 @@ SRC_URI:append = " ${@ "file://"+d.getVar('SWU_SIGN_SCRIPT') if d.getVar('SWU_SI do_install[cleandirs] = "${D}/usr/share/swupdate-signing/ \ ${D}/usr/bin/" do_install() { - if [ ! -f ${WORKDIR}/${SWU_SIGN_SCRIPT} ] ]; then + if [ ! -f "${WORKDIR}/${SWU_SIGN_SCRIPT}" ]; then bbfatal "You must add a '${SWU_SIGN_SCRIPT}' to execute the signing process" fi install -m 0700 ${WORKDIR}/${SWU_SIGN_SCRIPT} ${D}/usr/bin/sign-swu - if [ -z ${SWU_SIGN_KEY} ] ]; then + if [ -z "${SWU_SIGN_KEY}" ]; then bbfatal "You must set SWU_SIGN_KEY and provide the required file as artifacts to this recipe" fi TARGET=${D}/usr/share/swupdate-signing/ diff --git a/recipes-devtools/swupdate-certificates/swupdate-certificates.inc b/recipes-devtools/swupdate-certificates/swupdate-certificates.inc index 0553c25..10b6833 100644 --- a/recipes-devtools/swupdate-certificates/swupdate-certificates.inc +++ b/recipes-devtools/swupdate-certificates/swupdate-certificates.inc @@ -18,7 +18,7 @@ SWU_SIGN_CERT ??= "" SRC_URI:append = " ${@ "file://"+d.getVar('SWU_SIGN_CERT') if d.getVar('SWU_SIGN_CERT') else '' }" do_install() { - if [ -z ${SWU_SIGN_CERT} ] ]; then + if [ -z "${SWU_SIGN_CERT}" ]; then bbfatal "You must set SWU_SIGN_CERT and provide the required file as artifacts to this recipe" fi TARGET=${D}/usr/share/swupdate-signing/