9737 2016-06-09 14:13:11 +0000 [PATCH] ui/knotty: give out start time of task 2016-09-27 10:17:01 +0000 1 1 7 Build System, Metadata & Runtime BitBake bitbake unspecified x86 Multiple RESOLVED FIXED Medium+ enhancement 2.2 M4 1 enrico.scholz richard.purdie alex.kanavin poky.bs.watcher poky.watcher --- --- No (bug/feature does not impact docs) oldest_to_newest 62811 0 3222 enrico.scholz 2016-06-09 14:13:11 +0000 Created attachment 3222 patch For long running tasks like compiling qt it might be interesting to know when the task has been started. Patch prints this information by using the %X modifier (locale dependent presentation of time without date). 66480 1 richard.purdie 2016-09-22 12:53:44 +0000 I like this idea but I think there is a better thing to do which is to show task elapsed time. If we display the start time, the next thing the user will manually do is look at the current time and compare so we may as well do it for them. I'll send such a patch to bitbake-devel. 66481 2 enrico.scholz 2016-09-22 12:58:11 +0000 Showing the elapsed time means that display must be refreshed periodically. 66547 3 richard.purdie 2016-09-23 14:11:55 +0000 (In reply to comment #2) > Showing the elapsed time means that display must be refreshed periodically. Which we already do due to the progress changes anyway so this doesn't really add that much overhead. I did rate limit it to 5s rather than 1s intervals to try and reduce the refresh problem. http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=0d76de85bd2fa8fc6b0a3a5a65f2817a5d0b2186 66618 4 alex.kanavin 2016-09-27 08:33:52 +0000 Sorry, have to reopen this. With the change, highlighting the terminal tab happens constantly, whereas previously it was highlighted only when a task has been finished. For people running long tasks in a edit-rib-debug cycle (i.e. me) that is an unpleasant regresssion, as now the terminal tab has to be polled to see if the task is done yet, whereas previously I could just glance at it while doing other stuff in a different tab. 66621 5 alex.kanavin 2016-09-27 10:17:01 +0000 Okay, I found that the terminal provides a workaround for this. Let's open a separate issue for UI notifications using d-bus. 3222 2016-06-09 14:13:11 +0000 2016-06-09 14:35:18 +0000 patch 0004-ui-knotty-give-out-start-time-of-task.patch text/plain 2069 enrico.scholz RnJvbSA3NzZjNzNmNmZjMTU5NTkyNzVhOWQ0ZTM4NzhmMmQ5MWNlMDg2NDFmIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBFbnJpY28gU2Nob2x6IDxlbnJpY28uc2Nob2x6QHNpZ21hLWNo ZW1uaXR6LmRlPgpEYXRlOiBXZWQsIDIyIEF1ZyAyMDEyIDE4OjMwOjAxICswMjAwClN1YmplY3Q6 IFtQQVRDSCAwNC8xMF0gdWkva25vdHR5OiBnaXZlIG91dCBzdGFydCB0aW1lIG9mIHRhc2sKCkZv ciBsb25nIHJ1bm5pbmcgdGFza3MgbGlrZSBjb21waWxpbmcgcXQgaXQgbWlnaHQgYmUgaW50ZXJl c3RpbmcgdG8ga25vdwp3aGVuIHRoZSB0YXNrIGhhcyBiZWVuIHN0YXJ0ZWQuICBQYXRjaCBwcmlu dHMgdGhpcyBpbmZvcm1hdGlvbiBieSB1c2luZwp0aGUgJVggbW9kaWZpZXIgKGxvY2FsZSBkZXBl bmRlbnQgcHJlc2VudGF0aW9uIG9mIHRpbWUgd2l0aG91dCBkYXRlKS4KClNpZ25lZC1vZmYtYnk6 IEVucmljbyBTY2hvbHogPGVucmljby5zY2hvbHpAc2lnbWEtY2hlbW5pdHouZGU+Ci0tLQogbGli L2JiL3VpL2tub3R0eS5weSAgIHwgMyArKy0KIGxpYi9iYi91aS91aWhlbHBlci5weSB8IDQgKysr LQogMiBmaWxlcyBjaGFuZ2VkLCA1IGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCgpkaWZm IC0tZ2l0IGEvbGliL2JiL3VpL2tub3R0eS5weSBiL2xpYi9iYi91aS9rbm90dHkucHkKaW5kZXgg OTYwNWM4ZS4uYTNlMmQ4OSAxMDA2NDQKLS0tIGEvbGliL2JiL3VpL2tub3R0eS5weQorKysgYi9s aWIvYmIvdWkva25vdHR5LnB5CkBAIC0yMDMsNyArMjAzLDggQEAgY2xhc3MgVGVybWluYWxGaWx0 ZXIob2JqZWN0KToKICAgICAgICAgICAgIHJldHVybgogICAgICAgICB0YXNrcyA9IFtdCiAgICAg ICAgIGZvciB0IGluIHJ1bm5pbmdwaWRzOgotICAgICAgICAgICAgdGFza3MuYXBwZW5kKCIlcyAo cGlkICVzKSIgJSAoYWN0aXZldGFza3NbdF1bInRpdGxlIl0sIHQpKQorICAgICAgICAgICAgdGFz a3MuYXBwZW5kKCIlcyAtICVzIChwaWQgJXMpIiAlCisgICAgICAgICAgICAgICAgICAgICAgICAg KGFjdGl2ZXRhc2tzW3RdWyJzdGFydGVkX3N0ciJdLCBhY3RpdmV0YXNrc1t0XVsidGl0bGUiXSwg dCkpCiAKICAgICAgICAgaWYgc2VsZi5tYWluLnNodXRkb3duOgogICAgICAgICAgICAgY29udGVu dCA9ICJXYWl0aW5nIGZvciAlcyBydW5uaW5nIHRhc2tzIHRvIGZpbmlzaDoiICUgbGVuKGFjdGl2 ZXRhc2tzKQpkaWZmIC0tZ2l0IGEvbGliL2JiL3VpL3VpaGVscGVyLnB5IGIvbGliL2JiL3VpL3Vp aGVscGVyLnB5CmluZGV4IGRiNzBiNzYuLjdmNDRmYTAgMTAwNjQ0Ci0tLSBhL2xpYi9iYi91aS91 aWhlbHBlci5weQorKysgYi9saWIvYmIvdWkvdWloZWxwZXIucHkKQEAgLTE4LDYgKzE4LDcgQEAK ICMgNTEgRnJhbmtsaW4gU3RyZWV0LCBGaWZ0aCBGbG9vciwgQm9zdG9uLCBNQSAwMjExMC0xMzAx IFVTQS4KIAogaW1wb3J0IGJiLmJ1aWxkCitpbXBvcnQgdGltZQogCiBjbGFzcyBCQlVJSGVscGVy OgogICAgIGRlZiBfX2luaXRfXyhzZWxmKToKQEAgLTMxLDcgKzMyLDggQEAgY2xhc3MgQkJVSUhl bHBlcjoKIAogICAgIGRlZiBldmVudEhhbmRsZXIoc2VsZiwgZXZlbnQpOgogICAgICAgICBpZiBp c2luc3RhbmNlKGV2ZW50LCBiYi5idWlsZC5UYXNrU3RhcnRlZCk6Ci0gICAgICAgICAgICBzZWxm LnJ1bm5pbmdfdGFza3NbZXZlbnQucGlkXSA9IHsgJ3RpdGxlJyA6ICIlcyAlcyIgJSAoZXZlbnQu X3BhY2thZ2UsIGV2ZW50Ll90YXNrKSB9CisgICAgICAgICAgICBzZWxmLnJ1bm5pbmdfdGFza3Nb ZXZlbnQucGlkXSA9IHsgJ3RpdGxlJyA6ICIlcyAlcyIgJSAoZXZlbnQuX3BhY2thZ2UsIGV2ZW50 Ll90YXNrKSwKKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAn c3RhcnRlZF9zdHInIDogdGltZS5zdHJmdGltZSgiJVgiKSB9CiAgICAgICAgICAgICBzZWxmLnJ1 bm5pbmdfcGlkcy5hcHBlbmQoZXZlbnQucGlkKQogICAgICAgICAgICAgc2VsZi5uZWVkVXBkYXRl ID0gVHJ1ZQogICAgICAgICBpZiBpc2luc3RhbmNlKGV2ZW50LCBiYi5idWlsZC5UYXNrU3VjY2Vl ZGVkKToKLS0gCjIuNS41Cgo=