From b81290a6d7936e23ea4b5ac89aa7b3427ae848ed Mon Sep 17 00:00:00 2001
From: k7g03z <88199521+k7g03z@users.noreply.github.com>
Date: Tue, 16 Aug 2022 08:17:26 +0200
Subject: [PATCH] Remove unnessary check (#553)

Signed-off-by: Kevin Goez <kevin.goez@apex.ai>

Signed-off-by: Kevin Goez <kevin.goez@apex.ai>
---
 scripts/build_firmware.sh     | 3 ++-
 scripts/configure_firmware.sh | 8 ++++----
 scripts/create_firmware_ws.sh | 2 +-
 scripts/flash_firmware.sh     | 2 +-
 4 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/scripts/build_firmware.sh b/scripts/build_firmware.sh
index a570b232..96a2c865 100755
--- a/scripts/build_firmware.sh
+++ b/scripts/build_firmware.sh
@@ -69,7 +69,8 @@ fi
 # Building specific firmware folder
 echo "Building firmware for $RTOS platform $PLATFORM"
 
-if [ $PLATFORM != "generic" ] && [ -d "$PREFIX/config/$RTOS/generic" ]; then
+# Use the generic platform if directory found
+if [ -d "$PREFIX/config/$RTOS/generic" ]; then
     . $PREFIX/config/$RTOS/generic/build.sh
 else
     . $PREFIX/config/$RTOS/$PLATFORM/build.sh
diff --git a/scripts/configure_firmware.sh b/scripts/configure_firmware.sh
index dca58e16..467710ea 100755
--- a/scripts/configure_firmware.sh
+++ b/scripts/configure_firmware.sh
@@ -1,4 +1,4 @@
-#! /bin/bash 
+#! /bin/bash
 
 set -e
 set -o nounset
@@ -17,7 +17,7 @@ else
 fi
 
 # Check if configure script exists
-if [ $PLATFORM != "generic" ] && [ -d "$PREFIX/config/$RTOS/generic" ]; then
+if [ -d "$PREFIX/config/$RTOS/generic" ]; then
     if [ ! -f $PREFIX/config/$RTOS/generic/configure.sh ]; then
         echo "No configuration step needed for generic platform $PLATFORM"
         exit 0
@@ -32,7 +32,7 @@ fi
 # Parsing micro-ROS arguments
 if [ $# -lt 1 ]; then
   echo "micro-ROS application name must be provided: ros2 run micro_ros_setup configure_firmware.sh [app name] [options]"
-  # Check if RTOS has app listing funcions and source in case 
+  # Check if RTOS has app listing funcions and source in case
   if [ -f $PREFIX/config/$RTOS/list_apps.sh ]; then
       . $PREFIX/config/$RTOS/list_apps.sh
       print_available_apps
@@ -80,7 +80,7 @@ while [[ $# -gt 0 ]]; do
 done
 
 # Configure specific firmware folder if needed
-if [ $PLATFORM != "generic" ] && [ -d "$PREFIX/config/$RTOS/generic" ]; then
+if [ -d "$PREFIX/config/$RTOS/generic" ]; then
     echo "Configuring firmware for $RTOS platform $PLATFORM"
     exec $PREFIX/config/$RTOS/generic/configure.sh $@
 else
diff --git a/scripts/create_firmware_ws.sh b/scripts/create_firmware_ws.sh
index 6a8659a8..4ec28bf9 100755
--- a/scripts/create_firmware_ws.sh
+++ b/scripts/create_firmware_ws.sh
@@ -76,7 +76,7 @@ fi
 SKIP="microxrcedds_agent microxrcedds_client microcdr rosidl_typesupport_connext_cpp rosidl_typesupport_connext_c rosidl_typesupport_opensplice_cpp rosidl_typesupport_opensplice_c rmw_opensplice_cpp ros-${ROS_DISTRO}-cyclonedds  ros-${ROS_DISTRO}-rti-connext-dds-cmake-module ros-${ROS_DISTRO}-rmw-connextdds-common ros-${ROS_DISTRO}-rmw-connextdds ros-${ROS_DISTRO}-rmw-cyclonedds-cpp google_benchmark_vendor performance_test_fixture ros-${ROS_DISTRO}-mimick-vendor rmw_cyclonedds_cpp rmw_connext_cpp rti-connext-dds-5.3.1 rmw_connextdds $EXTERNAL_SKIP"
 
 # Check generic build
-if [ $PLATFORM != "generic" ] && [ -d "$PREFIX/config/$RTOS/generic" ]; then
+if [ -d "$PREFIX/config/$RTOS/generic" ]; then
     TARGET_FOLDER=generic
 else
     TARGET_FOLDER=$PLATFORM
diff --git a/scripts/flash_firmware.sh b/scripts/flash_firmware.sh
index 23b080f2..15192dbd 100755
--- a/scripts/flash_firmware.sh
+++ b/scripts/flash_firmware.sh
@@ -17,7 +17,7 @@ else
 fi
 
 # Flash specific firmware folder if needed
-if [ $PLATFORM != "generic" ] && [ -d "$PREFIX/config/$RTOS/generic" ]; then
+if [ -d "$PREFIX/config/$RTOS/generic" ]; then
     if [ -f $PREFIX/config/$RTOS/generic/flash.sh ]; then
       echo "Flashing firmware for $RTOS platform $PLATFORM"
       . $PREFIX/config/$RTOS/generic/flash.sh