Skip to content

Commit cda268a

Browse files
author
Developer
committed
fix: Improve GitHub workflow with better CTS build configuration
- Add proper OpenVX include paths to CMake configuration - Add debug output for build failures - Set OPENVX_LIBRARIES and OPENVX_DEFINITIONS correctly - Use VERBOSE=1 for make to aid debugging - Add error log capture on CMake failure This should fix CI failures where CTS build couldn't find OpenVX headers.
1 parent 5c6c3b6 commit cda268a

1 file changed

Lines changed: 18 additions & 3 deletions

File tree

.github/workflows/openvx-conformance.yml

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,19 +88,34 @@ jobs:
8888
exit 1
8989
fi
9090
91+
# Prepare include directory
92+
mkdir -p include
93+
if [ -d "../include" ]; then
94+
echo "Copying OpenVX headers..."
95+
cp -r ../include/* include/ 2>/dev/null || true
96+
fi
97+
9198
mkdir -p build
9299
cd build
93100
94-
# Configure with verbose output
101+
# Configure with verbose output and proper include paths
95102
echo "Running cmake..."
96-
cmake .. -DCMAKE_BUILD_TYPE=Release 2>&1 || {
103+
cmake .. \
104+
-DCMAKE_BUILD_TYPE=Release \
105+
-DOPENVX_INCLUDES="${{ github.workspace }}/include;${{ github.workspace }}/OpenVX-cts/include" \
106+
-DOPENVX_LIBRARIES="${{ github.workspace }}/target/release/libopenvx_core.so;${{ github.workspace }}/target/release/libopenvx_ffi.so" \
107+
-DOPENVX_DEFINITIONS="OPENVX_CONFORMANCE_VISION" \
108+
2>&1 || {
97109
echo "CMAKE FAILED!"
110+
echo "CMake output:"
111+
cat CMakeFiles/CMakeOutput.log 2>/dev/null | tail -100 || echo "No CMakeOutput.log"
112+
cat CMakeFiles/CMakeError.log 2>/dev/null | tail -100 || echo "No CMakeError.log"
98113
exit 1
99114
}
100115
101116
# Build
102117
echo "Running make..."
103-
make -j$(nproc) 2>&1 || {
118+
make VERBOSE=1 -j$(nproc) 2>&1 || {
104119
echo "MAKE FAILED!"
105120
exit 1
106121
}

0 commit comments

Comments
 (0)