-
Notifications
You must be signed in to change notification settings - Fork 224
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
CI - build multiple targets #95
Comments
In light of the conversation in nasa/cFE#758, this needs more discussion as to how it should be implemented. Some options
|
Not sure if this is one of the options you describe above, but my original thought was that .travis.yml would have a simple sed script that would uncomment the CPU2 entries in targets.cmake prior to build/test/etc. Probably the most minimal impact. |
Note the changes in nasa/cFE#776 to use name-based (rather than number-based) target definitions make this substantially easier to implement. Shouldn't be a need for sed scripts to hack the targets.cmake file if we go this route for future CFE versions. |
CCB 2020-09-02 Keep open, see implementation in SBN for inspiration. |
This bit indicates that the PDU has 64-bit size and offset fields. CF currently does not support large file sizes. It needs to reject these packets as they will corrupt the data because they are not decoded properly (decode is fixed at 32 bit sizes).
Fix nasa#95, reject PDUs with large bit set
Is your feature request related to a problem? Please describe.
The build process becomes increasingly more complex when building for multiple targets. CI should build for at least two targets to ensure that the build system is functioning properly.
Describe the solution you'd like
Have CI create relevant build configuration to build for multiple targets.
Requester Info
Christopher.D.Knight@nasa.gov
The text was updated successfully, but these errors were encountered: