config_host.mk.in |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 3909c3ea95992e9edfb8e1213edef732abf50cbf
Author:     Christian Lohmaier <lohmaier+libreoff...@googlemail.com>
AuthorDate: Wed May 8 14:35:11 2024 +0200
Commit:     Balazs Varga <balazs.varga.ext...@allotropia.de>
CommitDate: Wed Oct 2 19:47:38 2024 +0200

    make sure KEEP_AWAKE_CMD gets only evaluated once
    
    if using deferred evaluation then the creation of targets via $(call
    gb_Top_GbuildModuleRules,foo) would result in it getting processed over
    900 times, and on windows spawning a shell with the two ps and awk
    command that takes 5 minutes.
    
    If future commands need to be run when the rule is run, then an
    alternative is to escape the variable in the define for the
    gb_Top_GbuildModuleRules macro via double-$ → $$(KEEP_AWAKE_CMD) - but I
    cannot think of any method where that should be necessary
    
    Change-Id: I126e4d317bd77f359ca9f39adf00cc88846af281
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167351
    Tested-by: Jenkins
    Reviewed-by: Christian Lohmaier <lohmaier+libreoff...@googlemail.com>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/174253
    Tested-by: Balazs Varga <balazs.varga.ext...@allotropia.de>
    Reviewed-by: Balazs Varga <balazs.varga.ext...@allotropia.de>

diff --git a/config_host.mk.in b/config_host.mk.in
index 707dc94f7768..a8b458c6ee00 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -787,7 +787,7 @@ include $(BUILDDIR)/config_$(gb_Side)_lang.mk
 # does use some of the variables defined above
 include @SRC_ROOT@/download.lst
 
-KEEP_AWAKE_CMD=@KEEP_AWAKE_CMD@
+KEEP_AWAKE_CMD:=@KEEP_AWAKE_CMD@
 
 # used for WSL-as-helper-builds where the build runs from within git-bash/MSYS 
that otherwise would
 # mess with anything that looks like a path (starts with a /) but cannot be 
resolved to a target

Reply via email to