From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 2A0632BCE for ; Thu, 10 Mar 2016 11:54:05 +0100 (CET) Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (Postfix) with ESMTPS id A2416C00F22D; Thu, 10 Mar 2016 10:54:04 +0000 (UTC) Received: from sopuli.koti.laiskiainen.org.com (vpn1-4-209.ams2.redhat.com [10.36.4.209]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u2AAs2w0000307; Thu, 10 Mar 2016 05:54:03 -0500 From: Panu Matilainen To: dev@dpdk.org Date: Thu, 10 Mar 2016 12:53:36 +0200 Message-Id: <2e7fcffec863b7f60cf62707b3be3dd81a5de991.1457607217.git.pmatilai@redhat.com> In-Reply-To: References: In-Reply-To: References: X-Scanned-By: MIMEDefang 2.68 on 10.5.11.24 Subject: [dpdk-dev] [PATCH 2/3] scripts: avoid editing defconfig_* files in validate-abi.sh 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: , X-List-Received-Date: Thu, 10 Mar 2016 10:54:05 -0000 The defconfig_* files are templates which are not supposed to be edited, and doing so tends to leave unwanted cruft behind. Edit the "working copy" config instead, which is the intended DPDK way. Signed-off-by: Panu Matilainen --- scripts/validate-abi.sh | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/scripts/validate-abi.sh b/scripts/validate-abi.sh index f094582..ea60639 100755 --- a/scripts/validate-abi.sh +++ b/scripts/validate-abi.sh @@ -90,11 +90,11 @@ cleanup_and_exit() { # 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 - sed -i -e"$ a\CONFIG_RTE_KNI_KMOD=n" config/defconfig_$TARGET + sed -i -e"$ a\CONFIG_RTE_BUILD_SHARED_LIB=y" $TARGET/.config + sed -i -e"$ a\CONFIG_RTE_NEXT_ABI=n" $TARGET/.config + sed -i -e"$ a\CONFIG_RTE_EAL_IGB_UIO=n" $TARGET/.config + sed -i -e"$ a\CONFIG_RTE_LIBRTE_KNI=n" $TARGET/.config + sed -i -e"$ a\CONFIG_RTE_KNI_KMOD=n" $TARGET/.config } ########################################### @@ -177,8 +177,6 @@ log "INFO" "Checking out version $TAG1 of the dpdk" # Move to the old version of the tree git checkout $HASH1 -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 # with -g @@ -189,6 +187,8 @@ export EXTRA_LDFLAGS="$EXTRA_LDFLAGS -g" log "INFO" "Configuring DPDK $TAG1" make config T=$TARGET O=$TARGET > $VERBOSE 2>&1 +fixup_config + log "INFO" "Building DPDK $TAG1. This might take a moment" make $MAKEJOBS O=$TARGET > $VERBOSE 2>&1 @@ -214,12 +214,12 @@ git reset --hard log "INFO" "Checking out version $TAG2 of the dpdk" git checkout $HASH2 -fixup_config - # Now configure the build log "INFO" "Configuring DPDK $TAG2" make config T=$TARGET O=$TARGET > $VERBOSE 2>&1 +fixup_config + log "INFO" "Building DPDK $TAG2. This might take a moment" make $MAKEJOBS O=$TARGET > $VERBOSE 2>&1 -- 2.5.0