Skip to content
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 dependencies #16

Closed
wants to merge 3 commits into from
Closed

Conversation

JerrickRowe
Copy link

The dependency on empy is broken. Resolve it by freezing the version of empy to 3.3.4. Other dependencies are also frozen at their current versions.
Some refinements to testing scripts have been made for better environmental compatibility.

@JerrickRowe JerrickRowe marked this pull request as draft December 12, 2023 11:04
@JerrickRowe JerrickRowe force-pushed the fix_dependencies branch 2 times, most recently from 37da35b to cb3e56a Compare December 12, 2023 11:53
@JerrickRowe JerrickRowe marked this pull request as ready for review December 12, 2023 12:03
@Valle125
Copy link

Valle125 commented Dec 15, 2023

@JerrickRowe Thank you very much, this saved my day!
Freezing the version of empy to 3.3.4 fixes the following error, when trying to compile dsdl messages

Caught exception! 'dict' object has no attribute 'unknownScriptName'

@dslc
Copy link

dslc commented Feb 8, 2024

Thanks for this suggestion.
Compilation seems to be completely broken with the latest version of the empy package.
Downgrading resolved the issue.

@tridge
Copy link
Member

tridge commented Jul 16, 2024

this conflicts with the existing change to empy version

@JerrickRowe
Copy link
Author

This PR is too old to be merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants