xiaoxiang781216 commented on code in PR #3333: URL: https://github.com/apache/nuttx-apps/pull/3333#discussion_r2698410178
########## canutils/candump/CMakeLists.txt: ########## @@ -20,7 +20,7 @@ # # ############################################################################## -if(CONFIG_CANUTILS_CANDUMP) +if(CONFIG_CANUTILS_CANDUMP AND CONFIG_ENABLE_ALL_SIGNALS) Review Comment: let's add depends on ENABLE_ALL_SIGNALS on Kconfig directly ########## testing/drivers/nand_sim/CMakeLists.txt: ########## @@ -20,7 +20,7 @@ # # ############################################################################## -if(CONFIG_TESTING_NAND_SIM) +if(CONFIG_TESTING_NAND_SIM AND CONFIG_ENABLE_ALL_SIGNALS) Review Comment: ditto ########## testing/drivers/nand_sim/Make.defs: ########## @@ -21,5 +21,7 @@ ############################################################################ ifneq ($(CONFIG_TESTING_NAND_SIM),) +ifneq ($(CONFIG_ENABLE_ALL_SIGNALS),) Review Comment: move to Kconfig ########## system/sensortest/Make.defs: ########## @@ -21,5 +21,7 @@ ############################################################################ ifneq ($(CONFIG_SYSTEM_SENSORTEST),) +ifneq ($(CONFIG_ENABLE_ALL_SIGNALS),) Review Comment: ditto ########## examples/xedge_demo/Make.defs: ########## @@ -21,5 +21,7 @@ ############################################################################ ifneq ($(CONFIG_EXAMPLES_XEDGE_DEMO),) +ifneq ($(CONFIG_ENABLE_ALL_SIGNALS),) Review Comment: revert ########## examples/zerocross/Make.defs: ########## @@ -21,5 +21,7 @@ ############################################################################ ifneq ($(CONFIG_EXAMPLES_ZEROCROSS),) +ifneq ($(CONFIG_ENABLE_ALL_SIGNALS),) Review Comment: move to Kconfig ########## canutils/candump/Make.defs: ########## @@ -21,5 +21,7 @@ ############################################################################/ ifneq ($(CONFIG_CANUTILS_CANDUMP),) +ifneq ($(CONFIG_ENABLE_ALL_SIGNALS),) Review Comment: ditto ########## examples/oneshot/Make.defs: ########## @@ -21,5 +21,7 @@ ############################################################################ ifneq ($(CONFIG_EXAMPLES_ONESHOT),) +ifneq ($(CONFIG_ENABLE_ALL_SIGNALS),) Review Comment: move to Kconfig ########## examples/zerocross/CMakeLists.txt: ########## @@ -20,7 +20,7 @@ # # ############################################################################## -if(CONFIG_EXAMPLES_ZEROCROSS) +if(CONFIG_EXAMPLES_ZEROCROSS AND CONFIG_ENABLE_ALL_SIGNALS) Review Comment: ditto ########## examples/i2sloop/Make.defs: ########## @@ -21,5 +21,7 @@ ############################################################################ ifneq ($(CONFIG_EXAMPLES_I2SLOOP),) +ifneq ($(CONFIG_ENABLE_ALL_SIGNALS),) Review Comment: ditto -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
