logos-co/logos-cpp-sdk
1
0
Fork 0
mirror of https://github.com/logos-co/logos-cpp-sdk.git synced 2026-08-29 00:31:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(doctests): override logos-lidl alongside every logos-cpp-sdk override

Browse Source 
The doc-tests build downstream repos (logoscore-cli, capability_module,
accounts_module) with --override-input logos-cpp-sdk. Nix does not carry the
overridden input's OWN lock, so those builds got this branch's cpp-sdk source
while still resolving logos-lidl from their own, older locks. The shipped
share/lidl-frontend/lidl_compat.h then calls accessors that lidl does not
have:

  lidl_compat.h:46: error: 'paramValueType' has not been declared in 'lidl'
  lidl_compat.h:92: error: 'fieldValueType' was not declared in this scope

Every --override-input logos-cpp-sdk now has a matching
--override-input <same-path>/logos-cpp-sdk/logos-lidl.

This is specific to the override path. A normal consumer running
'nix flake update logos-cpp-sdk' inherits cpp-sdk's own lock, which pins the
lidl carrying these accessors, and is unaffected.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Dario Gabriel Lipicar
2026-07-31 01:19:27 -03:00
co-authored by Claude Opus 5
parent cd94f88dd1
commit bef3ef57d3
3 changed files with 38 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
doctests/cpp-sdk-module-composition.test.yaml
+14
View File
@@ -539,12 +539,14 @@ sections:
nix build 'path:./greeter_module#lgx' \
--override-input logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o greeter-lgx
code_block: |
# From inside the greeter clone this is simply:
# nix build '.#lgx' --override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk'
nix build 'path:./greeter_module#lgx' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o greeter-lgx
post_text: "The greeter package is under `./greeter-lgx/`:"
extra_run:
@@ -562,13 +564,17 @@ sections:
--override-input greeter_module 'path:./greeter_module' \
--override-input logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input greeter_module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input greeter_module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o orchestrator-lgx
code_block: |
nix build 'path:./orchestrator_module#lgx' \
--override-input greeter_module 'path:./greeter_module' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input greeter_module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input greeter_module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o orchestrator-lgx
post_text: "The orchestrator package is under `./orchestrator-lgx/`:"
extra_run:
@@ -585,17 +591,25 @@ sections:
run: |
nix build 'github:logos-co/logos-logoscore-cli{release}' \
--override-input logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-liblogos/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-liblogos/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-module-client/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-module-client/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-capability-module/logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--out-link ./logos
code_block: |
nix build 'github:logos-co/logos-logoscore-cli' \
--override-input logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-liblogos/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-liblogos/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-module-client/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-module-client/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--out-link ./logos
check_file: "logos/bin/logoscore"
doctests/cpp-sdk-module-runtime.test.yaml
+10
View File
@@ -76,17 +76,25 @@ sections:
run: |
nix build 'github:logos-co/logos-logoscore-cli{release}' \
--override-input logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-liblogos/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-liblogos/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-module-client/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-module-client/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-capability-module/logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--out-link ./logos
code_block: |
nix build 'github:logos-co/logos-logoscore-cli' \
--override-input logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-liblogos/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-liblogos/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-module-client/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-module-client/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--out-link ./logos
check_file: "logos/bin/logoscore"
post_text: |
@@ -140,12 +148,14 @@ sections:
nix build 'path:./logos-accounts-module#lgx' \
--override-input logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o accounts-lgx
code_block: |
# From inside the clone this is simply:
# nix build '.#lgx' --override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk'
nix build 'path:./logos-accounts-module#lgx' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o accounts-lgx
post_text: "The `.lgx` package is now under `./accounts-lgx/`:"
extra_run:
doctests/cpp-sdk-worker-thread-http.test.yaml
+14
View File
@@ -412,10 +412,12 @@ sections:
nix build 'path:./sensor_module#lgx' \
--override-input logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o sensor-lgx
code_block: |
nix build 'path:./sensor_module#lgx' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o sensor-lgx
post_text: "The sensor package is under `./sensor-lgx/`:"
extra_run:
@@ -431,14 +433,18 @@ sections:
--override-input sensor_module 'path:./sensor_module' \
--override-input logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input sensor_module/logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input sensor_module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input sensor_module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o http-lgx
code_block: |
nix build 'path:./http_module#lgx' \
--override-input sensor_module 'path:./sensor_module' \
--override-input logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input sensor_module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input sensor_module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
-o http-lgx
post_text: "The http package is under `./http-lgx/`:"
extra_run:
@@ -454,17 +460,25 @@ sections:
run: |
nix build 'github:logos-co/logos-logoscore-cli{release}' \
--override-input logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-liblogos/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-liblogos/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-module-client/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-module-client/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-capability-module/logos-module-builder 'github:logos-co/logos-module-builder{release}' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk{release}' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--out-link ./logos
code_block: |
nix build 'github:logos-co/logos-logoscore-cli' \
--override-input logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-liblogos/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-liblogos/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-module-client/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-module-client/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk 'github:logos-co/logos-cpp-sdk' \
--override-input logos-capability-module/logos-module-builder/logos-cpp-sdk/logos-lidl 'github:logos-co/logos-lidl' \
--out-link ./logos
check_file: "logos/bin/logoscore"