[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/11] Ui 20171013 patches
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PULL 00/11] Ui 20171013 patches |
Date: |
Mon, 16 Oct 2017 11:19:13 +0200 |
Hi,
> I'm afraid this is quite likely to fail build smoke test again. I've
> just tried a build on OpenBSD with the bash -> sh fix in it, and I
> found that it still tried to build the keycodemap files in parallel
> with checking out the GIT submodules. It also tried to run the
> mkdir -p dtc/libfdt in parallel and this caused git to refuse to
> checkout the dtc module due to that empty dir existing. So there's
> still some deps problems in here I think that let make build in
> the wrong order :-(
> In all my debugging the one thing I've seen work correctly is the
> re-running of configure (via config.status), which always happens
> earlier and is serialized wrt everything else. So I wonder if we
> should change direction slightly, and have configure checkout the
> submodules. Then just make sure that config.status is triggered
> when submodules are out of date.
I've noticed configure running *in parallel* to other stuff.
Reproducer:
(1) Apply patch #1
(2) run normal build
(3) make -C dtc clean
(4) touch configure
(5) make
Watch configure and dtc build running in parallel.
I think the added Makefile dependency breaks it.
Incremental fix (also pushed to queue/ui):
--- a/Makefile
+++ b/Makefile
@@ -35,8 +35,6 @@ endif
.git-submodule-status: git-submodule-update config-host.mak
-Makefile: .git-submodule-status
-
# Check that we're not trying to do an out-of-tree build from
# a tree that's been used for an in-tree build.
ifneq ($(realpath $(SRC_PATH)),$(realpath .))
@@ -107,6 +105,7 @@ endif
GENERATED_FILES += $(TRACE_HEADERS)
GENERATED_FILES += $(TRACE_SOURCES)
GENERATED_FILES += $(BUILD_DIR)/trace-events-all
+GENERATED_FILES += .git-submodule-status
trace-group-name = $(shell dirname $1 | sed -e 's/[^a-zA-Z0-9]/_/g')
cheers,
Gerd
- [Qemu-devel] [PULL 06/11] ui: don't export qemu_input_event_new_key, (continued)
- [Qemu-devel] [PULL 06/11] ui: don't export qemu_input_event_new_key, Gerd Hoffmann, 2017/10/13
- [Qemu-devel] [PULL 05/11] ui: convert key events to QKeyCodes immediately, Gerd Hoffmann, 2017/10/13
- [Qemu-devel] [PULL 11/11] gtk: fix wrong id between texture and framebuffer, Gerd Hoffmann, 2017/10/13
- [Qemu-devel] [PULL 03/11] ui: add keycodemapdb repository as a GIT submodule, Gerd Hoffmann, 2017/10/13
- [Qemu-devel] [PULL 02/11] docker: don't rely on submodules existing in the main checkout, Gerd Hoffmann, 2017/10/13
- [Qemu-devel] [PULL 01/11] build: automatically handle GIT submodule checkout for dtc, Gerd Hoffmann, 2017/10/13
- [Qemu-devel] [PULL 07/11] tools: add qemu-keymap, Gerd Hoffmann, 2017/10/13
- [Qemu-devel] [PULL 04/11] ui: convert common input code to keycodemapdb, Gerd Hoffmann, 2017/10/13
- [Qemu-devel] [PULL 10/11] ui/gtk: Fix deprecation of vte_terminal_copy_clipboard, Gerd Hoffmann, 2017/10/13
- Re: [Qemu-devel] [PULL 00/11] Ui 20171013 patches, Daniel P. Berrange, 2017/10/13
- Re: [Qemu-devel] [PULL 00/11] Ui 20171013 patches,
Gerd Hoffmann <=
- [Qemu-devel] [PULL 09/11] pc-bios/keymaps: keymaps update, Gerd Hoffmann, 2017/10/13