Re: [PATCH iproute2 v5 7/7] configure: add the --libdir option

On Thu, Oct 14, 2021 at 01:02:41PM +0200, Andrea Claudi wrote:

On Thu, Oct 14, 2021 at 12:10:53PM +0200, Phil Sutter wrote:

quoted

Hi Andrea,

On Thu, Oct 14, 2021 at 10:50:55AM +0200, Andrea Claudi wrote:
[...]

quoted

diff --git a/Makefile b/Makefile
index 5eddd504..f6214534 100644
--- a/Makefile
+++ b/Makefile

@@ -1,6 +1,8 @@
 # SPDX-License-Identifier: GPL-2.0
 # Top level Makefile for iproute2
 
+-include config.mk
+

Assuming config.mk may be missing (as dash-prefix is used).

quoted

 ifeq ("$(origin V)", "command line")
 VERBOSE = $(V)
 endif

@@ -13,7 +15,6 @@ MAKEFLAGS += --no-print-directory
 endif
 
 PREFIX?=/usr
-LIBDIR?=$(PREFIX)/lib

Dropping this leads to trouble if config.mk is missing or didn't define
it. Can't you just leave it in place? Usually config.mk would override
it anyway, no?

config.mk may miss at the first make call, but the "all" target calls
config.mk, which in turns re-generate it. Thus LIBDIR is defined when
the target all executes.

Ah, I forgot the call to configure from make. So full series:

Acked-by: Phil Sutter <phil@nwl.cc>

Thanks, Phil