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

Minor fixes for cmake unit test build #97

Closed
skliper opened this issue Sep 30, 2019 · 8 comments
Closed

Minor fixes for cmake unit test build #97

skliper opened this issue Sep 30, 2019 · 8 comments
Labels
Milestone

Comments

@skliper
Copy link
Contributor

skliper commented Sep 30, 2019

The CFE unit test build was assuming the use of "-pg --coverage" options for gcc. Most targets do support this for coverage analysis however some do not (RTEMS, for example) as the runtime component of this feature may not be present in the target's c library. Therefore the UT build currently fails for these targets.

Also, the OSAL build scripts have a feature to allow installation of the unit test executables to the target binary directory. This is based on the value of a cmake variable called "INSTALL_TARGET_LIST", but the current build scripts are not setting this variable.

This ticket will

  • Generate an appropriate value for INSTALL_TARGET_LIST
  • Use the UT_C_FLAGS as computed by the OSAL build
  • Add an "install" line for the CFE unit tests as well

These are all fairly minor changes and only affect unit test.

@skliper skliper added this to the 6.5.0 milestone Sep 30, 2019
@skliper skliper self-assigned this Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Imported from trac issue 66. Created by jphickey on 2015-06-08T23:28:30, last modified: 2019-03-05T14:57:55

@skliper skliper added the bug label Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2015-06-08 23:36:03:

Pushed commit [changeset:e99459e] for review

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2015-06-23 13:47:45:

NOTE - this changeset will also be rolled into #103 as a prerequisite for those changes.

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2015-07-09 16:47:14:

This is included in the {{{ic-2015-07-14}}} integration candidate for testing

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by glimes on 2015-07-13 14:32:40:

Accept.

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by glimes on 2015-07-16 12:41:50:

Included in [changeset:fe8f06c 2015-07-14 development merge],
now present on Babelfish and IPAS-LIB.

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by glimes on 2016-02-25 10:17:32:

these will be fixed in CFE 6.5

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jhageman on 2019-03-05 14:57:55:

Milestone renamed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant