diff mbox series

[1/1] Skip strip for builds with debug symbols

Message ID 20210514002739.93907-1-james.hilliard1@gmail.com
State Rejected
Headers show
Series [1/1] Skip strip for builds with debug symbols | expand

Commit Message

James Hilliard May 14, 2021, 12:27 a.m. UTC 
Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
---
 Makefile.flags | 1 +
 1 file changed, 1 insertion(+)

Comments

Stefano Babic May 14, 2021, 8:53 a.m. UTC | #1 
On 14.05.21 02:27, James Hilliard wrote:
> Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
> ---
>   Makefile.flags | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/Makefile.flags b/Makefile.flags
> index d3ca49d..9c979e5 100644
> --- a/Makefile.flags
> +++ b/Makefile.flags
> @@ -77,6 +77,7 @@ KBUILD_CFLAGS += $(call cc-option,-fno-asynchronous-unwind-tables,)
>   ifneq ($(CONFIG_DEBUG),y)
>   KBUILD_CFLAGS += $(call cc-option,-Os,$(call cc-option,-O2,))
>   else
> +SKIP_STRIP = y
>   KBUILD_CFLAGS += $(call cc-option,-g,)
>   #KBUILD_CFLAGS += "-D_FORTIFY_SOURCE=2"
>   ifeq ($(CONFIG_DEBUG_PESSIMIZE),y)
> 

This is not required. After the build, there is a swupdate_unstripped 
and swupdate. The first one contains debug symbols and can be used for 
debugging (of course, if -g is passed to gcc).

Best regards,
Stefano Babic
diff mbox series

Patch

diff --git a/Makefile.flags b/Makefile.flags
index d3ca49d..9c979e5 100644
--- a/Makefile.flags
+++ b/Makefile.flags
@@ -77,6 +77,7 @@  KBUILD_CFLAGS += $(call cc-option,-fno-asynchronous-unwind-tables,)
 ifneq ($(CONFIG_DEBUG),y)
 KBUILD_CFLAGS += $(call cc-option,-Os,$(call cc-option,-O2,))
 else
+SKIP_STRIP = y
 KBUILD_CFLAGS += $(call cc-option,-g,)
 #KBUILD_CFLAGS += "-D_FORTIFY_SOURCE=2"
 ifeq ($(CONFIG_DEBUG_PESSIMIZE),y)