From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v3] mk: enable next abi preview Date: Mon, 13 Jul 2015 10:48:36 +0200 Message-ID: <1531097.HZDUsMDq83@xps13> References: <1436367323-19968-3-git-send-email-thomas.monjalon@6wind.com> <1436373856-29468-1-git-send-email-thomas.monjalon@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: "Mcnamara, John" Return-path: Received: from mail-wi0-f175.google.com (mail-wi0-f175.google.com [209.85.212.175]) by dpdk.org (Postfix) with ESMTP id 98B7C569A for ; Mon, 13 Jul 2015 10:49:50 +0200 (CEST) Received: by widjy10 with SMTP id jy10so62634272wid.1 for ; Mon, 13 Jul 2015 01:49:50 -0700 (PDT) In-Reply-To: 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" 2015-07-13 07:32, Mcnamara, John: > This change to enable CONFIG_RTE_NEXT_ABI=n breaks validate-abi.sh > because master won't compile with CONFIG_RTE_BUILD_SHARED_LIB=y and > CONFIG_RTE_NEXT_ABI=n: My bad. I thought I was testing both cases (next ABI and stable one) but it appears only the "next one" was tested. The error is trivial: - $(Q)ln -s -f $< $(RTE_OUTPUT)/lib/$(LIBSONAME) + $(Q)ln -s -f $< $(basename $(basename $@)) The double basename should apply to NEXT_ABI case only.