All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Eduardo Habkost" <ehabkost@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>
Subject: [PULL 04/10] build: fix device module builds
Date: Tue,  7 Jul 2020 15:42:23 +0200	[thread overview]
Message-ID: <20200707134229.9773-5-kraxel@redhat.com> (raw)
In-Reply-To: <20200707134229.9773-1-kraxel@redhat.com>

Slightly hackish workaround, works ok as long as we don't
have target-specific modules.  meson will obsolete this.

See comment in the patch for the --verbose description.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Message-id: 20200624131045.14512-5-kraxel@redhat.com

[ kraxel: updated comment from discussions ]

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 Makefile.target | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/Makefile.target b/Makefile.target
index 8ed1eba95b9c..02bd9d7117dd 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -179,6 +179,20 @@ endif # CONFIG_SOFTMMU
 dummy := $(call unnest-vars,,obj-y)
 all-obj-y := $(obj-y)
 
+#
+# common-obj-m has some crap here, probably as side effect from
+# unnest-vars recursing into target directories to fill obj-y and not
+# properly handling the -m case.
+#
+# Clear common-obj-m as workaround.  Fixes suspious dependency errors
+# when building devices as modules.  A bit hackish, but should be ok
+# as long as we do not have any target-specific modules.
+#
+# The meson-based build system currently in development doesn't need
+# unnest-vars and will obsolete this workaround.
+#
+common-obj-m :=
+
 include $(SRC_PATH)/Makefile.objs
 dummy := $(call unnest-vars,.., \
                authz-obj-y \
-- 
2.18.4



  parent reply	other threads:[~2020-07-07 13:43 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-07 13:42 [PULL 00/10] Modules 20200707 patches Gerd Hoffmann
2020-07-07 13:42 ` [PULL 01/10] module: qom module support Gerd Hoffmann
2020-07-07 13:42 ` [PULL 02/10] object: " Gerd Hoffmann
2020-07-07 13:42 ` [PULL 03/10] qdev: device " Gerd Hoffmann
2020-07-07 13:42 ` Gerd Hoffmann [this message]
2020-07-07 13:42 ` [PULL 05/10] ccid: build smartcard as module Gerd Hoffmann
2020-07-07 13:42 ` [PULL 06/10] usb: build usb-redir " Gerd Hoffmann
2020-07-07 13:42 ` [PULL 07/10] vga: build qxl " Gerd Hoffmann
2020-07-07 13:42 ` [PULL 08/10] vga: build virtio-gpu only once Gerd Hoffmann
2020-07-07 13:42 ` [PULL 09/10] vga: build virtio-gpu as module Gerd Hoffmann
2020-07-07 13:42 ` [PULL 10/10] chardev: enable modules, use for braille Gerd Hoffmann
2020-07-09 19:01 ` [PULL 00/10] Modules 20200707 patches Peter Maydell
2020-07-10  8:12 ` Thomas Huth
2020-07-10 16:58 ` Philippe Mathieu-Daudé
  -- strict thread matches above, loose matches on Subject: below --
2020-07-02 12:20 [PULL 00/10] Modules 20200702 patches Gerd Hoffmann
2020-07-02 12:20 ` [PULL 04/10] build: fix device module builds Gerd Hoffmann
2020-07-03  9:01   ` Claudio Fontana
2020-07-03 10:25     ` Gerd Hoffmann
2020-07-03 13:00       ` Claudio Fontana
2020-07-06 13:14         ` Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200707134229.9773-5-kraxel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=berrange@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.