From patchwork Thu Sep 6 14:53:34 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Emil Velikov X-Patchwork-Id: 10590695 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id AF82C5A4 for ; Thu, 6 Sep 2018 14:58:17 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 940DB2AC01 for ; Thu, 6 Sep 2018 14:58:17 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 87C5B2AB8E; Thu, 6 Sep 2018 14:58:17 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, FREEMAIL_FROM,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 282EE2AB7C for ; Thu, 6 Sep 2018 14:58:17 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 43D266E6CA; Thu, 6 Sep 2018 14:58:15 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-wm0-x230.google.com (mail-wm0-x230.google.com [IPv6:2a00:1450:400c:c09::230]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5FDF76E6CE for ; Thu, 6 Sep 2018 14:58:14 +0000 (UTC) Received: by mail-wm0-x230.google.com with SMTP id s12-v6so11759901wmc.0 for ; Thu, 06 Sep 2018 07:58:14 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=YPdAUPbyLs4pp9CecGswWqCJcWT2SlfJ+c7tdtnNT+g=; b=Xh8B3+qQGPRrY2TBGvmv4dGvDV5HuicQC8HvhF5SSkL3R1v5b2X+XZndeCjWU14jwp scRqBrg+Iw/VudQbco2SBCc6kTat5FHHw2XednQDoeL1g8NkYJTnWI3K6DERpImvs+Wu 9lYKvsi5VDL/rRUy1cbPEwdQ4IXrmdWYb+uRnnmpvSkZUKLRjz1R9Aps8x0kR/R2ICX8 zib1m9RGGV02QK5j+I44OXA1QY/xRQPHCRGfQ9G7Pp8r+HXytMdFhlkQnQDUpXMkVQZb RZwa6+GlnB16UvZ4WFnbSJx9BH3vj8TeEk5E0JJnyDIH8Joz082Jl+Mr9MLfoHERuOH6 yiDQ== X-Gm-Message-State: APzg51C9f2OmKRxiHYcQEBI96LbjwDn86mDbtpZT/gSq7Hz5P/KZRWoc KuzAhpP5hjcNFokj01Ulbl5FufKq X-Google-Smtp-Source: ANB0VdavltjVa9WJlUQQBuQ5WfatggXRJQ0ixO0ElBL5qOC+bE4Sm3vqIyXWaBp3ZCEEOPudGt0zvA== X-Received: by 2002:a1c:1510:: with SMTP id 16-v6mr2303056wmv.74.1536245891918; Thu, 06 Sep 2018 07:58:11 -0700 (PDT) Received: from arch-x1c3.cbg.collabora.co.uk ([2a00:5f00:102:0:9665:9cff:feee:aa4d]) by smtp.gmail.com with ESMTPSA id v46-v6sm5502668wrc.63.2018.09.06.07.58.10 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 06 Sep 2018 07:58:10 -0700 (PDT) From: Emil Velikov To: dri-devel@lists.freedesktop.org Subject: [PATCH libdrm 2/2] *-symbols-check: error out when using unset variables Date: Thu, 6 Sep 2018 15:53:34 +0100 Message-Id: <20180906145334.9070-2-emil.l.velikov@gmail.com> X-Mailer: git-send-email 2.18.0 In-Reply-To: <20180906145334.9070-1-emil.l.velikov@gmail.com> References: <20180906145334.9070-1-emil.l.velikov@gmail.com> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Eric Engestrom , emil.l.velikov@gmail.com MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP From: Emil Velikov It will make bugs like the one fixed with previous patch dead obvious. Cc: Eric Engestrom Signed-off-by: Emil Velikov --- amdgpu/amdgpu-symbol-check | 2 ++ etnaviv/etnaviv-symbol-check | 2 ++ exynos/exynos-symbol-check | 2 ++ freedreno/freedreno-symbol-check | 2 ++ intel/intel-symbol-check | 2 ++ libkms/kms-symbol-check | 2 ++ nouveau/nouveau-symbol-check | 2 ++ omap/omap-symbol-check | 2 ++ radeon/radeon-symbol-check | 2 ++ tegra/tegra-symbol-check | 2 ++ 10 files changed, 20 insertions(+) diff --git a/amdgpu/amdgpu-symbol-check b/amdgpu/amdgpu-symbol-check index 90b7a1d6..07477243 100755 --- a/amdgpu/amdgpu-symbol-check +++ b/amdgpu/amdgpu-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.am/libdrm_amdgpuinclude_HEADERS diff --git a/etnaviv/etnaviv-symbol-check b/etnaviv/etnaviv-symbol-check index bc509615..18910688 100755 --- a/etnaviv/etnaviv-symbol-check +++ b/etnaviv/etnaviv-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.sources/LIBDRM_ETNAVIV_H_FILES diff --git a/exynos/exynos-symbol-check b/exynos/exynos-symbol-check index e9f1b04d..49d611e6 100755 --- a/exynos/exynos-symbol-check +++ b/exynos/exynos-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.am/libdrm_exynos*_HEADERS diff --git a/freedreno/freedreno-symbol-check b/freedreno/freedreno-symbol-check index e732c995..6da9d667 100755 --- a/freedreno/freedreno-symbol-check +++ b/freedreno/freedreno-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.sources/LIBDRM_FREEDRENO_H_FILES diff --git a/intel/intel-symbol-check b/intel/intel-symbol-check index 4d30a4b1..de377bef 100755 --- a/intel/intel-symbol-check +++ b/intel/intel-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.sources/LIBDRM_INTEL_H_FILES diff --git a/libkms/kms-symbol-check b/libkms/kms-symbol-check index a5c2120d..30f444f7 100755 --- a/libkms/kms-symbol-check +++ b/libkms/kms-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.sources/LIBKMS_H_FILES diff --git a/nouveau/nouveau-symbol-check b/nouveau/nouveau-symbol-check index b3a24101..6296244c 100755 --- a/nouveau/nouveau-symbol-check +++ b/nouveau/nouveau-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.sources/LIBDRM_NOUVEAU_H_FILES diff --git a/omap/omap-symbol-check b/omap/omap-symbol-check index 0fb4a0f2..16da3c40 100755 --- a/omap/omap-symbol-check +++ b/omap/omap-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.am/libdrm_omap*HEADERS diff --git a/radeon/radeon-symbol-check b/radeon/radeon-symbol-check index 7d79d901..da605bb8 100755 --- a/radeon/radeon-symbol-check +++ b/radeon/radeon-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first five) are taken from the public headers. # A list of the latter should be available Makefile.sources/LIBDRM_RADEON_H_FILES diff --git a/tegra/tegra-symbol-check b/tegra/tegra-symbol-check index 509b678c..8539b95b 100755 --- a/tegra/tegra-symbol-check +++ b/tegra/tegra-symbol-check @@ -1,5 +1,7 @@ #!/bin/bash +set -u + # The following symbols (past the first nine) are taken from tegra.h. FUNCS=$($NM -D --format=bsd --defined-only ${1-.libs/libdrm_tegra.so} | awk '{print $3}'| while read func; do