From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] mk: enable next abi in static libs Date: Mon, 06 Jul 2015 15:18:51 +0200 Message-ID: <10048439.cxXN29b43P@xps13> References: <1435874746-32095-1-git-send-email-thomas.monjalon@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit To: dev@dpdk.org Return-path: Received: from mail-wi0-f176.google.com (mail-wi0-f176.google.com [209.85.212.176]) by dpdk.org (Postfix) with ESMTP id 81BFB2A58 for ; Mon, 6 Jul 2015 15:20:01 +0200 (CEST) Received: by wiga1 with SMTP id a1so228668688wig.0 for ; Mon, 06 Jul 2015 06:20:01 -0700 (PDT) In-Reply-To: <1435874746-32095-1-git-send-email-thomas.monjalon@6wind.com> 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" Any comment or ack? 2015-07-03 00:05, Thomas Monjalon: > When a change makes really hard to keep ABI compatibility, > instead of waiting next release to break the ABI, it is smoother > to introduce the new code and enable it only for static libraries. > The flag RTE_NEXT_ABI may be used to "ifdef" the new code. > When the release is out, a dynamically linked application can use > the new shared libraries without rebuild while developpers can prepare > their application for the next ABI by reading the deprecation notice > and easily testing the new code. > When starting the next release cycle, the "ifdefs" will be removed > and the ABI break will be marked by incrementing LIBABIVER. > > The new option CONFIG_RTE_NEXT_ABI is not defined in the configuration > templates because it is deduced from CONFIG_RTE_BUILD_SHARED_LIB. > It is automatically enabled for static libraries and disabled for > shared libraries. > It can be forced to another value by editing the generated .config file. > It shouldn't be enabled for shared libraries because it would break the > ABI without changing the version number LIBABIVER. That's why a warning > is printed in this case. > > The guideline is also updated to integrate this new possibility. > > Signed-off-by: Thomas Monjalon