Fix for #298 and workaround for Sybil 4 bug

[thirtytwobits]

#298 will be fixed but this will also get another refactor when we switch to CETL.

[thirtytwobits]

This PR is blocked by OpenCyphal/docker_toolchains#34

[thirtytwobits]

For the love of Pete! Is the entire Python universe broken!?! (working on a fix for RTD)

[pavel-kirienko]

Not sure if I get it correctly but CYPHAL_PATH contains the list of directories that contain DSDL root namespaces in them. That's how it's defined in PyCyphal et al. Is this compatible with the semantics of your env var here?

[thirtytwobits]

Not sure if I get it correctly but CYPHAL_PATH contains the list of directories that contain DSDL root namespaces in them. That's how it's defined in PyCyphal et al. Is this compatible with the semantics of your env var here?

I believe so? Look at test_CYPHAL_PATH and let me know.

[pavel-kirienko]

I believe so? Look at test_CYPHAL_PATH and let me know.

It seems like Nunavut expects the env vars to contain the list of root namespace directories, not the list of directories that contain root namespace directories in them. Observe:

outer/
    scotec/             <-- a DSDL root namespace
        Thing.1.0.dsdl
    ns/                 <-- a DSDL root namespace
        Foo.1.0.dsdl

CYPHAL_PATH would contain outer while your path variables would contain outer/scotec:outer/ns.

[thirtytwobits]

It seems like Nunavut expects the env vars to contain the list of root namespace directories,...

Undoing then. Can you open an issue to add support for CYPHAL_PATH?

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
3 Code Smells

100.0% Coverage
0.0% Duplication