Skip to content

Commit fec0ca2

Browse files
committed
[ci] fixes for linux sh
1 parent 6b655b9 commit fec0ca2

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

.github/scripts/build-linux.sh

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@
66
# Uses BUILD_TYPE environment variable to create build and install directories
77
setup_directories() {
88
echo "Setting up build directories..."
9-
mkdir -p ./build_${BUILD_TYPE,,}
10-
mkdir -p ./build_as_3rdparty_${BUILD_TYPE,,}
11-
mkdir -p ../popsift_install_${BUILD_TYPE,,}
9+
mkdir -p ./build_"${BUILD_TYPE,,}"
10+
mkdir -p ./build_as_3rdparty_"${BUILD_TYPE,,}"
11+
mkdir -p ../popsift_install_"${BUILD_TYPE,,}"
1212
}
1313

1414
# Configures CMake for PopSift build with specified options
@@ -23,7 +23,7 @@ configure_cmake() {
2323
local install_dir="../popsift_install_${build_type,,}"
2424

2525
echo "Configuring CMake for $build_type..."
26-
cd "./$build_dir"
26+
cd "./$build_dir" || exit
2727
cmake .. \
2828
-DCMAKE_BUILD_TYPE="$build_type" \
2929
-DBUILD_SHARED_LIBS:BOOL=ON \
@@ -42,7 +42,7 @@ build_and_install() {
4242
local build_dir="build_${build_type,,}"
4343

4444
echo "Building and installing $build_type..."
45-
cd "./$build_dir"
45+
cd "./$build_dir" || exit
4646
cmake --build . --config "$build_type" --parallel
4747
cmake --install . --config "$build_type"
4848
cd ..
@@ -61,7 +61,7 @@ build_as_third_party() {
6161
local install_dir="../popsift_install_${build_type,,}"
6262

6363
echo "Testing third-party build for $build_type..."
64-
cd "./$build_dir"
64+
cd "./$build_dir" || exit
6565
cmake ../src/application \
6666
-DBUILD_SHARED_LIBS:BOOL=ON \
6767
-DCMAKE_BUILD_TYPE="$build_type" \

0 commit comments

Comments
 (0)