11772 2017-07-07 12:28:34 +0000 [PATCH] event: temporarily disable verbose logs 2021-11-11 17:17:28 +0000 1 1 7 Build System, Metadata & Runtime BitBake bitbake 2.5 x86 Multiple RESOLVED OBSOLETE Medium normal 4.99 1 enrico.scholz richard.purdie poky.bs.watcher poky.watcher --- --- No (bug/feature does not impact docs) oldest_to_newest 74861 0 3860 enrico.scholz 2017-07-07 12:28:34 +0000 Created attachment 3860 patch Because stderr of event handlers is directly printed to the console, this will pollute output when using e.g. 'buildhistory' which calls a shell function. This can be reproduced/demonstrated e.g. by setting BB_VERBOSE_LOGS = "1" INHERIT += "buildhistory" Patch disables 'set -x' temporarily while executing the event handler. 91883 1 richard.purdie 2021-11-03 13:06:05 +0000 Sorry this has taken so long to get back to, I do keep looking at it periodically and the world has changed a bit since the bug was opened. I think part of the confusion was the two different behaviours of verbose logs previously. The bitbake -v option would have things appear on the console, BB_VERBOSE_LOGS would have things appear in the logfiles. I generally read the summary of the bug as the "-v" option and this meant I didn't really see the issue of the event shell output on the console when there was already so much output. Since the bug is about BB_VERBOSE_LOGS, I think the issue was the event handler data appearing on the console when there wasn't anything else there? The good news is that bitbake's logging has changed in the intervening time. The buildhistory event handler output now goes to the bitbake-cookerdaemon log file which I think is an appropriate interpretation of the option. The console remains clear. Would you agree the patch is therefore now obsolete and not needed? 91979 2 richard.purdie 2021-11-11 17:17:28 +0000 No longer an issue as per Comment 1. 3860 2017-07-07 12:28:34 +0000 2017-07-07 12:28:34 +0000 patch 0005-event-temporarily-disable-verbose-logs.patch text/plain 1452 enrico.scholz RnJvbSA3MGJlMWZjMWQ2NDk4ZTQ4NzE1ZDllYmVmMzQ2ZTgzNmNhNWQ4NGUyIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBFbnJpY28gU2Nob2x6IDxlbnJpY28uc2Nob2x6QHNpZ21hLWNo ZW1uaXR6LmRlPgpEYXRlOiBUaHUsIDEzIERlYyAyMDEyIDE4OjExOjMzICswMTAwClN1YmplY3Q6 IFtQQVRDSCA1LzhdIGV2ZW50OiB0ZW1wb3JhcmlseSBkaXNhYmxlIHZlcmJvc2UgbG9ncwoKQmVj YXVzZSBzdGRlcnIgb2YgZXZlbnQgaGFuZGxlcnMgaXMgZGlyZWN0bHkgcHJpbnRlZCB0byB0aGUg Y29uc29sZSwKdGhpcyB3aWxsIHBvbGx1dGUgb3V0cHV0IHdoZW4gdXNpbmcgZS5nLiAnYnVpbGRo aXN0b3J5JyB3aGljaCBjYWxscyBhCnNoZWxsIGZ1bmN0aW9uLgoKVGhpcyBjYW4gYmUgcmVwcm9k dWNlZC9kZW1vbnN0cmF0ZWQgZS5nLiBieSBzZXR0aW5nCgogIEJCX1ZFUkJPU0VfTE9HUyA9ICIx IgogIElOSEVSSVQgKz0gImJ1aWxkaGlzdG9yeSIKClBhdGNoIGRpc2FibGVzICdzZXQgLXgnIHRl bXBvcmFyaWx5IHdoaWxlIGV4ZWN1dGluZyB0aGUgZXZlbnQgaGFuZGxlci4KClNpZ25lZC1vZmYt Ynk6IEVucmljbyBTY2hvbHogPGVucmljby5zY2hvbHpAc2lnbWEtY2hlbW5pdHouZGU+Ci0tLQog bGliL2JiL2V2ZW50LnB5IHwgMyArKysKIDEgZmlsZSBjaGFuZ2VkLCAzIGluc2VydGlvbnMoKykK CmRpZmYgLS1naXQgYS9saWIvYmIvZXZlbnQucHkgYi9saWIvYmIvZXZlbnQucHkKaW5kZXggNmQ4 NDkzYi4uODY0Y2MxZiAxMDA2NDQKLS0tIGEvbGliL2JiL2V2ZW50LnB5CisrKyBiL2xpYi9iYi9l dmVudC5weQpAQCAtMTAyLDcgKzEwMiw5IEBAIGRlZiBleGVjdXRlX2hhbmRsZXIobmFtZSwgaGFu ZGxlciwgZXZlbnQsIGQpOgogICAgIGlmICdkJyBub3QgaW4gYnVpbHRpbnM6CiAgICAgICAgIGJ1 aWx0aW5zWydkJ10gPSBkCiAgICAgICAgIGFkZGVkZCA9IFRydWUKKyAgICBvbGRfdmVyYm9zZSA9 IGJiLm1zZy5sb2dnZXJWZXJib3NlTG9ncwogICAgIHRyeToKKyAgICAgICAgYmIubXNnLmxvZ2dl clZlcmJvc2VMb2dzID0gRmFsc2UKICAgICAgICAgcmV0ID0gaGFuZGxlcihldmVudCkKICAgICBl eGNlcHQgKGJiLnBhcnNlLlNraXBSZWNpcGUsIGJiLkJCSGFuZGxlZEV4Y2VwdGlvbik6CiAgICAg ICAgIHJhaXNlCkBAIC0xMTYsNiArMTE4LDcgQEAgZGVmIGV4ZWN1dGVfaGFuZGxlcihuYW1lLCBo YW5kbGVyLCBldmVudCwgZCk6CiAgICAgICAgICAgICBsb2dnZXIuZXJyb3IoIkV4ZWN1dGlvbiBv ZiBldmVudCBoYW5kbGVyICclcycgZmFpbGVkIiAlIG5hbWUpCiAgICAgICAgIHJhaXNlCiAgICAg ZmluYWxseToKKyAgICAgICAgYmIubXNnLmxvZ2dlclZlcmJvc2VMb2dzID0gb2xkX3ZlcmJvc2UK ICAgICAgICAgZGVsIGV2ZW50LmRhdGEKICAgICAgICAgaWYgYWRkZWRkOgogICAgICAgICAgICAg ZGVsIGJ1aWx0aW5zWydkJ10KLS0KMi45LjQK