-
Notifications
You must be signed in to change notification settings - Fork 208
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
Display extended version information from the build #44
Comments
Imported from trac issue 13. Created by jphickey on 2014-12-30T21:39:53, last modified: 2019-03-05T14:57:55 |
Trac comment by jphickey on 2015-01-05 10:52:44: Git commit [changeset:ae3b0f84] on branch "trac-13-extended_version_info" addresses this. |
Trac comment by jphickey on 2015-04-06 11:38:44: This is ready for review/merge |
Trac comment by sstrege on 2015-04-06 14:30:41: This change creates multiple definitions for the CFE_ES_INITSTATS_INF_EID |
Trac comment by jphickey on 2015-04-06 14:44:35: Are you referring to the fact that the init code is now performing two CFE_EVS_SendEvent() calls using the same CFE_ES_INITSTATS_INF_EID? If this will adversely affect code that is listening for these events, I could add a new EID for these. The only real reason for making two calls is that it was too much information to try to stuff into a single call. |
Trac comment by acudmore on 2015-04-07 08:54:37: Do the changes depend on Cmake and Git? |
Trac comment by sstrege on 2015-04-07 12:28:56: Yes, a new EID should be added. This supports the short event message that does not include the text string. |
Trac comment by glimes on 2015-04-07 12:52:33: Tested changeset [changeset:ae3b0f84] as part of the ic-2015-03-10 merge. |
Trac comment by jphickey on 2015-04-09 14:53:25: commit [changeset:545cc1d] has been added to address the concerns brought up in the CCB discussion on 4/7:
|
Trac comment by glimes on 2015-04-13 14:45:01: Confirmed, tested as changeset [changeset:545cc1d] |
Trac comment by glimes on 2015-04-13 14:46:00: Part of integration candidate 2015-03-10, |
Trac comment by glimes on 2016-02-16 13:16:45: Susie confirmed these tickets have been approved for CFE 6.5 |
Trac comment by jhageman on 2019-03-05 14:57:55: Milestone renamed |
The CFE build currently contains a 4-part version number, i.e. "6.4.1.0". This version number is is #define'd in a header file and is manually updated with each official release.
However, git and Cmake (see ticket #32) offer additional build information where the current git commitid and most recent tag name are built into a global object that is accessible at runtime. When it is available, this extended build information should be displayed along with the manually updated official version number.
The major benefit here is that it updates automatically with every commit, not just official releases. It also indicates the whether the source code tree has been modified from the pristine version in the commit ("dirty") or not. It is extremely useful to have this information built into the executable in order to verify the correct binary is loaded as well as being able to reproduce running binaries when needed.
The text was updated successfully, but these errors were encountered: