-
Notifications
You must be signed in to change notification settings - Fork 220
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 #357: Make parameter const #356
Conversation
When OS_INCLUDE_MODULE_LOADER is disabled, the function header does not match the definition.
Good catch. We are definitely planning to enhance our CI to catch more issues like this. There are multiple opportunities for improvement (testing on multiple platforms, range of configuration settings, etc). As a start see the matrix in nasa/cFS#40. Could you write a corresponding issue (and mark this pull as fixing it)? Our contribution guide is still in work, but the preference is that pull requests have an associated issue they are implementing/solving. Also, if you can let me know when the CLA is sent (email preferred) it'll expedite the process. Thanks for contributing! |
We should discuss, we might lend a helping hand.
Opened #357 .
Will push for it and will let you know. |
CCB 20200122 - Reviewed and approved for IC (pending CLA) |
Status update - We have received a CLA for cFE (18.128), but not OSAL (see https://github.com/nasa/osal/blob/master/doc/GSC_18370_Corp_CLA_form_1219.pdf). Pull request pending receipt of OSAL CLA. |
This is true. We have noticed that on our side already but a person who is supposed to sign the OSAL CCLA is not around for a few days. We will do it this week hopefully. |
CLA for OSAL received, thanks! |
@astrogeco Heads up this one is ready for merge to IC. |
Describe the contribution
When OS_INCLUDE_MODULE_LOADER is not defined, the function header does not
match the definition. This issue was introduced in PR #21 .
Longer term: is there a continuous integration strategy to avoid this sort of situation happening in the future?
Testing performed
Build with module loader disabled with cFS master distribution.
Expected behavior changes
None
System(s) tested on
Contributor Info
Andrei-Costin Zisu of Planetary Transportation Systems GmbH (Berlin, Germany). Company-wide CLA is in the process of being signed and should be available soon.