diff mbox

daemon: fix parallel build

Message ID b7b648fdc787feae767f23b1a334ce5e7f13072a.1494266507.git.baruch@tkos.co.il
State Accepted
Commit c23236d387a84a73940b490f363c3d3d364e3c96
Headers show

Commit Message

Baruch Siach May 8, 2017, 6:01 p.m. UTC
The 'ready' target creates a symlink in the libslack/ subdirectory to make up
the path to header files. Unfortunately, the target building sources files
that need on that symlink, does not depend on the 'ready' target. This might
break highly parallel builds. As a workaround make the 'ready' target before
making 'all'.

Should fix:
http://autobuild.buildroot.net/results/ed2/ed21ac166f2151aa69a7790a17ff05f05afa512d/
http://autobuild.buildroot.net/results/333/3338539743c8b0399c6b0fcbbb7c28b58bf3f387/

Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 package/daemon/daemon.mk | 1 +
 1 file changed, 1 insertion(+)

Comments

Peter Korsgaard May 9, 2017, 3:14 p.m. UTC | #1
>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:

 > The 'ready' target creates a symlink in the libslack/ subdirectory to make up
 > the path to header files. Unfortunately, the target building sources files
 > that need on that symlink, does not depend on the 'ready' target. This might

Dropped 'on' from 'need on that' and committed, thanks.
diff mbox

Patch

diff --git a/package/daemon/daemon.mk b/package/daemon/daemon.mk
index dfa573f42652..79cf3f7aab74 100644
--- a/package/daemon/daemon.mk
+++ b/package/daemon/daemon.mk
@@ -14,6 +14,7 @@  define DAEMON_CONFIGURE_CMDS
 endef
 
 define DAEMON_BUILD_CMDS
+	$(TARGET_MAKE_ENV) $(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D) ready
 	$(TARGET_MAKE_ENV) $(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D)
 endef