[riot-notifications] [RIOT-OS/RIOT] make: disable dwarf compression on CI builds (#12198)

Kaspar Schleiser 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:
https://github.com/RIOT-OS/RIOT/pull/12198#discussion_r323111150
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190911/98933463/attachment-0001.htm>


More information about the notifications mailing list