Skip to content

BOOT_BANNER: show KERNEL_VERSION and BUILD_VERSION differently #16917

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

Merged

Conversation

marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Jun 19, 2019

Zephyr has to unrelated build _VERSIONs: KERNEL_VERSION and
BUILD_VERSION. Prefix them slightly differently in BOOT_BANNER so anyone
can instantly zoom in on which one is being used without having to
compare the implementation details of both.

Signed-off-by: Marc Herbert [email protected]

kernel/init.c Outdated
@@ -50,10 +50,10 @@ LOG_MODULE_REGISTER(os);
#endif

#ifdef BUILD_VERSION
#define BOOT_BANNER "Booting Zephyr OS " \
#define BOOT_BANNER "Booting Zephyr build " \
Copy link
Member

@nashif nashif Jun 19, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Booting Zephyr OS - Build/Version <blah>

Zephyr has two unrelated build _VERSIONs: KERNEL_VERSION and
BUILD_VERSION. Prefix them slightly differently in BOOT_BANNER so anyone
can instantly zoom in on which one is being used without having to
compare the implementation details of both.

Signed-off-by: Marc Herbert <[email protected]>
@marc-hb marc-hb force-pushed the show-different-zephyr-versions branch from 5a97627 to ee8c08d Compare June 19, 2019 18:00
@marc-hb marc-hb requested a review from nashif June 19, 2019 18:49
@nashif nashif merged commit d4df6ba into zephyrproject-rtos:master Jun 19, 2019
@marc-hb marc-hb deleted the show-different-zephyr-versions branch June 25, 2019 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Build System area: Kernel Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants