From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] scripts: fix relpath.sh output when $prefix is set in environment Date: Fri, 11 Dec 2015 14:04:01 +0100 Message-ID: <5188347.04ruq6Yd6l@xps13> References: <37b46e91bdb51e6116df9a7edc2eb79206237f1c.1449837863.git.pmatilai@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-f41.google.com (mail-wm0-f41.google.com [74.125.82.41]) by dpdk.org (Postfix) with ESMTP id 263108E7B for ; Fri, 11 Dec 2015 14:05:18 +0100 (CET) Received: by wmnn186 with SMTP id n186so30575155wmn.0 for ; Fri, 11 Dec 2015 05:05:17 -0800 (PST) In-Reply-To: <37b46e91bdb51e6116df9a7edc2eb79206237f1c.1449837863.git.pmatilai@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-12-11 14:44, Panu Matilainen: > When relpath.sh is called from install target with prefix set, eg > "make install DESTDIR=/tmp/dpdk-root prefix=/usr", the prefix > from the environment leaks to relpath.sh internal helper variable and > causes incorrect symlinks to be generated in sdk $(targetdir): > > include -> /usr../../../include/dpdk > lib -> /usr../../../lib > > Initialize the local variable to empty to avoid side-effects from > environment. > > Signed-off-by: Panu Matilainen Good catch! Fixes: 6b62a72a70d0 ("mk: install a standard cutomizable tree") Applied, thanks