laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/16157 )
Change subject: fix DLSMS logging category color: '[1:38m' isn't actually defined ...................................................................... fix DLSMS logging category color: '[1:38m' isn't actually defined Instead it apparently renders as bright white, so just use that constant instead. Change-Id: Ic775b6e37ccf61dc71a540b41d6a16a8a9291dc2 --- M src/logging.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/src/logging.c b/src/logging.c index 7baa777..4d6224d 100644 --- a/src/logging.c +++ b/src/logging.c @@ -163,7 +163,7 @@ .name = "DLSMS", .description = "Layer3 Short Message Service (SMS)", .enabled = 1, .loglevel = LOGL_NOTICE, - .color = "\033[1;38m", + .color = OSMO_LOGCOLOR_BRIGHTWHITE, }, [INT2IDX(DLCTRL)] = { .name = "DLCTRL", -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/16157 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ic775b6e37ccf61dc71a540b41d6a16a8a9291dc2 Gerrit-Change-Number: 16157 Gerrit-PatchSet: 3 Gerrit-Owner: neels <nhofm...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <axilira...@gmail.com> Gerrit-Reviewer: laforge <lafo...@osmocom.org> Gerrit-Reviewer: pespin <pes...@sysmocom.de> Gerrit-MessageType: merged