From af0efaf694a496dafcd828e553fb7c603cb57bb6 Mon Sep 17 00:00:00 2001 From: mahima-yoga Date: Thu, 23 Jul 2026 15:56:49 +0200 Subject: [PATCH] fix(commander): nack DO_PARACHUTE when the parachute module is not in the build The unconditional ignore left DO_PARACHUTE without any ack on builds without the parachute module. Only ignore the command when the module is there to ack it, otherwise fall back to the unsupported nack. Signed-off-by: mahima-yoga --- src/modules/commander/Commander.cpp | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/modules/commander/Commander.cpp b/src/modules/commander/Commander.cpp index fdf63083492..2ff3ff48975 100644 --- a/src/modules/commander/Commander.cpp +++ b/src/modules/commander/Commander.cpp @@ -1715,7 +1715,12 @@ Commander::handle_command(const vehicle_command_s &cmd) case vehicle_command_s::VEHICLE_CMD_REQUEST_MESSAGE: case vehicle_command_s::VEHICLE_CMD_DO_WINCH: case vehicle_command_s::VEHICLE_CMD_DO_GRIPPER: +#if defined(CONFIG_MODULES_PARACHUTE) + + // only ignore when the parachute module is there to ack it, otherwise fall through to the + // unsupported nack below case vehicle_command_s::VEHICLE_CMD_DO_PARACHUTE: +#endif // CONFIG_MODULES_PARACHUTE case vehicle_command_s::VEHICLE_CMD_EXTERNAL_POSITION_ESTIMATE: case vehicle_command_s::VEHICLE_CMD_REQUEST_CAMERA_INFORMATION: case vehicle_command_s::VEHICLE_CMD_EXTERNAL_ATTITUDE_ESTIMATE: