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

Martine Lenders notifications at github.com
Wed Sep 11 10:29:40 CEST 2019


miri64 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))

Mh... Then maybe a combination of the two. I did not consider that `ccache` is also run locally sometimes (no, I don't use it, only when doing multi-board builds, because having just a small-ish SSD in my PC here, I value HDD space over time (also I have an octo-core so it is pretty fast anyway) ;-P). I was just thinking that the dependency is not "the CI", but `ccache` in this instance. There are other CIs used in our ecosystem than Murdock and not all use `ccache`. 

Maybe a combination of the two then?

```suggestion
ifneq (1,$(RIOT_CI_BUILD))
ifeq (,$(CCACHE))
```

-- 
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_r323118317
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190911/e9e62467/attachment.htm>


More information about the notifications mailing list