fix: Switch to compile-commands (#4879)
Fix SonarCloud analysis failures by switching the C/C++ configuration from the deprecated `sonar.cfamily.build-wrapper-output` property to the current `sonar.cfamily.compile-commands` property. **Changes** - Remove `sonar.cfamily.build-wrapper-output` from `sonar-project.properties`. - In `.github/workflows/build-steps.yml`, pass `--define sonar.cfamily.compile-commands="$BUILD_WRAPPER_OUT_DIR/compile_commands.json"`. **Root cause** The SonarCloud config still used the deprecated `sonar.cfamily.build-wrapper-output` property. The current CFamily analyzer expects a `compile_commands.json` (produced by build-wrapper). Using the old property caused scanner failures and blocked analysis. **Validation** Regenerated `bw_output/compile_commands.json`, ran `sonar-scanner`, and confirmed the scan completed and uploaded successfully. **Tests** CI-only change. Verified locally by running the scanner against a fresh build-wrapper capture. --------- Signed-off-by: Vanessa Valderrama <vvalderrama@linuxfoundation.org>
This commit is contained in:
@@ -186,8 +186,8 @@ jobs:
|
||||
ls -l /__w/OpenImageIO/OpenImageIO/bw_output
|
||||
echo "BUILD_OUTPUT_DIR is " "${{ env.BUILD_WRAPPER_OUT_DIR }}"
|
||||
find . -name "*.gcov" -print
|
||||
# sonar-scanner --define sonar.cfamily.build-wrapper-output="${{ env.BUILD_WRAPPER_OUT_DIR }}"
|
||||
time sonar-scanner --define sonar.host.url="${{ env.SONAR_SERVER_URL }}" --define sonar.cfamily.build-wrapper-output="$BUILD_WRAPPER_OUT_DIR" --define sonar.cfamily.gcov.reportsPath="_coverage" --define sonar.cfamily.threads="$PARALLEL"
|
||||
# sonar-scanner --define sonar.cfamily.compile-commands="${{ env.BUILD_WRAPPER_OUT_DIR }}/compile_commands.json"
|
||||
time sonar-scanner --define sonar.host.url="${{ env.SONAR_SERVER_URL }}" --define sonar.cfamily.compile-commands="$BUILD_WRAPPER_OUT_DIR/compile_commands.json" --define sonar.cfamily.gcov.reportsPath="_coverage" --define sonar.cfamily.threads="$PARALLEL"
|
||||
# Consult https://docs.sonarcloud.io/advanced-setup/ci-based-analysis/sonarscanner-cli/ for more information and options
|
||||
- name: Check ABI
|
||||
if: inputs.abi_check != ''
|
||||
|
||||
@@ -21,7 +21,6 @@ sonar.exclusions=src/doc/**,src/build-scripts/**,src/include/OpenImageIO/detail/
|
||||
sonar.sourceEncoding=UTF-8
|
||||
|
||||
# C/C++ analyzer properties
|
||||
sonar.cfamily.build-wrapper-output=/__w/OpenImageIO/OpenImageIO/bw_output
|
||||
sonar.cfamily.gcov.reportsPath=_coverage
|
||||
sonar.coverage.exclusions=src/iv/**,src/include/OpenImageIO/detail/pugixml/**,src/include/OpenImageIO/detail/fmt/**,src/libOpenImageIO/kissfft.hh
|
||||
|
||||
|
||||
Reference in New Issue
Block a user