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

Fix #357: Make parameter const #356

Merged
merged 1 commit into from
Mar 9, 2020
Merged

Conversation

matzipan
Copy link
Contributor

@matzipan matzipan commented Jan 22, 2020

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

  • PC, Ubuntu 19.04

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.

When OS_INCLUDE_MODULE_LOADER is disabled, the function header does not
match the definition.
@skliper
Copy link
Contributor

skliper commented Jan 22, 2020

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!

@skliper skliper added this to the 5.1.0 milestone Jan 22, 2020
@skliper skliper added the bug label Jan 22, 2020
@matzipan matzipan changed the title Fix OS_ModuleLoad_Impl when loader is disabled Fix #357. Make parameter const Jan 22, 2020
@matzipan matzipan changed the title Fix #357. Make parameter const Fix #357: Make parameter const Jan 22, 2020
@matzipan
Copy link
Contributor Author

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.

We should discuss, we might lend a helping hand.

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.

Opened #357 .

Also, if you can let me know when the CLA is sent (email preferred) it'll expedite the process.

Will push for it and will let you know.

@skliper
Copy link
Contributor

skliper commented Jan 22, 2020

CCB 20200122 - Reviewed and approved for IC (pending CLA)

@skliper skliper added CCB:PendingCLA External contribution pending CLA confirmation CCB:Approved Indicates code review and approval by community CCB labels Jan 22, 2020
@skliper skliper linked an issue Feb 6, 2020 that may be closed by this pull request
@skliper
Copy link
Contributor

skliper commented Mar 2, 2020

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.

@stanislaw
Copy link
Contributor

stanislaw commented Mar 2, 2020

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.

@stanislaw
Copy link
Contributor

stanislaw commented Mar 3, 2020

@skliper we have just signed the OSAL CCLA and sent it to to the GSFC-SoftwareRelease (Andrei Zisu and Stanislav Pankevich).

cc @matzipan

@skliper
Copy link
Contributor

skliper commented Mar 6, 2020

CLA for OSAL received, thanks!

@skliper skliper removed the CCB:PendingCLA External contribution pending CLA confirmation label Mar 6, 2020
@skliper
Copy link
Contributor

skliper commented Mar 6, 2020

@astrogeco Heads up this one is ready for merge to IC.

@astrogeco astrogeco changed the base branch from master to ic-20200226 March 9, 2020 15:13
@astrogeco astrogeco merged commit c53094f into nasa:ic-20200226 Mar 9, 2020
@astrogeco astrogeco mentioned this pull request Mar 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug CCB:Approved Indicates code review and approval by community CCB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OS_ModuleLoad_Impl function header does not match definition
4 participants