Skip to content

Commit 81de231

Browse files
authored
Remove quick build flag (#181)
1 parent b2226ec commit 81de231

File tree

2 files changed

+5
-23
lines changed

2 files changed

+5
-23
lines changed

.devcontainer/config/sailbot_workspace.code-workspace

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,7 @@
299299
"label": "Build All",
300300
"detail": "Build all packages in the workspace",
301301
"type": "shell",
302-
"command": "./build.sh ${input:quickBuild}",
302+
"command": "./build.sh",
303303
"group": {
304304
"kind": "build",
305305
"isDefault": true
@@ -310,7 +310,7 @@
310310
"label": "Build Package",
311311
"detail": "Build a specific package in the workspace",
312312
"type": "shell",
313-
"command": "./build.sh ${input:quickBuild} -p ${input:package}",
313+
"command": "./build.sh -p ${input:package}",
314314
"group": {
315315
"kind": "build",
316316
"isDefault": true
@@ -457,12 +457,6 @@
457457
"network_systems",
458458
]
459459
},
460-
{
461-
"id": "quickBuild",
462-
"type": "promptString",
463-
"description": "Build arguments: -q disables static analysis and unit tests for quicker builds",
464-
"default": "-q"
465-
},
466460
]
467461
}
468462
}

build.sh

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -16,35 +16,23 @@ function helpMessage() {
1616
# If still empty after argument parsing, all ROS packages are built
1717
PACKAGE=""
1818

19-
# Boolean for quick build (default is full build)
20-
QUICK_BUILD="false"
21-
2219
# Parse command-line options (all are optional arguments)
2320
while getopts "hp:q" flag; do
2421
case ${flag} in
2522
p ) PACKAGE="${OPTARG}" ;;
26-
q ) QUICK_BUILD="true" ;;
2723
h ) helpMessage; exit 0 ;;
2824
\? ) echo "Invalid option: -${flag}"; helpMessage; exit 1 ;;
2925
: ) echo "Option -${flag} requires an argument"; helpMessage; exit 1 ;;
3026
esac
3127
done
3228

33-
# Assign build type default
34-
BUILD_TYPE="RelWithDebInfo"
29+
# Assign build type debug
30+
BUILD_TYPE="Debug"
3531

3632
# Whether to run clang-tidy during build (unnecessary since we have separate CI and task)
3733
STATIC_ANALYSIS="OFF"
3834

39-
# Configuration for build (full or quick)
40-
if [[ $QUICK_BUILD = "true" ]]; then
41-
UNIT_TEST="OFF"
42-
else
43-
UNIT_TEST="ON"
44-
fi
45-
if [[ $UNIT_TEST = "ON" ]]; then
46-
BUILD_TYPE="Debug"
47-
fi
35+
UNIT_TEST="ON"
4836

4937
# Build ROS packages in src directory
5038
colcon build \

0 commit comments

Comments
 (0)