You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by "raiden00pl (via GitHub)" <gi...@apache.org> on 2023/06/12 07:13:58 UTC

[GitHub] [nuttx] raiden00pl commented on a diff in pull request #9499: debug: change generate debug symbols default enable

raiden00pl commented on code in PR #9499:
URL: https://github.com/apache/nuttx/pull/9499#discussion_r1226187267


##########
Kconfig:
##########
@@ -1904,7 +1904,7 @@ config HEAP_COLORATION
 
 config DEBUG_SYMBOLS
 	bool "Generate Debug Symbols"
-	default n
+	default y

Review Comment:
   Enabling this option by default can lead to big f*ckups when ELF file with symbols is sent to users. 
   DJI used to ship their pro drone cameras with debug symbols, which made it possible to reverse engineer their products but also showed a violation of the GPL license. I don't think the company was happy about that.
   Potentially this is too dangerous for the fw provider.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org