From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?windows-1252?Q?Simon_K=E5gstr=F6m?= Subject: Re: [PATCH] kni: Use utsrelease.h to determine Ubuntu kernel version Date: Thu, 28 May 2015 12:37:17 +0200 Message-ID: <5566EFDD.5040209@netinsight.net> References: <20150527134524.5f107cac@miho> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: "dev@dpdk.org" To: "Wodkowski, PawelX" , "Zhang, Helin" , Alexander Guy , Julien Cretin , "Buriez, Patrice" Return-path: Received: from ernst.netinsight.se (ernst.netinsight.se [194.16.221.21]) by dpdk.org (Postfix) with SMTP id 5B58611C5 for ; Thu, 28 May 2015 12:37:28 +0200 (CEST) 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" On 2015-05-28 12:05, Wodkowski, PawelX wrote: >>> >>> -UBUNTU_KERNEL_CODE := $(shell cut -d' ' -f2 /proc/version_signature | \ >>> - cut -d'~' -f1 | cut -d- -f1,2 | tr .- $(comma)) >>> +UBUNTU_KERNEL_CODE := $(shell echo `grep UTS_RELEASE >>> $(RTE_KERNELDIR)/include/generated/utsrelease.h \ >>> + | cut -d '"' -f2 | cut -d- -f1,2 | tr .- $(comma)`,1) > > It is fine for me if it do the job and does not break build on other OS (also other > Ubuntu versions especially 12.04 if we still support it). > Please only check if UTS_RELEASE is available on all Ubuntu versions DPDK support. >>From some digging, it appears it entered the kernel tree in 2006 and moved to include/generated/ in 2009 so I guess that should be fine for DPDK builds? // Simon