diff mbox series

[iproute2-next,v2] Makefile: use make -C to change directory

Message ID 3734f49cbe4b7543f09236d02cbe78b515af1e28.1560448299.git.aclaudi@redhat.com
State Accepted
Delegated to: David Ahern
Headers show
Series [iproute2-next,v2] Makefile: use make -C to change directory | expand

Commit Message

Andrea Claudi June 13, 2019, 5:59 p.m. UTC
make provides a handy -C option to change directory before reading
the makefiles or doing anything else.

Use that instead of the "cd dir && make && cd .." pattern, thus
simplifying sintax for some makefiles.

Changes from v1:
- Drop an obviously wrong leftover in testsuite/iproute2/Makefile

Signed-off-by: Andrea Claudi <aclaudi@redhat.com>
---
 Makefile                    | 3 ++-
 testsuite/Makefile          | 6 +++---
 testsuite/iproute2/Makefile | 6 +++---
 3 files changed, 8 insertions(+), 7 deletions(-)

Comments

Matteo Croce June 13, 2019, 8:52 p.m. UTC | #1
On Thu, Jun 13, 2019 at 8:02 PM Andrea Claudi <aclaudi@redhat.com> wrote:
>
> make provides a handy -C option to change directory before reading
> the makefiles or doing anything else.
>
> Use that instead of the "cd dir && make && cd .." pattern, thus
> simplifying sintax for some makefiles.
>
> Changes from v1:
> - Drop an obviously wrong leftover in testsuite/iproute2/Makefile
>
> Signed-off-by: Andrea Claudi <aclaudi@redhat.com>
>

Reviewed-and-tested-by: Matteo Croce <mcroce@redhat.com>
David Ahern June 14, 2019, 1:55 p.m. UTC | #2
On 6/13/19 11:59 AM, Andrea Claudi wrote:
> make provides a handy -C option to change directory before reading
> the makefiles or doing anything else.
> 
> Use that instead of the "cd dir && make && cd .." pattern, thus
> simplifying sintax for some makefiles.
> 
> Changes from v1:
> - Drop an obviously wrong leftover in testsuite/iproute2/Makefile
> 
> Signed-off-by: Andrea Claudi <aclaudi@redhat.com>
> ---
>  Makefile                    | 3 ++-
>  testsuite/Makefile          | 6 +++---
>  testsuite/iproute2/Makefile | 6 +++---
>  3 files changed, 8 insertions(+), 7 deletions(-)

Looks good.
applied to iproute2-next. Thanks
diff mbox series

Patch

diff --git a/Makefile b/Makefile
index 48f469b0d0a85..a87826fee84dd 100644
--- a/Makefile
+++ b/Makefile
@@ -114,7 +114,8 @@  clobber:
 distclean: clobber
 
 check: all
-	cd testsuite && $(MAKE) && $(MAKE) alltests
+	$(MAKE) -C testsuite
+	$(MAKE) -C testsuite alltests
 	@if command -v man >/dev/null 2>&1; then \
 		echo "Checking manpages for syntax errors..."; \
 		$(MAKE) -C man check; \
diff --git a/testsuite/Makefile b/testsuite/Makefile
index 7f247bbc1c803..94a4a8c646ef7 100644
--- a/testsuite/Makefile
+++ b/testsuite/Makefile
@@ -26,10 +26,10 @@  endif
 .PHONY: compile listtests alltests configure $(TESTS)
 
 configure:
-	echo "Entering iproute2" && cd iproute2 && $(MAKE) configure && cd ..;
+	$(MAKE) -C iproute2 configure
 
 compile: configure generate_nlmsg
-	echo "Entering iproute2" && cd iproute2 && $(MAKE) && cd ..;
+	$(MAKE) -C iproute2
 
 listtests:
 	@for t in $(TESTS); do \
@@ -51,7 +51,7 @@  clean: testclean
 	$(MAKE) -C tools clean
 
 distclean: clean
-	echo "Entering iproute2" && cd iproute2 && $(MAKE) distclean && cd ..;
+	$(MAKE) -C iproute2 distclean
 
 $(TESTS): generate_nlmsg testclean
 ifeq (,$(IPVERS))
diff --git a/testsuite/iproute2/Makefile b/testsuite/iproute2/Makefile
index b8a7d5153175a..f8946058b3a99 100644
--- a/testsuite/iproute2/Makefile
+++ b/testsuite/iproute2/Makefile
@@ -4,7 +4,7 @@  SUBDIRS := $(filter-out Makefile,$(wildcard *))
 
 all: configure
 	@for dir in $(SUBDIRS); do \
-		echo "Entering $$dir" && cd $$dir && $(MAKE) && cd ..; \
+		$(MAKE) -C $$dir; \
 	done
 
 link:
@@ -19,12 +19,12 @@  configure: link
 
 clean: link
 	@for dir in $(SUBDIRS); do \
-		echo "Entering $$dir" && cd $$dir && $(MAKE) clean && cd ..; \
+		$(MAKE) -C $$dir clean; \
 	done
 
 distclean: clean
 	@for dir in $(SUBDIRS); do \
-		echo "Entering $$dir" && cd $$dir && $(MAKE) distclean && cd ..; \
+		$(MAKE) -C $$dir distclean; \
 	done
 
 show: link