From patchwork Thu Sep 24 07:50:58 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Panu Matilainen X-Patchwork-Id: 7150 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 13D9B8E76; Thu, 24 Sep 2015 09:52:00 +0200 (CEST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 72C2A8E65 for ; Thu, 24 Sep 2015 09:51:57 +0200 (CEST) Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (Postfix) with ESMTPS id F2E532FAA67 for ; Thu, 24 Sep 2015 07:51:56 +0000 (UTC) Received: from dhcp195.koti.laiskiainen.org.com (vpn1-7-167.ams2.redhat.com [10.36.7.167]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t8O7pqjC004719 for ; Thu, 24 Sep 2015 03:51:56 -0400 From: Panu Matilainen To: dev@dpdk.org Date: Thu, 24 Sep 2015 10:50:58 +0300 Message-Id: <9e5f056c976858f5c69f1e66183743ec2c2a39b3.1443080778.git.pmatilai@redhat.com> In-Reply-To: References: In-Reply-To: References: X-Scanned-By: MIMEDefang 2.68 on 10.5.11.22 Subject: [dpdk-dev] [PATCH 2/3] scripts: move two identical config fixups into a function X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Signed-off-by: Panu Matilainen --- scripts/validate-abi.sh | 23 +++++++++++------------ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/scripts/validate-abi.sh b/scripts/validate-abi.sh index b9c9989..12946d4 100755 --- a/scripts/validate-abi.sh +++ b/scripts/validate-abi.sh @@ -81,6 +81,15 @@ cleanup_and_exit() { exit $1 } +# Make sure we configure SHARED libraries +# Also turn off IGB and KNI as those require kernel headers to build +fixup_config() { + sed -i -e"$ a\CONFIG_RTE_BUILD_SHARED_LIB=y" config/defconfig_$TARGET + sed -i -e"$ a\CONFIG_RTE_NEXT_ABI=n" config/defconfig_$TARGET + sed -i -e"$ a\CONFIG_RTE_EAL_IGB_UIO=n" config/defconfig_$TARGET + sed -i -e"$ a\CONFIG_RTE_LIBRTE_KNI=n" config/defconfig_$TARGET +} + ########################################### #START ############################################ @@ -154,12 +163,7 @@ log "INFO" "Checking out version $TAG1 of the dpdk" # Move to the old version of the tree git checkout $TAG1 -# Make sure we configure SHARED libraries -# Also turn off IGB and KNI as those require kernel headers to build -sed -i -e"$ a\CONFIG_RTE_BUILD_SHARED_LIB=y" config/defconfig_$TARGET -sed -i -e"$ a\CONFIG_RTE_NEXT_ABI=n" config/defconfig_$TARGET -sed -i -e"$ a\CONFIG_RTE_EAL_IGB_UIO=n" config/defconfig_$TARGET -sed -i -e"$ a\CONFIG_RTE_LIBRTE_KNI=n" config/defconfig_$TARGET +fixup_config # Checking abi compliance relies on using the dwarf information in # The shared objects. Thats only included in the DSO's if we build @@ -196,12 +200,7 @@ git reset --hard log "INFO" "Checking out version $TAG2 of the dpdk" git checkout $TAG2 -# Make sure we configure SHARED libraries -# Also turn off IGB and KNI as those require kernel headers to build -sed -i -e"$ a\CONFIG_RTE_BUILD_SHARED_LIB=y" config/defconfig_$TARGET -sed -i -e"$ a\CONFIG_RTE_NEXT_ABI=n" config/defconfig_$TARGET -sed -i -e"$ a\CONFIG_RTE_EAL_IGB_UIO=n" config/defconfig_$TARGET -sed -i -e"$ a\CONFIG_RTE_LIBRTE_KNI=n" config/defconfig_$TARGET +fixup_config # Now configure the build log "INFO" "Configuring DPDK $TAG2"