From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH v2 1/2] version: switch to year/month version numbers Date: Wed, 10 Feb 2016 15:31:08 +0000 Message-ID: <20160210153107.GA10620@bricha3-MOBL3> References: <1450704384-9986-2-git-send-email-bruce.richardson@intel.com> <1455114792-1603-1-git-send-email-john.mcnamara@intel.com> <1877379.9acYIlBRDX@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: Thomas Monjalon Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 6F3B537A6 for ; Wed, 10 Feb 2016 16:31:12 +0100 (CET) Content-Disposition: inline In-Reply-To: <1877379.9acYIlBRDX@xps13> 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" On Wed, Feb 10, 2016 at 04:11:40PM +0100, Thomas Monjalon wrote: > 2016-02-10 14:33, John McNamara: > > /** > > * Major version number i.e. the x in x.y.z > > */ > > -#define RTE_VER_MAJOR 2 > > +#define RTE_VER_MAJOR 16 > > > > /** > > * Minor version number i.e. the y in x.y.z > > */ > > -#define RTE_VER_MINOR 3 > > +#define RTE_VER_MINOR 4 > > > > I liked the idea of Bruce of renaming these constants to > *_YEAR and *_MONTH. > I also think RTE_VER_PATCH_LEVEL should be RTE_VER_MINOR. > And maybe that RTE_VER_PATCH_RELEASE should be RTE_VER_RELEASE. > > It shouldn't hurt applications as they must use RTE_VERSION > and rte_version(). > > Opinions? Well, I always tend to like Bruce's ideas too, so +1 from me :-) /Bruce