-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
lib/ukdebug: Add LVLC_THREAD
macro
#929
lib/ukdebug: Add LVLC_THREAD
macro
#929
Conversation
If `LIBUKDEBUG_PRINT_THREAD` is selected in the configuration menu and `LIBUKDEBUG_ANSI_COLOR` is not, a build error is thrown because the `LVLC_THREAD` macro is not set. Fix this by setting the macro to an empty string if `LIBUKDEBUG_ANSI_COLOR` is not set. Signed-off-by: Stefan Jumarea <stefanjumarea02@gmail.com>
✅ Checkpatch passed Beep boop! I ran Unikraft's
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected!
Reviewed-by: Eduard-Florin Mihailescu mihailescu.eduard@gmail.com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good, what a pity @StefanJum found this first...
Reviewed-by: Maria Sfiraiala maria.sfiraiala@gmail.com
@razvand Let's merge this long awaited PR! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved-by: Razvan Deaconescu razvand@unikraft.io
If
LIBUKDEBUG_PRINT_THREAD
is selected in the configuration menu andLIBUKDEBUG_ANSI_COLOR
is not, a build error is thrown because theLVLC_THREAD
macro is not set.Fix this by setting the macro to an empty string if
LIBUKDEBUG_ANSI_COLOR
is not set.You can replicate the crash by enabling
ukdebug -> Show thread identifier
and making sureukdebug -> Colored output
is not selected in themenuconfig
.Prerequisite checklist
checkpatch.uk
on your commit series before opening this PR;