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

Update required tools check #170

Merged
merged 2 commits into from
Oct 10, 2023
Merged

Update required tools check #170

merged 2 commits into from
Oct 10, 2023

Conversation

thomas-bc
Copy link
Contributor

Originating Project/Creator
Affected Component
Affected Architectures(s)
Related Issue(s)
Has Unit Tests (y/n)
Builds Without Errors (y/n)
Unit Tests Pass (y/n)
Documentation Included (y/n)

Throwing this in now to not forget about it.

Change Description

Updates tool check list with regards to nasa/fprime#2293

Rationale

fprime-fpp is not a required package anymore, as it has been broken down in many subpackages

@thomas-bc
Copy link
Contributor Author

Should these only be

fpp-depend
fpp-to-xml found 
fpp-to-cpp found 
fpp-locate-defs

These seem to be the only ones the build system is checking for.

@bocchino
Copy link
Collaborator

bocchino commented Oct 9, 2023

fprime-util --check needs fpp-check.

I think it's good to provide all the tools, so users can use them if desired.

@thomas-bc thomas-bc merged commit f76cbf7 into nasa:devel Oct 10, 2023
28 checks passed
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.

None yet

3 participants