-
Notifications
You must be signed in to change notification settings - Fork 93
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
[nasa/nos3#445] - Move yamcs component definitions #480
Conversation
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.
After the cmake fixes I made, confirmed YAMCS works, and the CSS and EPS devices are fixed and work with cFS.
Looks like some of your updates to submodules broke that merge request here. Can you merge dev or any changes blocking the merge back into the generic_adcs modules (or others that may have broken as well) |
@rjbrown2 Can look, though I don't believe I touched ADCS. I touched EPS and CSS, and a couple of top level files (port number fixes). |
Ahh, sorry to blame you. I thought you had to go through and fix a couple of CMakes. Might just need Dev pulled into the module. If you didn't do it I'll look into it. |
No major changes to running. This includes a fix to EPS that was causing commanding to break.
Major changes here are just to move the XTCE files into their components instead of a huge directory in YAMCS. Will help to keep the mdb files up to date, and easier to find.
Currently seem to be some merge conflicts to work out first.