blob: 1b2e6dc50b4706292a1428833919c96dbf2dda14 (
plain) (
blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
|
performance: 0.911
ppc: 0.680
graphic: 0.535
files: 0.533
device: 0.527
vnc: 0.503
PID: 0.503
semantic: 0.469
network: 0.461
socket: 0.444
register: 0.439
architecture: 0.424
risc-v: 0.416
permissions: 0.411
kernel: 0.384
hypervisor: 0.383
debug: 0.380
peripherals: 0.365
TCG: 0.342
VMM: 0.315
i386: 0.289
boot: 0.281
arm: 0.269
virtual: 0.241
KVM: 0.235
user-level: 0.233
x86: 0.196
assembly: 0.186
mistranslation: 0.167
Missing 'cmp' utility makes build take 10 times as long
I have been doing some work cross compiling qemu for Windows using a minimal Fedora container. Recently I started hitting some timeouts on the CI service and noticed a build of all targets was going over 1 hour.
It seems like the 'cmp' utility from diffutils is used somewhere in the process and if it's missing, either a configure or a make gets run way too many times - I'll try to pull logs from the CI system at some stage soon.
Could a warning or error be added if cmp is missing?
cmp is used in the makefiles.
And there is some kind of warning during build if it is missing:
/bin/sh: cmp: command not found
But perhaps it should abort the build in this case.
Something like that helps:
diff --git a/Makefile b/Makefile
index 40e4f7677bde..05e029bd99db 100644
--- a/Makefile
+++ b/Makefile
@@ -482,6 +482,7 @@ include $(SRC_PATH)/tests/Makefile.include
all: $(DOCS) $(if $(BUILD_DOCS),sphinxdocs) $(TOOLS) $(HELPERS-y) recurse-all modules $(vhost-user-json-y)
qemu-version.h: FORCE
+ @type cmp
$(call quiet-command, \
(printf '#define QEMU_PKGVERSION "$(QEMU_PKGVERSION)"\n'; \
printf '#define QEMU_FULL_VERSION "$(FULL_VERSION)"\n'; \
Does this problem still persist with the latest version of QEMU (since we switched the build system mostly to meson now)?
[Expired for QEMU because there has been no activity for 60 days.]
|
