From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 7/7] mk: Add hierarchy-file support (linux mod) Date: Fri, 27 Nov 2015 16:40:51 +0100 Message-ID: <7380042.Av6xXtCM9v@xps13> References: <1442608390-12537-1-git-send-email-mario.alfredo.c.arevalo@intel.com> <1442608390-12537-8-git-send-email-mario.alfredo.c.arevalo@intel.com> <5600FB8B.6070909@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Panu Matilainen Return-path: Received: from mail-wm0-f47.google.com (mail-wm0-f47.google.com [74.125.82.47]) by dpdk.org (Postfix) with ESMTP id 735415683 for ; Fri, 27 Nov 2015 16:42:11 +0100 (CET) Received: by wmww144 with SMTP id w144so59845830wmw.1 for ; Fri, 27 Nov 2015 07:42:11 -0800 (PST) In-Reply-To: <5600FB8B.6070909@redhat.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" 2015-09-22 09:56, Panu Matilainen: > On 09/18/2015 11:33 PM, Mario Carrillo wrote: > > +MOD_DIR := $(DESTDIR)/lib/modules > > +KERNEL_DIR := $(shell uname -r)/extra > > Please don't assume one is always building for the running kernel. > Defaulting around uname -r is perfectly reasonable, but there needs to > be a way to override it from the cli. For example rte.vars.mk has this: > > # can be overriden by make command line or exported environment variable > RTE_KERNELDIR ?= /lib/modules/$(shell uname -r)/build For info, the ?= operator is a way to give a default value after having included other makefiles which may set a different value. Both ?= and := (or simply =) are overriden by the command line. Only "override VAR =" will not be overriden by the command line.