[riot-notifications] [RIOT-OS/RIOT] make: disable dwarf compression on CI builds (#12198)
notifications at github.com
Wed Sep 11 10:12:16 CEST 2019
kaspar030 commented on this pull request.
> @@ -39,9 +39,10 @@ endif
CFLAGS += -fno-common
# Compress debug info. This saves approximately 50% of disk usage.
-# It has no effect if debugging information is not emitted, so it can be left
-# on unconditionally.
-OPTIONAL_CFLAGS += -gz
+# This makes ccache less effective, so disable for CI builds.
+ifneq (1, $(RIOT_CI_BUILD))
That seems reasonable, but on my laptop, the performance hit is only ~20%, not 50%. It is quite beefy compared to the multicore CI workers (per-core wise). This might be more affecting build performance on those CPU or memory bound setups.
Also, ccache doesn't compress cache contents by default ("compression=false"), so this PR does save 50% space. (even when no debug symbols are generated...) CI enables ccache compression, which already brings down cache content size to the same level as with "-gz".
So I think locally this is might be useful.
That said, shouldn't using ccache be the first hint to every RIOT developer? Is any of you guys *waiting for local single builds* longer than a couple of seconds??
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications