Re: [PATCH v3 1/3] remove CONFIG_DEBUG_INFO

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 07, 2019 at 09:06:57AM +0200, Rouven Czerwinski wrote:
> From: Oleksij Rempel <linux@xxxxxxxxxxxxxxxx>
> 
> The ELF file should have debug symbols, the binary should have no symbols any way.
> 
> Signed-off-by: Oleksij Rempel <linux@xxxxxxxxxxxxxxxx>
> Signed-off-by: Rouven Czerwinski <r.czerwinski@xxxxxxxxxxxxxx>
> ---
>  Makefile       | 4 +---
>  common/Kconfig | 6 ------
>  2 files changed, 1 insertion(+), 9 deletions(-)

Applied, thanks

Sascha

> 
> diff --git a/Makefile b/Makefile
> index 29347b6780..900117e727 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -457,9 +457,7 @@ endif # $(dot-config)
>  
>  include $(srctree)/arch/$(ARCH)/Makefile
>  
> -ifdef CONFIG_DEBUG_INFO
> -CFLAGS		+= -g
> -endif
> +CFLAGS		+= -ggdb3
>  
>  # Force gcc to behave correct even for buggy distributions
>  CFLAGS          += $(call cc-option, -fno-stack-protector)
> diff --git a/common/Kconfig b/common/Kconfig
> index 7832df5c55..ffb7537592 100644
> --- a/common/Kconfig
> +++ b/common/Kconfig
> @@ -999,12 +999,6 @@ config DEFAULT_LOGLEVEL
>  	  7    debug-level messages (debug)
>  	  8    verbose debug messages (vdebug)
>  
> -config DEBUG_INFO
> -	bool
> -	prompt "enable debug symbols"
> -	help
> -	  Enable build of barebox with -g.
> -
>  config DEBUG_LL
>  	bool
>  	depends on HAS_DEBUG_LL
> -- 
> 2.21.0
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux