-
-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix for #298 and workaround for Sybil 4 bug #303
Conversation
This PR is blocked by OpenCyphal/docker_toolchains#34 |
For the love of Pete! Is the entire Python universe broken!?! (working on a fix for RTD) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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. |
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:
|
Undoing then. Can you open an issue to add support for CYPHAL_PATH? |
Kudos, SonarCloud Quality Gate passed! |
#298 will be fixed but this will also get another refactor when we switch to CETL.