Skip to content

fix(Build.py + CI): add --userdataobj flag; fix lcov path + coverage artifacts#64

Merged
kiritigowda merged 1 commit into
KhronosGroup:openvx_1.3from
simonCatBot:fix/ci-improvements-rustvx-clean
Jun 21, 2026
Merged

fix(Build.py + CI): add --userdataobj flag; fix lcov path + coverage artifacts#64
kiritigowda merged 1 commit into
KhronosGroup:openvx_1.3from
simonCatBot:fix/ci-improvements-rustvx-clean

Conversation

@simonCatBot

Copy link
Copy Markdown
Contributor

Summary

This PR fixes two issues that currently prevent the OpenVX CI workflow from running successfully:

1. Build.py missing --userdataobj flag (causes CI build failure)

The CI workflow added --userdataobj to the Build.py invocation, but Build.py does not recognize this argument, causing an immediate SystemExit: 2 (unknown option) failure.

Fix: Added --userdataobj option to Build.py that maps to -DOPENVX_USE_USER_DATA_OBJECT=ON.

2. Coverage collection broken in cts-baseline

Two related issues prevented lcov from successfully collecting coverage data:

  • Missing build artifacts: The build-debug job only uploaded install/ and include/, but lcov needs the build/Linux/x64/Debug/ tree containing .gcno and .gcda files.
  • Wrong lcov working directory: The cts-baseline job runs from inside build-cts/, so the relative path build/Linux/x64/Debug resolves incorrectly.

Fix:

  • Added build/Linux/x64/Debug/ to the openvx-debug artifact upload paths.
  • Changed lcov directory argument to absolute: ${{ github.workspace }}/build/Linux/x64/Debug.

Related

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Simon seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@simonCatBot simonCatBot force-pushed the fix/ci-improvements-rustvx-clean branch 4 times, most recently from ff7e621 to bae5b22 Compare June 21, 2026 08:15
@simonCatBot simonCatBot force-pushed the fix/ci-improvements-rustvx-clean branch from bae5b22 to ebb5f49 Compare June 21, 2026 17:24
@kiritigowda kiritigowda merged commit 27884a8 into KhronosGroup:openvx_1.3 Jun 21, 2026
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants