nasa / fprime

F´ - A flight software and embedded systems framework
https://fprime.jpl.nasa.gov
Apache License 2.0
10.02k stars 1.29k forks source link

Add JSON dictionary generation to CMake system #2598

Closed thomas-bc closed 4 months ago

thomas-bc commented 5 months ago
Related Issue(s) https://github.com/nasa/fprime/issues/2591
Has Unit Tests (y/n)
Documentation Included (y/n)

Change Description

Address https://github.com/nasa/fprime/issues/2591

Opening as draft to start get some feedback to improve upon

thomas-bc commented 5 months ago

This branch requires these changes to FPP: https://github.com/jwest115/fpp/pull/1

thomas-bc commented 4 months ago

Rebased to catch latest devel, including fpp v2.1.0a7 with fpp-to-dict

thomas-bc commented 4 months ago

@LeStarch I finally managed to fix the bug on the RPI build by explicitly calling out the global version target as a dependency to the add_custom_command() for the dictionary... I have no clue why though, especially why it would only fail on that specific deployment and not others... 🤷‍♂️

This should be ready for review

LeStarch commented 4 months ago

@thomas-bc I bet this is because RPI does not use SystemResources and as such there is no dependence on version for the deployment.....except in the install step.