From bef3ef57d3f489073672e70a786c550df7edd003 Mon Sep 17 00:00:00 2001 From: Dario Gabriel Lipicar Date: Fri, 31 Jul 2026 01:19:27 -0300 Subject: [PATCH] test(doctests): override logos-lidl alongside every logos-cpp-sdk override 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 /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 --- doctests/cpp-sdk-module-composition.test.yaml | 14 ++++++++++++++ doctests/cpp-sdk-module-runtime.test.yaml | 10 ++++++++++ doctests/cpp-sdk-worker-thread-http.test.yaml | 14 ++++++++++++++ 3 files changed, 38 insertions(+) diff --git a/doctests/cpp-sdk-module-composition.test.yaml b/doctests/cpp-sdk-module-composition.test.yaml index ba9c3ed..69ff91e 100644 --- a/doctests/cpp-sdk-module-composition.test.yaml +++ b/doctests/cpp-sdk-module-composition.test.yaml @@ -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" diff --git a/doctests/cpp-sdk-module-runtime.test.yaml b/doctests/cpp-sdk-module-runtime.test.yaml index 5f1e039..1d5671e 100644 --- a/doctests/cpp-sdk-module-runtime.test.yaml +++ b/doctests/cpp-sdk-module-runtime.test.yaml @@ -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: diff --git a/doctests/cpp-sdk-worker-thread-http.test.yaml b/doctests/cpp-sdk-worker-thread-http.test.yaml index 7947877..a18e409 100644 --- a/doctests/cpp-sdk-worker-thread-http.test.yaml +++ b/doctests/cpp-sdk-worker-thread-http.test.yaml @@ -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"