Skip to content

Add classic-diagnostic-adapter dependency with git override#240

Draft
opajonk wants to merge 2 commits into
mainfrom
integrate-opensovd-cda
Draft

Add classic-diagnostic-adapter dependency with git override#240
opajonk wants to merge 2 commits into
mainfrom
integrate-opensovd-cda

Conversation

@opajonk
Copy link
Copy Markdown
Contributor

@opajonk opajonk commented May 26, 2026

bazel build @classic-diagnostic-adapter//:opensovd-cda --experimental_isolated_extension_usages --@classic-diagnostic-adapter//:tls_backend=mbedtls

which passes for me right now like this:

INFO: Invocation ID: f0048ab0-8f71-4519-acac-5cddfdb24947
WARNING: For repository 'rules_shell', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'rules_cc', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'rules_rust', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'rules_python', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
INFO: Analyzed target @@classic-diagnostic-adapter+//:opensovd-cda (570 packages loaded, 34636 targets configured).
INFO: Found 1 target...
Target @@classic-diagnostic-adapter+//cda-main:opensovd-cda up-to-date:
  bazel-bin/external/classic-diagnostic-adapter+/cda-main/opensovd-cda
INFO: Elapsed time: 5.598s, Critical Path: 0.18s
INFO: 4 processes: 1427 action cache hit, 2 disk cache hit, 2 internal.
INFO: Build completed successfully, 4 total actions+

Note especially the patches mentioned in bazel_common/score_modules_target_sw.MODULE.bazel. They are required to make the build pass; for cross-compilation, two more patches are needed (commented out). Those are horrible patches which contains hard-coded, architecture-specific paths and can be considered PoC at best.

- Introduced a new bazel dependency for classic-diagnostic-adapter.
- Added git override for classic-diagnostic-adapter to specify the commit and remote repository.
@opajonk opajonk force-pushed the integrate-opensovd-cda branch from 36598f4 to cf1f465 Compare May 26, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant