* [Qemu-devel] [PATCH 0/2] build: clean up TOOLS definition and usage
@ 2019-08-06 7:07 Paolo Bonzini
2019-08-06 7:07 ` [Qemu-devel] [PATCH 1/2] qemu-ga: clean up TOOLS variable Paolo Bonzini
2019-08-06 7:07 ` [Qemu-devel] [PATCH 2/2] configure: define CONFIG_TOOLS here Paolo Bonzini
0 siblings, 2 replies; 3+ messages in thread
From: Paolo Bonzini @ 2019-08-06 7:07 UTC (permalink / raw)
To: qemu-devel
Some more nasty things discovered during the Meson conversion project.
Paolo Bonzini (2):
qemu-ga: clean up TOOLS variable
configure: define CONFIG_TOOLS here
Makefile | 9 ++++-----
configure | 5 ++++-
2 files changed, 8 insertions(+), 6 deletions(-)
--
1.8.3.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Qemu-devel] [PATCH 1/2] qemu-ga: clean up TOOLS variable
2019-08-06 7:07 [Qemu-devel] [PATCH 0/2] build: clean up TOOLS definition and usage Paolo Bonzini
@ 2019-08-06 7:07 ` Paolo Bonzini
2019-08-06 7:07 ` [Qemu-devel] [PATCH 2/2] configure: define CONFIG_TOOLS here Paolo Bonzini
1 sibling, 0 replies; 3+ messages in thread
From: Paolo Bonzini @ 2019-08-06 7:07 UTC (permalink / raw)
To: qemu-devel
qemu-ga is included in the TOOLS variable without the .exe suffix, and this is
then worked around twice in the Makefile. Do the right thing in configure
instead.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
Makefile | 4 ++--
configure | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index cfb18f1..e532043 100644
--- a/Makefile
+++ b/Makefile
@@ -669,7 +669,7 @@ clean:
! -path ./roms/edk2/BaseTools/Source/Python/UPT/Dll/sqlite3.dll \
-exec rm {} +
rm -f $(edk2-decompressed)
- rm -f $(filter-out %.tlb,$(TOOLS)) $(HELPERS-y) qemu-ga$(EXESUF) TAGS cscope.* *.pod *~ */*~
+ rm -f $(filter-out %.tlb,$(TOOLS)) $(HELPERS-y) TAGS cscope.* *.pod *~ */*~
rm -f fsdev/*.pod scsi/*.pod
rm -f qemu-img-cmds.h
rm -f ui/shader/*-vert.h ui/shader/*-frag.h
@@ -832,7 +832,7 @@ ICON_SIZES=16x16 24x24 32x32 48x48 64x64 128x128 256x256 512x512
install: all $(if $(BUILD_DOCS),install-doc) install-datadir install-localstatedir \
$(if $(INSTALL_BLOBS),$(edk2-decompressed))
ifneq ($(TOOLS),)
- $(call install-prog,$(subst qemu-ga,qemu-ga$(EXESUF),$(TOOLS)),$(DESTDIR)$(bindir))
+ $(call install-prog,$(TOOLS),$(DESTDIR)$(bindir))
endif
ifneq ($(CONFIG_MODULES),)
$(INSTALL_DIR) "$(DESTDIR)$(qemu_moddir)"
diff --git a/configure b/configure
index 5c79145..8e17f44 100755
--- a/configure
+++ b/configure
@@ -6118,7 +6118,7 @@ if [ "$guest_agent" != "no" ]; then
if [ "$softmmu" = no -a "$want_tools" = no ] ; then
guest_agent=no
elif [ "$linux" = "yes" -o "$bsd" = "yes" -o "$solaris" = "yes" -o "$mingw32" = "yes" ] ; then
- tools="qemu-ga $tools"
+ tools="qemu-ga\$(EXESUF) $tools"
guest_agent=yes
elif [ "$guest_agent" != yes ]; then
guest_agent=no
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Qemu-devel] [PATCH 2/2] configure: define CONFIG_TOOLS here
2019-08-06 7:07 [Qemu-devel] [PATCH 0/2] build: clean up TOOLS definition and usage Paolo Bonzini
2019-08-06 7:07 ` [Qemu-devel] [PATCH 1/2] qemu-ga: clean up TOOLS variable Paolo Bonzini
@ 2019-08-06 7:07 ` Paolo Bonzini
1 sibling, 0 replies; 3+ messages in thread
From: Paolo Bonzini @ 2019-08-06 7:07 UTC (permalink / raw)
To: qemu-devel
Defining CONFIG_TOOLS on the basis of $(TOOLS) has the disadvantage
of including it also if e.g. qemu-ga is requested. The correct
information is available in configure, define it there.
This also has the benefit of not installing the manpages for block layer
tools if the only "tool" being built is the guest agent.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
Makefile | 5 ++---
configure | 3 +++
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index e532043..dfd158c 100644
--- a/Makefile
+++ b/Makefile
@@ -87,8 +87,7 @@ endif
include $(SRC_PATH)/rules.mak
-# notempy and lor are defined in rules.mak
-CONFIG_TOOLS := $(call notempty,$(TOOLS))
+# lor is defined in rules.mak
CONFIG_BLOCK := $(call lor,$(CONFIG_SOFTMMU),$(CONFIG_TOOLS))
# Create QEMU_PKGVERSION and FULL_VERSION strings
@@ -797,7 +796,7 @@ ifdef CONFIG_POSIX
$(INSTALL_DATA) docs/interop/qemu-qmp-ref.7 "$(DESTDIR)$(mandir)/man7"
$(INSTALL_DATA) docs/qemu-block-drivers.7 "$(DESTDIR)$(mandir)/man7"
$(INSTALL_DATA) docs/qemu-cpu-models.7 "$(DESTDIR)$(mandir)/man7"
-ifneq ($(TOOLS),)
+ifeq ($(CONFIG_TOOLS),y)
$(INSTALL_DATA) qemu-img.1 "$(DESTDIR)$(mandir)/man1"
$(INSTALL_DIR) "$(DESTDIR)$(mandir)/man8"
$(INSTALL_DATA) qemu-nbd.8 "$(DESTDIR)$(mandir)/man8"
diff --git a/configure b/configure
index 8e17f44..c5a5b44 100755
--- a/configure
+++ b/configure
@@ -6597,6 +6597,9 @@ fi
if test "$profiler" = "yes" ; then
echo "CONFIG_PROFILER=y" >> $config_host_mak
fi
+if test "$want_tools" = "yes" ; then
+ echo "CONFIG_TOOLS=y" >> $config_host_mak
+fi
if test "$slirp" != "no"; then
echo "CONFIG_SLIRP=y" >> $config_host_mak
echo "CONFIG_SMBD_COMMAND=\"$smbd\"" >> $config_host_mak
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-08-06 7:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-06 7:07 [Qemu-devel] [PATCH 0/2] build: clean up TOOLS definition and usage Paolo Bonzini
2019-08-06 7:07 ` [Qemu-devel] [PATCH 1/2] qemu-ga: clean up TOOLS variable Paolo Bonzini
2019-08-06 7:07 ` [Qemu-devel] [PATCH 2/2] configure: define CONFIG_TOOLS here Paolo Bonzini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.