diff mbox series

[ovs-dev] make: don't prompt during build

Message ID 20210402190819.1772840-1-aidan.shribman@gmail.com
State Accepted
Headers show
Series [ovs-dev] make: don't prompt during build | expand

Commit Message

Aidan Shribman April 2, 2021, 7:08 p.m. UTC
When running repeated builds using `make build` you get prompts in cases
the `mv` command is about to overwrite a file which is write-protect.
This patch forced the `mv` w/o prompting for approval.

Signed-off-by: hunchback <aidan.shribman@gmail.com>
---
 Makefile.am     | 2 +-
 lib/automake.mk | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Ben Pfaff April 8, 2021, 3:59 p.m. UTC | #1
On Fri, Apr 02, 2021 at 07:08:19PM +0000, hunchback wrote:
> When running repeated builds using `make build` you get prompts in cases
> the `mv` command is about to overwrite a file which is write-protect.
> This patch forced the `mv` w/o prompting for approval.
> 
> Signed-off-by: hunchback <aidan.shribman@gmail.com>

I think you must be using an unusual configuration, since no one else
has reported this in the 10+ years of the project.

I applied this to master (but I changed "hunchback" to Aidan Shribman;
real names are better).

Thanks,

Ben.
diff mbox series

Patch

diff --git a/Makefile.am b/Makefile.am
index 691a005ad..cb8076433 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -168,7 +168,7 @@  SUFFIXES += .in
 	@if head -n 1 $@.tmp | grep '#!' > /dev/null; then \
 	  chmod +x $@.tmp; \
 	fi
-	$(AM_V_at) mv $@.tmp $@
+	$(AM_V_at) mv -f $@.tmp $@
 
 SUFFIXES += .xml
 %: %.xml
diff --git a/lib/automake.mk b/lib/automake.mk
index 39afbff9d..39901bd6d 100644
--- a/lib/automake.mk
+++ b/lib/automake.mk
@@ -590,7 +590,7 @@  lib/dirs.c: lib/dirs.c.in Makefile
 		-e 's,[@]sysconfdir[@],"$(sysconfdir)",g' \
 		-e 's,[@]pkgdatadir[@],"$(pkgdatadir)",g') \
 	     > lib/dirs.c.tmp && \
-	mv lib/dirs.c.tmp lib/dirs.c
+	mv -f lib/dirs.c.tmp lib/dirs.c
 
 lib/meta-flow.inc: $(srcdir)/build-aux/extract-ofp-fields include/openvswitch/meta-flow.h
 	$(AM_V_GEN)$(run_python) $< meta-flow $(srcdir)/include/openvswitch/meta-flow.h > $@.tmp